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

fix: add reference doc for Extra Config (#858) #921

Merged
merged 1 commit into from
Jun 8, 2023

Conversation

mortya
Copy link
Contributor

@mortya mortya commented May 31, 2023

What type of PR is this?

/kind documentation

What does this PR do / why we need it:

The reference for argocd does not include extraConfig. I spent a while looking for something like that. Issue #858 implies I'm not the only one.

Have you updated the necessary documentation?

  • Documentation update is required by this PR.
  • Documentation has been updated.

Which issue(s) this PR fixes:

Fixes #858

Signed-off-by: Morty Abzug <morty@frakir.org>
@jaideepr97
Copy link
Collaborator

jaideepr97 commented Jun 1, 2023

hi @mortya

This is already documented here: https://github.com/argoproj-labs/argocd-operator/blob/master/docs/usage/extra-config.md and is accessible through the side-nav bar on readthedocs

@mortya
Copy link
Contributor Author

mortya commented Jun 2, 2023

Thanks, @jaideepr97. I did eventually find that other doc. But it wasn't in the reference for the actual argocd CRD, which is where I initially looked. Normally that sort of type reference lists all features of the type at a basic level, even if they are also (better) documented elsewhere. So it should document extraConfig.

Copy link
Collaborator

@jannfis jannfis left a comment

Choose a reason for hiding this comment

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

LGTM! Thank you!

@jannfis jannfis merged commit d90af83 into argoproj-labs:master Jun 8, 2023
7 checks passed
ciiay pushed a commit to ciiay/argocd-operator that referenced this pull request Jun 8, 2023
…rgoproj-labs#921)

Signed-off-by: Morty Abzug <morty@frakir.org>
Co-authored-by: Morty Abzug <morty@frakir.org>
Signed-off-by: Yi Cai <yicai@redhat.com>
ciiay pushed a commit to ciiay/argocd-operator that referenced this pull request Jun 8, 2023
…rgoproj-labs#921)

Signed-off-by: Morty Abzug <morty@frakir.org>
Co-authored-by: Morty Abzug <morty@frakir.org>
Signed-off-by: Yi Cai <yicai@redhat.com>
ciiay pushed a commit to ciiay/argocd-operator that referenced this pull request Jun 20, 2023
…rgoproj-labs#921)

Signed-off-by: Morty Abzug <morty@frakir.org>
Co-authored-by: Morty Abzug <morty@frakir.org>
Signed-off-by: Yi Cai <yicai@redhat.com>
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.

.spec.extraConfig Missing
3 participants