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
Update workflow to add new dependencies #1224
Conversation
According to new automation based on rdoinfo.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
CBS but not pushed to any RDO repository. | ||
- `el7-build` (only available for Rocky and newer releases) | ||
is assigned to packages that only required to build other | ||
packages but not to deploy or run any OpenStack service. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
but are not a runtime requirement for any other package (it looks a bit confusing).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
|
||
If the package is found for the required CBS tag, it's already in RDO repositories | ||
and no more actions are needed to add it to the repos. | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We may want to add a simple flow to explain the flow:
-candidate -> -testing -> -release
-candidate -> -el7-build
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's kind of explained in the workflow https://github.com/amoralej/website/blob/d1bde77d996e0aa390db357e5cd95b234a1f0088/source/images/new-dependencies.png
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I've added the flow in second patch.
create a gerrit review as explained in step 5. | ||
|
||
<br /> | ||
7. Once the package is rebuilt in CBS (review in step 5 is merged) you can pushed |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
you can push it (typo in pushed)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
fixed
According to feedback in the PR.
Thanks! |
According to new automation based on rdoinfo.