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

Update RAID array name after scheduling its creation #6185

Merged
merged 3 commits into from
Feb 26, 2025

Conversation

vojtechtrefny
Copy link
Contributor

Blivet can change the name for multiple reasons (invalid chars,
name conflict etc.) so we need to make sure we have both the old
and new name saved in case it is used in other kickstart commands
like volgroup for example.


We already do the same for the volgroup command in _execute_volgroup_data. This is currently blocked by related pykickstart fix for RAID names, see pykickstart/pykickstart#534

@jkonecny12
Copy link
Member

This is unblocked as our newest container builds are failing on missing newest pykickstart version:
https://github.com/rhinstaller/anaconda/actions/runs/13534784020/job/37824326102

@jkonecny12 jkonecny12 removed the blocked Don't merge this pull request! label Feb 26, 2025
@jkonecny12
Copy link
Member

/kickstart-test --testtype smoke

@jkonecny12
Copy link
Member

The unit test failure is expected because our workflow will not build new container with a new pykickstart.
Tested locally.

@jkonecny12
Copy link
Member

/packit build

@KKoukiou
Copy link
Contributor

Merging and doing directly a container rebuild.

@KKoukiou
Copy link
Contributor

Actually let's put the change in the spec file, and let unit tests build a new container here.

vojtechtrefny and others added 3 commits February 26, 2025 13:09
Blivet can change the name for multiple reasons (invalid chars,
name conflict etc.) so we need to make sure we have both the old
and new name saved in case it is used in other kickstart commands
like volgroup for example.
@KKoukiou
Copy link
Contributor

/kickstart-tests --waive run already smoke tests

@KKoukiou
Copy link
Contributor

Ignore Web UI tests, and RPM tests, known issues.

@KKoukiou KKoukiou merged commit 12046bc into rhinstaller:main Feb 26, 2025
15 of 19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

3 participants