Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WizardV2: enable V2 by default on preview and ephemeral #2081

Merged
merged 1 commit into from
May 23, 2024

Conversation

ezr-ondrej
Copy link
Contributor

This removes the need for a feature flag in Beta environments.

I see this as a natural first step - enable it in all Beta (including Ephemeral) environments without feature flag.
If we prefer to do default in Ephemeral only, let me know :)

@ezr-ondrej
Copy link
Contributor Author

@jrusz ^^ :)

@ezr-ondrej
Copy link
Contributor Author

/retest

1 similar comment
@ezr-ondrej
Copy link
Contributor Author

/retest

@ezr-ondrej ezr-ondrej force-pushed the default_v2 branch 3 times, most recently from 623b27e to e798349 Compare May 21, 2024 14:12
@ezr-ondrej
Copy link
Contributor Author

@jrusz I think that now it's failing because it is trying the old workflow :)
Would you look into that, what is the strategy for this?

@jrusz
Copy link
Collaborator

jrusz commented May 22, 2024

@jrusz I think that now it's failing because it is trying the old workflow :) Would you look into that, what is the strategy for this?

Yes I'm looking into it, I can see the new wizard in ephemeral by default but I'm not sure why it's failing yet, having some trouble running tests locally against ephemeral.

@jrusz
Copy link
Collaborator

jrusz commented May 22, 2024

/retest

@jrusz
Copy link
Collaborator

jrusz commented May 23, 2024

Package search doesn't work in ephemeral, not sure why. I'll skip adding packages in ephemeral for now to unblock this.

@jrusz
Copy link
Collaborator

jrusz commented May 23, 2024

/retest

1 similar comment
@jrusz
Copy link
Collaborator

jrusz commented May 23, 2024

/retest

Copy link
Collaborator

@regexowl regexowl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense to me, thanks!

@jrusz jrusz merged commit 4182c00 into osbuild:main May 23, 2024
5 checks passed
@ezr-ondrej ezr-ondrej deleted the default_v2 branch May 23, 2024 09:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants