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

Add Overview of a CI/CD workflow with Zuul article #1157

Merged
merged 1 commit into from Jan 24, 2018

Conversation

TristanCacqueray
Copy link
Contributor

No description provided.

@yac
Copy link

yac commented Jan 22, 2018

good read 👍

Copy link
Contributor

@morucci morucci left a comment

Choose a reason for hiding this comment

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

Nice article, just some comments.


In this example, the depends-on list includes three changes:
* Pages roles added to zuul-jobs-distgit,
* Pages jobs configured in sf-cofnig, and
Copy link
Contributor

Choose a reason for hiding this comment

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

typo

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

deployment pipelines with powerfull cross repository dependencies management.

This article presented the Software Factory workflow where rpm packages are
being continously built, tested and delivered through code review.
Copy link
Contributor

Choose a reason for hiding this comment

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

Line 404, 405 could also be put at the beginning of the article.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done, thanks for the review Fabien!

This article presents some mechanisms to create such pipelines using
a rpm package distribution development workflow as a practical example.

To experiment with the configuration sample,
Copy link
Contributor

Choose a reason for hiding this comment

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

This paragraph could have been placed at the end. Like "experiment Zuul by yourself"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

@rbowen rbowen merged commit 11188b3 into redhat-openstack:master Jan 24, 2018
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.

None yet

4 participants