Skip to content

Update RHEL10 packages - #1935

Open
skyamgarp wants to merge 1 commit into
voxpupuli:4.xfrom
skyamgarp:RHEL-10-packages
Open

Update RHEL10 packages#1935
skyamgarp wants to merge 1 commit into
voxpupuli:4.xfrom
skyamgarp:RHEL-10-packages

Conversation

@skyamgarp

Copy link
Copy Markdown
Contributor

This is to use chrony as RHEL 10 failing with installing ntpdate

@skyamgarp

Copy link
Copy Markdown
Contributor Author

@bastelfreak Can you please review?

Comment thread lib/beaker/host_prebuilt_steps.rb Outdated
when /el-8/
RHEL8_PACKAGES
when /el-9/
when /el-(9|10)/

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Does it make sense to stay future proof and match on el9 and any newer version? With a bit of luck we won't have to touch it again on EL11.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Updated.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This is the 4.x branch. In master it's already future proof via 18e6e22, which was part of 6.4.0.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@bastelfreak , @ekohl could someone please take a look at this?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Do you actually need this in the beaker 4 branch? Can't you update to latest? It not, can you try to cherry-pick 18e6e22?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

#1853 had quite a few prerequisites to get the tests to pass so I'd be fine with updating this branch in a minimal way. However, I do agree people should upgrade to the latest Beaker branch.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Do you actually need this in the beaker 4 branch? Can't you update to latest? It not, can you try to cherry-pick 18e6e22?

Hi @bastelfreak , Yes we need this in 4.x. It might take some time to upgrade to beaker latest.

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.

4 participants