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

fix: properly generate labels meta in the UI #50

Merged
merged 1 commit into from
Mar 4, 2024

Conversation

Unix4ever
Copy link
Member

Make the UI use proper data format for the machine labels meta value when generating the image on the DownloadInstallationMedia page.

Make the UI use proper data format for the machine labels meta value when generating the image on the
`DownloadInstallationMedia` page.

Signed-off-by: Artem Chernyshev <[email protected]>
@Unix4ever Unix4ever force-pushed the fix-labels-generation-ui branch from 6f492be to 47d8429 Compare March 4, 2024 13:36
@smira
Copy link
Member

smira commented Mar 4, 2024

Can we update the schematic at least in the integration test to include the labels and assert on them in the test?

@Unix4ever
Copy link
Member Author

Unix4ever commented Mar 4, 2024

Can we update the schematic at least in the integration test to include the labels and assert on them in the test?

Yeah, can do that.
Though it won't cover the UI.
And just changing the schematic ID won't do the trick I think, as we override the kernel args 🤔

@Unix4ever
Copy link
Member Author

/m

@talos-bot talos-bot merged commit 47d8429 into main Mar 4, 2024
18 checks passed
@talos-bot talos-bot deleted the fix-labels-generation-ui branch March 4, 2024 14:04
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.

4 participants