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

Changes #21

Closed
wants to merge 1,424 commits into from
Closed

Changes #21

wants to merge 1,424 commits into from

Conversation

NagendraOpsmx
Copy link
Collaborator

Note on DCO:

If the DCO action in the integration test fails, one or more of your commits are not signed off. Please click on the Details link next to the DCO action for instructions on how to resolve this.

Checklist:

  • Either (a) I've created an enhancement proposal and discussed it with the community, (b) this is a bug fix, or (c) this does not need to be in the release notes.
  • The title of the PR states what changed and the related issues number (used for the release note).
  • I've included "Closes [ISSUE #]" or "Fixes [ISSUE #]" in the description to automatically close the associated issue.
  • I've updated both the CLI and UI to expose my feature, or I plan to submit a second PR with them.
  • Does this PR require documentation updates?
  • I've updated documentation as required by this PR.
  • Optional. My organization is added to USERS.md.
  • I have signed off all my commits as required by DCO
  • I have written unit and/or e2e tests for my change. PRs without these are unlikely to be merged.
  • My build is green (troubleshooting builds).

maheshkasabe and others added 30 commits January 9, 2024 21:05
* fix: allow to run codegen outside GOPATH

Signed-off-by: Alexandre Gaudreault <[email protected]>

* clientgen

Signed-off-by: Alexandre Gaudreault <[email protected]>

* openapigen

Signed-off-by: Alexandre Gaudreault <[email protected]>

* remove ensure-gopath

Signed-off-by: Alexandre Gaudreault <[email protected]>

---------

Signed-off-by: Alexandre Gaudreault <[email protected]>
* feat(sharding): use a cache

Signed-off-by: Alexandre Gaudreault <[email protected]>

* cluster cmd

Signed-off-by: Alexandre Gaudreault <[email protected]>

* - Assign shard 0 to in-cluster cluster and nil check updates
- Caching clusters while sharding: Fixing unit tests
- Update generated docs
- Debug e2e tests
- Default the shardNumber to the number of replicas if it is calculated to a higher value
- defered Unlock only when a lock is set
- Disabling temporarly other versions of k3s to check if e2e passes
- Do not fail if hostname format is not abc-n
- Fix unit test and skip some e2e
- Skip TestGitSubmoduleHTTPSSupport test
- Remove breaking defer c.lock.Unlock()
- Reverting testing all k3s version
- Default sharding fix
Signed-off-by: Akram Ben Aissi <[email protected]>

Signed-off-by: Akram Ben Aissi <[email protected]>

* fixes related to code review: renaming structure param, moving db initialisation

Signed-off-by: Akram Ben Aissi <[email protected]>

* Code review

Signed-off-by: Akram Ben Aissi <[email protected]>

* Set default shard to 0

Signed-off-by: Akram Ben Aissi <[email protected]>

* Set different default value for Sts and Deployment mode

Signed-off-by: Akram Ben Aissi <[email protected]>

* Expose ClusterShardingCache

Signed-off-by: Akram Ben Aissi <[email protected]>

* Removing use of argoDB.db for DistributionFunction

Signed-off-by: Akram Ben Aissi <[email protected]>

* Update generated documentation

Signed-off-by: Akram Ben Aissi <[email protected]>

* Fix comment about NoShardingDistributionFunction and NoShardingAlgorithm

Signed-off-by: Akram Ben Aissi <[email protected]>

---------

Signed-off-by: Alexandre Gaudreault <[email protected]>
Signed-off-by: Akram Ben Aissi <[email protected]>
Co-authored-by: Alexandre Gaudreault <[email protected]>
…-rbac/kustomization.yaml. (#16810)

* fix(manifests): applicationset-controller dir is not added to cluster-rbac/kustomization.yaml.

Related PR: #16699.

I missed adding a new folder(applicationset-controller) to kustomization.yaml.

So, i addressed it.

Signed-off-by: mugioka <[email protected]>

* chore: exec `make manifests`.

Signed-off-by: mugioka <[email protected]>

* chore: exec `make manifests`.

Signed-off-by: mugioka <[email protected]>

---------

Signed-off-by: mugioka <[email protected]>
…rver when adding a EKS cluster (#16767)

useful for argocd-servers which are not running in AWS and want to add multiple EKS clusters using
separate keys instead of assuming roles

#16766

Signed-off-by: Isaac Gaskin <[email protected]>
Co-authored-by: Soumya Ghosh Dastidar <[email protected]>
…16812) (#16816)

Use fully qualified application names in ApplicationWaitCommand

Closes: #16812

Signed-off-by: Sergiy Kulanov <[email protected]>
* Update Azure AD to Entra ID

https://learn.microsoft.com/en-us/entra/fundamentals/new-name

Signed-off-by: Ryan Flynn <[email protected]>

* Add formerly known as azuread

Signed-off-by: Ryan Flynn <[email protected]>

---------

Signed-off-by: Ryan Flynn <[email protected]>
* Preventing runnings jobs when updating documentation

Signed-off-by: Aymen Ben Tanfous <[email protected]>

* Empty line added to .md file

Signed-off-by: Aymen Ben Tanfous <[email protected]>

---------

Signed-off-by: Aymen Ben Tanfous <[email protected]>
Co-authored-by: Aymen Ben Tanfous <[email protected]>
…16711)

Bumps [github.com/go-git/go-git/v5](https://github.com/go-git/go-git) from 5.8.1 to 5.11.0.
- [Release notes](https://github.com/go-git/go-git/releases)
- [Commits](go-git/go-git@v5.8.1...v5.11.0)

---
updated-dependencies:
- dependency-name: github.com/go-git/go-git/v5
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…e` cmd (#16898)

Use fully qualified application names when operate with Applications

Closes: #16896

Signed-off-by: Sergiy Kulanov <[email protected]>
* docs: Fixed Slugify doc in  GoTemplate.md

Signed-off-by: Aymen Ben Tanfous <[email protected]>

* Update docs/operator-manual/applicationset/GoTemplate.md

Co-authored-by: Blake Pettersson <[email protected]>
Signed-off-by: Aymen Ben Tanfous <[email protected]>

* Update docs/operator-manual/applicationset/GoTemplate.md

Co-authored-by: Blake Pettersson <[email protected]>
Signed-off-by: Aymen Ben Tanfous <[email protected]>

---------

Signed-off-by: Aymen Ben Tanfous <[email protected]>
Co-authored-by: Blake Pettersson <[email protected]>
* feat(controller): add sync jitter

Signed-off-by: Alexandre Gaudreault <[email protected]>

* convert to duration for simplicity

Signed-off-by: Alexandre Gaudreault <[email protected]>

* docs

Signed-off-by: Alexandre Gaudreault <[email protected]>

* add config to manifests

Signed-off-by: Alexandre Gaudreault <[email protected]>

* fix tests

Signed-off-by: Alexandre Gaudreault <[email protected]>

---------

Signed-off-by: Alexandre Gaudreault <[email protected]>
* Add PITS Globale Datenrettungsdienste to user list

Signed-off-by: Arnold <[email protected]>

* Update USERS.md

Signed-off-by: Arnold <[email protected]>

---------

Signed-off-by: Arnold <[email protected]>
Co-authored-by: Soumya Ghosh Dastidar <[email protected]>
…on sync with auto create ns and server side apply (#16942)

* Uptake fix in gitops engine to fix auto create ns with server side apply

Signed-off-by: anandf <[email protected]>

* Moved the new e2e test to different location

Signed-off-by: anandf <[email protected]>

* Fix test name to be less than 63 char for creating ns

Signed-off-by: anandf <[email protected]>

* update gitops-engine with latest master

Signed-off-by: Leonardo Luz Almeida <[email protected]>

---------

Signed-off-by: anandf <[email protected]>
Signed-off-by: Leonardo Luz Almeida <[email protected]>
Co-authored-by: Leonardo Luz Almeida <[email protected]>
vipin-m-s and others added 22 commits March 24, 2024 01:56
Signed-off-by: Vipin M S <[email protected]>
Co-authored-by: Ishita Sequeira <[email protected]>
…ions (Issue #14725) (#17109)

* fix(repo-server): excess git requests, cache lock on revisions

Signed-off-by: nromriell <[email protected]>

* fix: pr feedback, simplify, add configurable variable

Signed-off-by: nromriell <[email protected]>

* fix: codegen, lint

Signed-off-by: nromriell <[email protected]>

* fix: test print, no opts set, var type nit

Signed-off-by: nromriell <[email protected]>

* chore: add additional logging for unexpected cache error

Signed-off-by: nromriell <[email protected]>

---------

Signed-off-by: nromriell <[email protected]>
Co-authored-by: Ishita Sequeira <[email protected]>
…17530)

* update argocd app history command to print app history group by thier sources along with all the REVISIONS

Signed-off-by: Mangaal <[email protected]>

* upadte unit test to ahve both Source and Sources and update function to overlooked source if sources is persent

Signed-off-by: Mangaal <[email protected]>

* remove magic no 7 and introduc a variable MAX_ALLOWED_REVISIONS

Signed-off-by: Mangaal <[email protected]>

* remove extra unit test

Signed-off-by: Mangaal <[email protected]>

* remove extra unit test TestPrintApplicationHistoryTableForWhenBothSourcesAndSourceFiledsExist()

Signed-off-by: Mangaal <[email protected]>

---------

Signed-off-by: Mangaal <[email protected]>
Co-authored-by: Ishita Sequeira <[email protected]>
…nt order (#17615)

* instead if ranging over the map varHistory to print the history, inroduced a string array varHistoryKeys

Signed-off-by: Mangaal <[email protected]>

* update unit test expectation, remove new line in the beginning

Signed-off-by: Mangaal <[email protected]>

---------

Signed-off-by: Mangaal <[email protected]>
* fix: Appcontroller keeps op running when denied by sync window

Signed-off-by: Charles Coupal-Jetté <[email protected]>

* fix: Update test name

Signed-off-by: Charles Coupal-Jetté <[email protected]>

---------

Signed-off-by: Charles Coupal-Jetté <[email protected]>
Co-authored-by: Blake Pettersson <[email protected]>
Signed-off-by: Michael Crenshaw <[email protected]>
Co-authored-by: Blake Pettersson <[email protected]>
Co-authored-by: Jann Fischer <[email protected]>
…17541)

Bumps [follow-redirects](https://github.com/follow-redirects/follow-redirects) from 1.15.5 to 1.15.6.
- [Release notes](https://github.com/follow-redirects/follow-redirects/releases)
- [Commits](follow-redirects/follow-redirects@v1.15.5...v1.15.6)

---
updated-dependencies:
- dependency-name: follow-redirects
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Add arturia as users

Signed-off-by: olivier beyler <[email protected]>
Bumps [express](https://github.com/expressjs/express) from 4.17.3 to 4.19.2.
- [Release notes](https://github.com/expressjs/express/releases)
- [Changelog](https://github.com/expressjs/express/blob/master/History.md)
- [Commits](expressjs/express@4.17.3...4.19.2)

---
updated-dependencies:
- dependency-name: express
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* sec: limit helm index max size

Signed-off-by: pashakostohrys <[email protected]>

* sec: limit helm index max size

Signed-off-by: pashakostohrys <[email protected]>

* feat: fix tests and linter

Signed-off-by: pashakostohrys <[email protected]>

---------

Signed-off-by: pashakostohrys <[email protected]>
* Update security.md

fix RBAC link

Signed-off-by: Deniz Erdogan <[email protected]>

* Update security.md

Signed-off-by: Deniz Erdogan <[email protected]>

* Update security.md

fix link to application-controller role

Signed-off-by: Deniz Erdogan <[email protected]>

* Update security.md

Signed-off-by: Deniz Erdogan <[email protected]>

---------

Signed-off-by: Deniz Erdogan <[email protected]>
* docs: added warning for multiple sources

Signed-off-by: Kostis (Codefresh) <[email protected]>

* docs: minor spelling

Signed-off-by: Kostis (Codefresh) <[email protected]>

---------

Signed-off-by: Kostis (Codefresh) <[email protected]>
…nd --refresh Flags (#17613)

* add const key value for ComponentRepoServer

Signed-off-by: Mangaal <[email protected]>

* update NewRepoServerClient() to look for service with  ComponentRepoServer labels , if the label exist construct label selector PortForward

Signed-off-by: Mangaal <[email protected]>

* add comment for the new constants

Signed-off-by: Mangaal <[email protected]>

* instead of passing nil which leads to  nil ptr referance error, pass empty ClusterSharding{}

Signed-off-by: Mangaal <[email protected]>

* check for operator install repo server name

Signed-off-by: Mangaal <[email protected]>

* handle empty nil ptr dereference error

Signed-off-by: Mangaal <[email protected]>

* handle  nil prt dereference

Signed-off-by: Mangaal <[email protected]>

* typo correction

Signed-off-by: Mangaal <[email protected]>

* run clidocsgen

Signed-off-by: Mangaal <[email protected]>

---------

Signed-off-by: Mangaal <[email protected]>
* metadata.annotations: too long

Signed-off-by: Surajyadav <[email protected]>

* added as a default case

Signed-off-by: Surajyadav <[email protected]>

---------

Signed-off-by: Surajyadav <[email protected]>
Add Shield.com as one of the users in the USER.md file

Signed-off-by: suhas-chikkanna <[email protected]>
kostis-codefresh and others added 6 commits April 2, 2024 13:07
… (#17706)

* Fix color generation for pod name in logs viewer

Signed-off-by: Philipp Trulson <[email protected]>

* Add rebuy to users.md

Signed-off-by: Philipp Trulson <[email protected]>

---------

Signed-off-by: Philipp Trulson <[email protected]>
…mand with `revisions` flag (#17650)

* Add support for multiple source to manifests --revision command

Signed-off-by: ishitasequeira <[email protected]>

* Update GetManifests to support multiple sources

Signed-off-by: ishitasequeira <[email protected]>

* remove testing logs

Signed-off-by: ishitasequeira <[email protected]>

* update cli docs

Signed-off-by: ishitasequeira <[email protected]>

* add extra validation for diff command

Signed-off-by: ishitasequeira <[email protected]>

* fix lint

Signed-off-by: ishitasequeira <[email protected]>

* Empty-Commit

Signed-off-by: ishitasequeira <[email protected]>

* revert apimachinery version

Signed-off-by: ishitasequeira <[email protected]>

* Update docs based on comments

Signed-off-by: ishitasequeira <[email protected]>

---------

Signed-off-by: ishitasequeira <[email protected]>
* test: unit test for respectIgnoreDifferences bug

Signed-off-by: Jesse Suen <[email protected]>

* test: simplify unit test

Signed-off-by: Jesse Suen <[email protected]>

* fix: fix calculating patch for respect ignore diff feature

Signed-off-by: Alexander Matyushentsev <[email protected]>

---------

Signed-off-by: Jesse Suen <[email protected]>
Signed-off-by: Alexander Matyushentsev <[email protected]>
Co-authored-by: Jesse Suen <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment