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: 1.29 Do not try to test this branch on Fedora rawhide #3396

Merged
merged 2 commits into from
May 22, 2024

Conversation

jirihnidek
Copy link
Contributor

  • This branch is intended only for RHEL 9. It does not make sense to try to test this for Fedora rawhide, which contains too many things not compatible with RHEL 9

@cnsnyder cnsnyder requested review from a team and ptoscano and removed request for a team May 20, 2024 15:16
@ptoscano
Copy link
Contributor

Please drop also "fedora latest", as it is in the same situation as fedora rawhide.

@jirihnidek
Copy link
Contributor Author

Please drop also "fedora latest", as it is in the same situation as fedora rawhide.

It is not in the same situation, but we will be there sooner or later... So, I will drop it too.

I would like to solve failing unit tests first.

@m-horky
Copy link
Contributor

m-horky commented May 21, 2024

Is this PR supposed to be targeting 1.29 instead of main? Also, seems like 1.29.40 needs the same patch.

@ptoscano
Copy link
Contributor

I would like to solve failing unit tests first.

#3405 did it.

@ptoscano
Copy link
Contributor

Also, once done please backport it to the subscription-manager-1.29.40 branch as well.

* This branch is intended only for RHEL 9. It does not make
  sense to try to test this for Fedora rawhide, which contains
  too many things not compatible with RHEL 9
@jirihnidek jirihnidek force-pushed the jhnidek/1.29_fix_build branch from f4d57c5 to a5c4f33 Compare May 22, 2024 08:19
@jirihnidek jirihnidek changed the base branch from main to subscription-manager-1.29 May 22, 2024 08:21
* This branch is intended only for RHEL 9. It does not make
  sense to try to test this for Fedora latest, because
  this branch becomes incompatible with the latest Fedora
  sooner or later.
@jirihnidek jirihnidek force-pushed the jhnidek/1.29_fix_build branch from a5c4f33 to 3510d47 Compare May 22, 2024 08:26
Copy link
Contributor

@m-horky m-horky left a comment

Choose a reason for hiding this comment

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

LGTM

@m-horky m-horky merged commit f1138bd into subscription-manager-1.29 May 22, 2024
4 checks passed
@m-horky m-horky deleted the jhnidek/1.29_fix_build branch May 22, 2024 08:40
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