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

Create documentation for Argo Rollouts Plugin. #20680

Merged
merged 8 commits into from
Jun 24, 2024

Conversation

thisisnotashwin
Copy link
Contributor

@thisisnotashwin thisisnotashwin commented Feb 20, 2024

Description

VERCEL LINK: https://consul-31z9s3g9y-hashicorp.vercel.app/consul/docs/k8s/deployment-configurations/argo-rollouts-configuration

Testing & Reproduction steps

Links

PR Checklist

  • updated test coverage
  • external facing docs updated
  • appropriate backport labels added
  • not a security concern

@thisisnotashwin thisisnotashwin added the type/docs Documentation needs to be created/updated/clarified label Feb 20, 2024
@thisisnotashwin thisisnotashwin force-pushed the ashwin-michael/add-docs-for-rollouts-plugin branch 2 times, most recently from 0cf4d1a to 2d4056b Compare February 20, 2024 20:59
@wilkermichael wilkermichael added the pr/no-changelog PR does not need a corresponding .changelog entry label Feb 20, 2024
@thisisnotashwin thisisnotashwin force-pushed the ashwin-michael/add-docs-for-rollouts-plugin branch from 2d4056b to a76a917 Compare February 21, 2024 14:53
@thisisnotashwin thisisnotashwin marked this pull request as ready for review February 21, 2024 15:01
@thisisnotashwin thisisnotashwin requested a review from a team as a code owner February 21, 2024 15:01
@thisisnotashwin thisisnotashwin added backport/1.15 This release series is no longer active on CE. Use backport/ent/1.15. backport/1.16 This release series is no longer active on CE. Use backport/ent/1.16. backport/1.17 This release series is no longer active on CE. Use backport/ent/1.17. backport/1.18 labels Feb 21, 2024
@thisisnotashwin thisisnotashwin force-pushed the ashwin-michael/add-docs-for-rollouts-plugin branch 2 times, most recently from f77ea18 to 4f48ba9 Compare February 21, 2024 15:17
Copy link
Contributor

@boruszak boruszak left a comment

Choose a reason for hiding this comment

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

I left suggestions to match the documentation style guide and page layouts. Approving so that you're not blocked, but there's a number of big structural changes, so please don't hesitate to reach out if you need additional review on this PR!

Approving on behalf of consul-docs

@thisisnotashwin thisisnotashwin force-pushed the ashwin-michael/add-docs-for-rollouts-plugin branch from fec02a6 to 5a2105c Compare February 21, 2024 19:35
Copy link
Contributor

@boruszak boruszak left a comment

Choose a reason for hiding this comment

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

LGTM! Just some really minor spacing/capitalization/wording suggestions to apply.

Copy link
Contributor

@wilkermichael wilkermichael left a comment

Choose a reason for hiding this comment

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

Just adding some fixes to the links

@david-yu david-yu removed the backport/1.15 This release series is no longer active on CE. Use backport/ent/1.15. label Mar 9, 2024
Copy link

github-actions bot commented May 9, 2024

This pull request has been automatically flagged for inactivity because it has not been acted upon in the last 60 days. It will be closed if no new activity occurs in the next 30 days. Please feel free to re-open to resurrect the change if you feel this has happened by mistake. Thank you for your contributions.

@github-actions github-actions bot added the meta/stale Automatically flagged for inactivity by stalebot label May 9, 2024
@github-actions github-actions bot removed the meta/stale Automatically flagged for inactivity by stalebot label May 29, 2024
@boruszak
Copy link
Contributor

boruszak commented Jun 7, 2024

@blake Should we merge these docs for the upcoming release? Argo rollouts were listed on your recent roadmap

@boruszak boruszak added backport/1.19 Changes are backported to 1.19 and removed backport/1.16 This release series is no longer active on CE. Use backport/ent/1.16. backport/1.17 This release series is no longer active on CE. Use backport/ent/1.17. labels Jun 11, 2024
@boruszak
Copy link
Contributor

Update: These docs are complete, but we should not publish until Argo Rollouts v1.7 is released. Currently they're only in the release candidate: https://github.com/argoproj/argo-rollouts/releases

@boruszak boruszak mentioned this pull request Jun 11, 2024
4 tasks
@wilkermichael wilkermichael merged commit 596a4cd into main Jun 24, 2024
97 checks passed
@wilkermichael wilkermichael deleted the ashwin-michael/add-docs-for-rollouts-plugin branch June 24, 2024 23:32
danielehc pushed a commit that referenced this pull request Jun 25, 2024
* Create documentation for Argo Rollouts Plugin.

* Create documentation for Argo Rollouts Plugin.

* Apply suggestions from code review

Co-authored-by: David Yu <[email protected]>

* Apply suggestions from code review

Co-authored-by: Jeff Boruszak <[email protected]>

* Update docs based on feedback

* Apply suggestions from code review

Co-authored-by: Jeff Boruszak <[email protected]>

* Update website/content/docs/k8s/deployment-configurations/argo-rollouts-configuration.mdx

* Update website/content/docs/k8s/deployment-configurations/argo-rollouts-configuration.mdx

---------

Co-authored-by: David Yu <[email protected]>
Co-authored-by: Jeff Boruszak <[email protected]>
Co-authored-by: Michael Wilkerson <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/1.19 Changes are backported to 1.19 pr/no-changelog PR does not need a corresponding .changelog entry type/docs Documentation needs to be created/updated/clarified
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants