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.

Sign up for GitHub

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

Jump to bottom

Add blog-post about downtream workflow #404

Merged
merged 8 commits into from
May 6, 2022
Merged

Add blog-post about downtream workflow #404

merged 8 commits into from
May 6, 2022

Conversation

lachmanfrantisek
Copy link
Member

@lachmanfrantisek lachmanfrantisek commented Mar 2, 2022 •
edited
Loading

Signed-off-by: Frantisek Lachman flachman@redhat.com
Fixes: #403

TODO:

@lachmanfrantisek lachmanfrantisek mentioned this pull request Mar 2, 2022
Closed
3 tasks
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

@lachmanfrantisek lachmanfrantisek force-pushed the blog-about-downstream-workflow branch 2 times, most recently from 4e79377 to f45f740 Compare March 15, 2022 08:02
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

jpopelka
jpopelka reviewed Mar 15, 2022
View reviewed changes
content/posts/downstream-automation.md Outdated Show resolved Hide resolved
content/posts/downstream-automation.md Outdated Show resolved Hide resolved
content/posts/downstream-automation.md Outdated Show resolved Hide resolved
@softwarefactory-project-zuul
Copy link
Contributor

Build failed.

@lachmanfrantisek lachmanfrantisek mentioned this pull request Mar 15, 2022
Merged
@lachmanfrantisek lachmanfrantisek force-pushed the blog-about-downstream-workflow branch from 514c4a7 to 2e8bc09 Compare March 31, 2022 07:26
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

@lachmanfrantisek lachmanfrantisek force-pushed the blog-about-downstream-workflow branch from 2e8bc09 to 5df6f95 Compare March 31, 2022 08:57
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

@lachmanfrantisek lachmanfrantisek changed the title WIP: Add blog-post about downtream workflow Add blog-post about downtream workflow Mar 31, 2022
@lachmanfrantisek lachmanfrantisek marked this pull request as ready for review March 31, 2022 10:04
@lachmanfrantisek lachmanfrantisek added the blocked We are blocked! label Mar 31, 2022
@lachmanfrantisek
Copy link
Member Author

Please review, but we agreed on merging this once we announce this in other channels as well.

@lachmanfrantisek
Copy link
Member Author

Screenshot from 2022-03-31 12-24-12
Screenshot from 2022-03-31 12-24-27
Screenshot from 2022-03-31 12-24-44
Screenshot from 2022-03-31 12-25-01
Screenshot from 2022-03-31 12-25-10

@jpopelka
Copy link
Member

jpopelka commented Apr 1, 2022

My comments are marked as resolved, but I don't see any change in the wording.

@lachmanfrantisek
Copy link
Member Author

lachmanfrantisek commented Apr 1, 2022 •
edited
Loading

My comments are marked as resolved, but I don't see any change in the wording.

Ah, I see This suggestion has been applied or marked resolved.. I probably applied the suggestion and forgot to pull that before the next (force) push...

Thanks for checking that!

edit: Should be fixed now

@lachmanfrantisek lachmanfrantisek force-pushed the blog-about-downstream-workflow branch from 5df6f95 to e7ae6d5 Compare April 1, 2022 13:07
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

@lachmanfrantisek lachmanfrantisek changed the title Add blog-post about downtream workflow WAIT-WITH-MERGING-FOR-ANNOUNCEMENT: Add blog-post about downtream workflow Apr 1, 2022
@lachmanfrantisek lachmanfrantisek mentioned this pull request Apr 11, 2022
Merged
4 tasks
@lachmanfrantisek lachmanfrantisek force-pushed the blog-about-downstream-workflow branch from e7ae6d5 to 77c0814 Compare May 3, 2022 12:10
@lachmanfrantisek
Copy link
Member Author

