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

feat(HMS-1772): Allow selecting custom resource group #346

Merged
merged 1 commit into from
Oct 17, 2023

Conversation

ezr-ondrej
Copy link
Member

@ezr-ondrej ezr-ondrej commented Oct 5, 2023

Screencast.from.2023-10-05.20-27-33.webm

Simple and straightforward I hope :)

UI part for RHEnVision/provisioning-backend#694
Update your backend env or test with stage! :)

const queryClient = new QueryClient({
logger: {
log: console.log,
warn: console.warn,
// ✅ no more errors on the console
error: () => {},
},
defaultOptions: {
queries: {
retry: queryRetries,
Copy link
Member Author

Choose a reason for hiding this comment

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

This is good practice in tests to have easier time to test for failures.

@ezr-ondrej ezr-ondrej force-pushed the azure_resource_group branch from ac8e7ce to 7df98f6 Compare October 5, 2023 18:30
@lzap
Copy link
Member

lzap commented Oct 6, 2023

Nice.

@akhil-jha
Copy link
Member

Ah, this would require wizard changes in QE tests since we have a new dropdown.

@ezr-ondrej
Copy link
Member Author

/retest

1 similar comment
@ezr-ondrej
Copy link
Member Author

/retest

@akhil-jha
Copy link
Member

@ezr-ondrej Can please you rebase this? Thanks!

@ezr-ondrej ezr-ondrej force-pushed the azure_resource_group branch from 7df98f6 to 5eb2d8b Compare October 17, 2023 09:35
@ezr-ondrej
Copy link
Member Author

@ezr-ondrej Can please you rebase this? Thanks!

rebased! :) I think that only now we will run the Azure tests, right?

@ezr-ondrej
Copy link
Member Author

I'll go ahead and self merge, I promise to fix any issues that may arrise, but at least now we've tested this with Azure tests on :)

@ezr-ondrej ezr-ondrej merged commit 3e73a67 into RHEnVision:main Oct 17, 2023
@ezr-ondrej ezr-ondrej deleted the azure_resource_group branch October 17, 2023 10:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants