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

feat(argo-cd): Add scrapeTimeout property to server and application… #2489

Merged
merged 1 commit into from Feb 12, 2024

Conversation

nikolay-o
Copy link
Contributor

I've added scrapeTimeout property to server and application, applicationSet controllers.

@nikolay-o nikolay-o changed the title feature(argo-cd) Add scrapeTimeout property to server and application… feat(argo-cd) Add scrapeTimeout property to server and application… Feb 9, 2024
@nikolay-o nikolay-o changed the title feat(argo-cd) Add scrapeTimeout property to server and application… feat(argo-cd): Add scrapeTimeout property to server and application… Feb 9, 2024
@nikolay-o nikolay-o force-pushed the feature/scrapeTimeout branch 2 times, most recently from bbea04c to cb277a0 Compare February 9, 2024 16:30
Comment on lines 29 to 30
- kind: changed
description: Improved documentation for various ingress setups
Copy link
Collaborator

Choose a reason for hiding this comment

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

Please remove old one.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed, thanks for review

Copy link
Collaborator

@yu-croco yu-croco left a comment

Choose a reason for hiding this comment

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

LGTM

charts/argo-cd/values.yaml Outdated Show resolved Hide resolved
charts/argo-cd/README.md Outdated Show resolved Hide resolved
@nikolay-o
Copy link
Contributor Author

@mkilchhofer I fixed documentation, could you please look?

…applicationset controllers.

Signed-off-by: Nikolay Osmolovskiy <nikolay.osmolovsky@gmail.com>
@mkilchhofer
Copy link
Member

@mkilchhofer I fixed documentation, could you please look?

Perfect, thank you so much for your contribution.
Lgtm👍

@mbevc1 mbevc1 merged commit b44f9a0 into argoproj:main Feb 12, 2024
6 checks passed
pdrastil added a commit to pdrastil/argo-helm that referenced this pull request Feb 13, 2024
* main:
  feat(argo-cd): Add scrapeTimeout property to server and application… (argoproj#2489)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants