-
Notifications
You must be signed in to change notification settings - Fork 349
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
pykickstart: Use RHEL10 kickstart commands #5712
Conversation
This switches the following commands to use the RHEL10 version: network nvdimm timezone It also bumps the pykickstart requirement to version 3.52.4 which contains the new commands. Changes in this version are: timezone: Remove the --isUtc, --nontp, and --ntpserver arguments packages: Remove the old camel case arguments on RHEL10 nvdimm: Remove support for the nvdimm command on RHEL10 network: Deprecate network team options on RHEL 10 Related: RHEL-36831
This requires pykickstart 3.52.4 to be built first. See pykickstart/pykickstart#494 |
@bcl please inform us when pykickstart is build and ready in RHEL-10 |
The new pykickstart is now in the nightly AppStream repo so this can be merged. |
/kickstart-test --testtype smoke |
Removed the blocked label, since pykickstart is ready |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me as well, thanks! :)
Hi all. This PR needs more work to fix the tests (or the code?). First, currently container builds are broken which is resolved by #5735 but that PR is failing because of new pykickstart version. If you build container in #5735 and test these changes you will see unit tests issues which needs to be resolved.
Three tests are failing. I was looking on the Here we are getting into a troubles with how to approach it. The test seems to be designed that modules shouldn't support / process kickstart commands which are removed from the code base. So how should we approach the
There are also other tests testing BTRFS functionality. Another thing to consider is if we want to get this into Until this is resolved and merged we won't get a new containers in quay.io for rhel-10 tag. |
Merged as part of #5740 |
This switches the following commands to use the RHEL10 version:
network
nvdimm
timezone
It also bumps the pykickstart requirement to version 3.52.4 which contains the new commands. Changes in this version are:
timezone: Remove the --isUtc, --nontp, and --ntpserver arguments
packages: Remove the old camel case arguments on RHEL10
nvdimm: Remove support for the nvdimm command on RHEL10
network: Deprecate network team options on RHEL 10
Related: RHEL-36831