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

EL9 yolo #9990

Closed
wants to merge 2 commits into from
Closed

EL9 yolo #9990

wants to merge 2 commits into from

Conversation

evgeni
Copy link
Member

@evgeni evgeni commented Nov 16, 2023

No description provided.

- name: "rhel-{{ rhel_9 }}-x86_64"
external_repos:
- "https://yum.puppet.com/puppet7/el/{{ rhel_9 }}/x86_64/"
#- "{{ foreman_staging }}/rhel-{{ rhel_9 }}-x86_64"
Copy link
Member Author

Choose a reason for hiding this comment

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

there is no staging repo yet for el9

Copy link
Member

Choose a reason for hiding this comment

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

What happens if you go ahead and define it here? As soon as you run this to create the EL9 chroot, this staging repository will exist (since that chroot will trigger Copr to create it).

Copy link
Member Author

Choose a reason for hiding this comment

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

Within this PR it would fail until someone with the right permissions calls obal copr-project … to create it.

But yes, this is the plan, I just wanted to see if things otherwise build OK.

- "https://yum.puppet.com/puppet7/el/{{ rhel_9 }}/x86_64/"
#- "{{ foreman_staging }}/rhel-{{ rhel_9 }}-x86_64"
comps_file: "{{ inventory_dir }}/comps/comps-foreman-el{{ rhel_9 }}.xml"
#buildroot_packages: "{{ core_buildroot_packages }}"
Copy link
Member Author

Choose a reason for hiding this comment

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

there is no foreman-build for el9 yet.

Copy link
Member

Choose a reason for hiding this comment

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

Does this break builds if we define the package to be included and it does not exist?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, mock freaks out it can't install things and aborts.

@ehelms
Copy link
Member

ehelms commented Nov 20, 2023

Something to consider, since we are bootstrapping for the first time in a while, and we know we will bootstrap again for EL10. For Pulp I introduced tiers in the package_manifest to represent the order in which things must get built to make the bootstrapping easier to repeat:

https://github.com/theforeman/pulpcore-packaging/blob/rpm/3.28/package_manifest.yaml#L36

@evgeni
Copy link
Member Author

evgeni commented Dec 1, 2023

this was a playground, #10015 was the final one we merged

@evgeni evgeni closed this Dec 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants