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

Merge downstream and upstream drivers for online download #1306

Merged
merged 4 commits into from
Aug 25, 2023

Conversation

Molter73
Copy link
Collaborator

@Molter73 Molter73 commented Aug 24, 2023

Description

Unify all downstream and upstream drivers for all archs in a single bucket so they can be downloaded by collector when running online.

Checklist

  • Investigated and inspected CI test results
  • Updated documentation accordingly

Testing Performed

  • Check the resulting merged bucket has the downstream built drivers

@openshift-ci
Copy link

openshift-ci bot commented Aug 24, 2023

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@Molter73 Molter73 force-pushed the mauro/merge-drivers branch 2 times, most recently from fca26aa to 0f9fbed Compare August 25, 2023 11:34
@Molter73 Molter73 marked this pull request as ready for review August 25, 2023 11:34
@Molter73 Molter73 requested a review from a team as a code owner August 25, 2023 11:34
shopt -s nullglob
shopt -s dotglob

# We use gsutil here because we need to ignore files that already
Copy link
Contributor

Choose a reason for hiding this comment

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

It would be nice to describe the overall schema of merging in plain text somewhere. A comment somewhere, or a new paragraph in the driver-build docs?

@Molter73 Molter73 merged commit dfa8b72 into master Aug 25, 2023
241 of 267 checks passed
@Molter73 Molter73 deleted the mauro/merge-drivers branch August 25, 2023 16:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants