Skip to content

Update docs for argo cd steps#3046

Merged
eddymoulton merged 10 commits intomainfrom
em/update-docs-for-argo-cd-steps
Mar 5, 2026
Merged

Update docs for argo cd steps#3046
eddymoulton merged 10 commits intomainfrom
em/update-docs-for-argo-cd-steps

Conversation

@eddymoulton
Copy link
Contributor

@eddymoulton eddymoulton commented Mar 3, 2026

Adds information about new Argo CD integration features such as:

  • Trigger sync
  • Verify Application Healthy
  • Wait for PRs to merge

Also does a broad cleanup on all the Argo CD docs here, fixing up the writing style and some structuring.

Relates to MD-1557

@eddymoulton eddymoulton changed the title Em/update docs for argo cd steps Update docs for argo cd steps Mar 3, 2026
@eddymoulton eddymoulton marked this pull request as ready for review March 4, 2026 02:57
Copy link
Contributor

@rain-on rain-on left a comment

Choose a reason for hiding this comment

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

nits only 👍

:::

If left open, the installation dialog waits for the gateway to establish a connection and run a health check. Once successful, the Octopus Argo CD gateway is ready for use!
If left open, the installation dialog waits for the gateway to establish a connection and run a health check. Once successful, the Octopus Argo CD gateway is ready for use.
Copy link
Contributor

Choose a reason for hiding this comment

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

this one made me sad - the exclamation point was an emotional high point of this page. But I will accept your change 👍

If the commit summary or description references a [Sensitive Variable](/docs/projects/variables/sensitive-variables) the deployment wil fail.
This ensures sensitive data is not leaked to Git via the commit/PR message.
:::
### Purge Argo CD Source Folder
Copy link
Contributor

Choose a reason for hiding this comment

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

? Not sure why this is under "Git Commit Settings" as a heading

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It matches the other page 😅

Probably a lot for a single option.

3. Container images can be defined, but are optional. These let you attach automated processes, such as external feed triggers, to the project.

The output section allows you to configure how changes are to be merged into your repository.
## Git Commit Settings
Copy link
Contributor

Choose a reason for hiding this comment

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

I haven't seen it - but is it worth a statement of "for generic step settings, please see "? eg commit-settings, trigger, verification etc.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Is the next line any good for this?

@eddymoulton eddymoulton merged commit fdde092 into main Mar 5, 2026
5 checks passed
@eddymoulton eddymoulton deleted the em/update-docs-for-argo-cd-steps branch March 5, 2026 03:01
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.

2 participants