Skip to content

docs: managedFieldsManagers example in docs needs double-quotes#14324

Merged
crenshaw-dev merged 1 commit intoargoproj:masterfrom
pgpx:patch-1
Jul 6, 2023
Merged

docs: managedFieldsManagers example in docs needs double-quotes#14324
crenshaw-dev merged 1 commit intoargoproj:masterfrom
pgpx:patch-1

Conversation

@pgpx
Copy link
Contributor

@pgpx pgpx commented Jul 3, 2023

Fixes #14323

On https://argo-cd.readthedocs.io/en/stable/user-guide/diffing/#application-level-configuration the managedFieldsManagers example should have double-quoted the * wildcards (e.g. "*") because * is a YAML special character, and without this the examples cannot be parsed by ArgoCD (raising an error).

* is a Yaml special character and must be quoted, otherwise ArgoCD fails to parse and shows an error.

Signed-off-by: Paul Martin <paul.martin@gmail.com>
@pgpx pgpx changed the title fix(docs): Fix quotes for ignoreDifferences example fix(docs): managedFieldsManagers example in docs needs double-quotes Jul 3, 2023
@codecov
Copy link

codecov bot commented Jul 3, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (d7632df) 49.76% compared to head (e37d36b) 49.76%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master   #14324   +/-   ##
=======================================
  Coverage   49.76%   49.76%           
=======================================
  Files         261      261           
  Lines       44676    44676           
=======================================
  Hits        22232    22232           
  Misses      20257    20257           
  Partials     2187     2187           

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

Copy link
Member

@ishitasequeira ishitasequeira left a comment

Choose a reason for hiding this comment

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

Thanks @pgpx , Looks good!!

@blakepettersson blakepettersson added the ready-for-review An approver should give a final review and merge the PR label Jul 6, 2023
Copy link
Member

@crenshaw-dev crenshaw-dev left a comment

Choose a reason for hiding this comment

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

Great catch, thank you @pgpx!

@crenshaw-dev crenshaw-dev changed the title fix(docs): managedFieldsManagers example in docs needs double-quotes docs: managedFieldsManagers example in docs needs double-quotes Jul 6, 2023
@crenshaw-dev
Copy link
Member

/cherry-pick release-2.8

@crenshaw-dev
Copy link
Member

/cherry-pick release-2.7

@crenshaw-dev
Copy link
Member

/cherry-pick release-2.6

@crenshaw-dev
Copy link
Member

/cherry-pick release-2.5

@crenshaw-dev crenshaw-dev merged commit ef8dae8 into argoproj:master Jul 6, 2023
gcp-cherry-pick-bot bot pushed a commit that referenced this pull request Jul 6, 2023
* is a Yaml special character and must be quoted, otherwise ArgoCD fails to parse and shows an error.

Signed-off-by: Paul Martin <paul.martin@gmail.com>
gcp-cherry-pick-bot bot pushed a commit that referenced this pull request Jul 6, 2023
* is a Yaml special character and must be quoted, otherwise ArgoCD fails to parse and shows an error.

Signed-off-by: Paul Martin <paul.martin@gmail.com>
gcp-cherry-pick-bot bot pushed a commit that referenced this pull request Jul 6, 2023
* is a Yaml special character and must be quoted, otherwise ArgoCD fails to parse and shows an error.

Signed-off-by: Paul Martin <paul.martin@gmail.com>
gcp-cherry-pick-bot bot pushed a commit that referenced this pull request Jul 6, 2023
* is a Yaml special character and must be quoted, otherwise ArgoCD fails to parse and shows an error.

Signed-off-by: Paul Martin <paul.martin@gmail.com>
crenshaw-dev pushed a commit that referenced this pull request Jul 6, 2023
…) (#14371)

* is a Yaml special character and must be quoted, otherwise ArgoCD fails to parse and shows an error.

Signed-off-by: Paul Martin <paul.martin@gmail.com>
Co-authored-by: Paul Martin <paul.martin@gmail.com>
crenshaw-dev pushed a commit that referenced this pull request Jul 6, 2023
…) (#14372)

* is a Yaml special character and must be quoted, otherwise ArgoCD fails to parse and shows an error.

Signed-off-by: Paul Martin <paul.martin@gmail.com>
Co-authored-by: Paul Martin <paul.martin@gmail.com>
crenshaw-dev pushed a commit that referenced this pull request Jul 6, 2023
…) (#14374)

* is a Yaml special character and must be quoted, otherwise ArgoCD fails to parse and shows an error.

Signed-off-by: Paul Martin <paul.martin@gmail.com>
Co-authored-by: Paul Martin <paul.martin@gmail.com>
crenshaw-dev pushed a commit that referenced this pull request Jul 6, 2023
…) (#14373)

* is a Yaml special character and must be quoted, otherwise ArgoCD fails to parse and shows an error.

Signed-off-by: Paul Martin <paul.martin@gmail.com>
Co-authored-by: Paul Martin <paul.martin@gmail.com>
schakrad pushed a commit to schakrad/argo-cd that referenced this pull request Jul 24, 2023
…proj#14324) (argoproj#14372)

* is a Yaml special character and must be quoted, otherwise ArgoCD fails to parse and shows an error.

Signed-off-by: Paul Martin <paul.martin@gmail.com>
Co-authored-by: Paul Martin <paul.martin@gmail.com>
Signed-off-by: schakrad <58915923+schakrad@users.noreply.github.com>
yyzxw pushed a commit to yyzxw/argo-cd that referenced this pull request Aug 9, 2023
…proj#14324)

* is a Yaml special character and must be quoted, otherwise ArgoCD fails to parse and shows an error.

Signed-off-by: Paul Martin <paul.martin@gmail.com>
@suzaku suzaku mentioned this pull request Aug 15, 2023
tesla59 pushed a commit to tesla59/argo-cd that referenced this pull request Dec 16, 2023
…proj#14324)

* is a Yaml special character and must be quoted, otherwise ArgoCD fails to parse and shows an error.

Signed-off-by: Paul Martin <paul.martin@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-for-review An approver should give a final review and merge the PR

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

managedFieldsManagers example in docs needs double-quotes

4 participants