@TomasTomecek @jpopelka @mfocko Can you please take another look? (I've rebased and added a few more commits on top of the original content.)

@softwarefactory-project-zuul

This comment was marked as outdated.

Sign in to view
@lachmanfrantisek

This comment was marked as outdated.

Sign in to view
@softwarefactory-project-zuul

This comment was marked as outdated.

Sign in to view
@lachmanfrantisek
Copy link
Member Author

recheck

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

mfocko
mfocko approved these changes May 4, 2022
View reviewed changes
Copy link
Member

@mfocko mfocko left a comment

Choose a reason for hiding this comment

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

nitpicks, otherwise lgtm :)

content/docs/guide.md Outdated Show resolved Hide resolved
content/docs/guide.md Outdated Show resolved Hide resolved
content/posts/downstream-automation.md Outdated Show resolved Hide resolved
content/posts/downstream-automation.md Show resolved Hide resolved
content/posts/downstream-automation.md Outdated Show resolved Hide resolved
content/posts/downstream-automation.md Outdated Show resolved Hide resolved
content/posts/downstream-automation.md Outdated Show resolved Hide resolved
@lachmanfrantisek
Copy link
Member Author

nitpicks, otherwise lgtm :)

@mfocko, thanks a lot for all the suggestions!

lachmanfrantisek and others added 8 commits May 6, 2022 09:58
@lachmanfrantisek
Add blog-post about downtream workflow
573127b 
Signed-off-by: Frantisek Lachman <flachman@redhat.com>
@lachmanfrantisek
Mention in blog and config that Bodhi update requires commit rights
160f2a3 
Signed-off-by: Frantisek Lachman <flachman@redhat.com>
@lachmanfrantisek
Mark bodhi updates as available
bc4ad52 
Signed-off-by: Frantisek Lachman <flachman@redhat.com>
@lachmanfrantisek
Cover downstream jobs in the onboarding guide
45e072d 
Signed-off-by: Frantisek Lachman <flachman@redhat.com>
@lachmanfrantisek
Use the correct alias for Bodhi updates
9a448e7 
Signed-off-by: Frantisek Lachman <flachman@redhat.com>
@lachmanfrantisek @mfocko
Apply suggestions from code review by @mfocko
07dea02 
Co-authored-by: Matej Focko <mfocko@users.noreply.github.com>
@lachmanfrantisek @mfocko
Update content/docs/guide.md by @mfocko
01d345a 
Co-authored-by: Matej Focko <mfocko@users.noreply.github.com>
@lachmanfrantisek
Option create_pr is available only for CLI
cd67a42 
Signed-off-by: Frantisek Lachman <flachman@redhat.com>
@lachmanfrantisek lachmanfrantisek force-pushed the blog-about-downstream-workflow branch from 20d49a3 to cd67a42 Compare May 6, 2022 07:58
@lachmanfrantisek lachmanfrantisek changed the title WAIT-WITH-MERGING-FOR-ANNOUNCEMENT: Add blog-post about downtream workflow Add blog-post about downtream workflow May 6, 2022
@lachmanfrantisek lachmanfrantisek added mergeit When set, zuul wil gate and merge the PR. and removed blocked We are blocked! labels May 6, 2022
@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded.

✔️ pre-commit SUCCESS in 2m 19s
✔️ hugo SUCCESS in 1m 15s

@softwarefactory-project-zuul
Copy link
Contributor

Build succeeded (gate pipeline).

✔️ pre-commit SUCCESS in 2m 03s

@softwarefactory-project-zuul softwarefactory-project-zuul bot merged commit d51c5a6 into packit:main May 6, 2022
@lachmanfrantisek lachmanfrantisek deleted the blog-about-downstream-workflow branch May 6, 2022 08:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mfocko mfocko mfocko approved these changes

@TomasTomecek TomasTomecek TomasTomecek approved these changes

@jpopelka jpopelka jpopelka approved these changes

Assignees
No one assigned
Labels
mergeit When set, zuul wil gate and merge the PR.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@lachmanfrantisek @jpopelka @TomasTomecek @mfocko