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

1924126: Fix profile upload on AWS systems #2556

Merged
merged 1 commit into from Jul 15, 2021
Merged

1924126: Fix profile upload on AWS systems #2556

merged 1 commit into from Jul 15, 2021

Conversation

jirihnidek
Copy link
Contributor

@jirihnidek jirihnidek commented Apr 7, 2021

@cnsnyder cnsnyder requested review from a team and wottop and removed request for a team April 7, 2021 14:06
@wottop
Copy link
Member

wottop commented Apr 7, 2021

Would like to see the result of the CI tests.

@jirihnidek jirihnidek changed the title 1924126: Fix profile upload on AWS systems WIP: 1924126: Fix profile upload on AWS systems Apr 8, 2021
@jirihnidek jirihnidek force-pushed the jhnidek/ent-3448 branch 3 times, most recently from e6048c4 to 005f9ad Compare April 30, 2021 14:53
@jirihnidek jirihnidek changed the base branch from jhnidek/ent-3448 to main July 1, 2021 15:25
@jirihnidek jirihnidek changed the title WIP: 1924126: Fix profile upload on AWS systems 1924126: Fix profile upload on AWS systems Jul 1, 2021
* BZ: https://bugzilla.redhat.com/show_bug.cgi?id=1924126
* Card ID: ENT-3583
* Fix AWS RHUI repo url, when data for profile are gathered.
  Replace REGION with value provided in AWS metadata
@jirihnidek
Copy link
Contributor Author

How to test this?

  1. Create scratch build
    $ tito build --srpm --test
    $ brew build --scratch --arch-override=x86_64 rhel-8.5.0-candidate /path/to/sub-man.src.rpm
  2. Install generated rpms on VM runnong on AWS
  3. Register system against production candlepin or stage candlepin
  4. run: dnf uploadprofile

@cnsnyder
Copy link
Member

@jirihnidek Is this still in progress? I'll be glad to review/test but don't want to do it before it's ready. If it is ready, please take off the "in progress" label, I'll check back early tomorrow :)

@jirihnidek
Copy link
Contributor Author

@cnsnyder Sorry for confusing you. It is not in progress. I only forgot to remove this WIP label.

@cnsnyder cnsnyder requested review from cnsnyder and removed request for wottop July 15, 2021 12:52
:return: Yields RHUI repositories
"""
for repo_name, repo in base.repos.items():
# TODO: we need more reliable mechanism for detection of RHUI repository, because
Copy link
Member

Choose a reason for hiding this comment

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

Is this captured somewhere besides this comment?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

We can create card for this task.

@cnsnyder cnsnyder merged commit dceb6bc into main Jul 15, 2021
@jirihnidek jirihnidek deleted the jhnidek/1924126 branch July 15, 2021 13:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants