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: hide ldap creds in in get req #4788

Merged
merged 8 commits into from
Mar 20, 2024
Merged

fix: hide ldap creds in in get req #4788

merged 8 commits into from
Mar 20, 2024

Conversation

prakash100198
Copy link
Contributor

@prakash100198 prakash100198 commented Mar 13, 2024

Description

Fixes #4720

Checklist:

  • The title of the PR states what changed and the related issues number (used for the release note).
  • Does this PR requires documentation updates?
  • I've updated documentation as required by this PR.
  • I have performed a self-review of my own code.
  • I have commented my code, particularly in hard-to-understand areas.
  • I have tested it for all user roles.
  • I have added all the required unit/api test cases.

Does this PR introduce a user-facing change?


Copy link

sonarcloud bot commented Mar 20, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@prakash100198 prakash100198 merged commit 8e4cc36 into main Mar 20, 2024
4 checks passed
@prakash100198 prakash100198 deleted the secure-ldap-creds branch March 20, 2024 11:39
prakarsh-dt added a commit that referenced this pull request Mar 28, 2024
* feat: active inactive user phase 3 (#4649)

* listing user and group changes

* default value chnage

* group listing optimisation

* wire_gen

* order by in group

* default values

* discard

* updated on

* case insensitive

* script number change

* specs

* review chnages

* sql update

* id for user

* script number chnage

* review comments-1

* review comments

* review comments

* rest handler remove methods

* validation for delete

* delete user handling

* review comments

* review comments

* bulk delete user and permission group

* legacy code fix from loop in query to bulk query

* export csv filters support

* Revert "export csv filters support"

This reverts commit 4448c9e.

* export csv filters

* bulk delete support with filters

* validation

* method break

* refactor the method

* condition

* open api  specs

* specs

* refactoring filters in bulk delete rolegroup

* comments

* comments

* script

* group listing

* review comments

* review comments

* review comments

* review comments

* user delete self-review

* error handling

* renaming helpers

* commets

* checks and renaming

* last login order

* script number change

* rolegroup migration

* name change

* backward compatibility handling

* self review name change

* chart-group -manager-fix

* userrolegroups

* change chnage operation

* specs update

* groups

* find by componet id

* job project id

* sql script chnage

* script number change

* fix: extra check added for mono-repo migraiton (#4764)

* extra check added for mono-repo migraiton

* comparison of repoNameWithoutPrefix and appstore name for mono repo condition

* refactor

* remove prefix

* fix

* remove-gitops-prefix-oss

* refactor

* refactor

* refactor

* added request method in audit logger (#4817)

* fix:  rolefilters correction with all applications and particular application selected. (#4820)

* merging issue resolved

* all namespace, cluster, kind remove as it is used in key

* removing unnecessary code

* fix: 5xx 4.0 iter (#4620)

* bugs revert

* 4th iter code changes

* delete dag exec

* initial poc (ongoing) of handling grpc errors

* grpc error handling at install helm chart and template services

* refactoring

* refactor

* code review changes with some refactoring

* removed unused files

* refactor

* chore: refactoring v4 (#4775)

* removed registerInArgo multiple impls

* extracted app metrics code

* migrated envLevel app metrics code to new service

* chore: Removed unused jira and migration  integration (#4498)

* removed unsued jira integration

* removed test-suite-code

* db migration conf removal

* chore: removed unused injection

* chore: removed dead code

* added: migration script

---------

Co-authored-by: Ash-exp <asutosh2000ad@gmail.com>

* chore: App store dead code cleanup and restructuring  (#4497)

* moved chart-group in seperate code

* removed unused dependency

* removed dead code

* extracted resource tree

* moved notes

* resource movement

* removed unused code

* removed unused dependency

* commit methods

* extracted status update

* chore: clean up unused dead code

* updated: EA mode docker file

* updated: migration number

---------

Co-authored-by: Ash-exp <asutosh2000ad@gmail.com>

* chart ref refactoring

* removed infra metrics db calls

* moved app metrics repositories from /internal to /pkg

* moved: const and types to bean

* removed: unused const

* review comments

* migrated some methods from chartService to chartRefService

* added dt validation service interface

* minor refactoring

* moved validation method - 1

* wip

* removed redundant appMetrics req obj

* moved app metrics bindings  to wireset

* removed multiple dead code

* remove redundant dependency

* moved ChartGroup router and rest handler to respective folder

* stage 1

* gitOps refactoring

* moved gitClient code to a common wrapper service

* chore: AppStoreDeployment Install flow refactoring

* review changes

* wip

* fix for unsupported charts

* refactoring: App Store deployment services

* minor cleanup

* renamed remote package to git

* renamed gitOpsRemoteOpService If and impl

* migrated usages of gitService to gitOperationService

* shifted git service and all gitOps clients to pkg

* gitops repository usages refactor

* refactored gitOpsRepository usages

* gitlab client creation refactoring

* renamed util/ChartService

* reverted renaming changes

* reverted renaming changes

* reverted renaming changes

* wip

* wip

* removed typo

* changes

* changes

* extracted trigger cd, nats subscriptions from wfDAGExec service

* removed gitOpsRepoName fetch logic duplicacy

* minor change for cd trigger method

* removed redundant imports

* extracted deployment bulk trigger publish event logic from workflowDag

* extracted manifest creation code from WorkflowDagExecutor

* moved WorkflowStatusUpdateHandler

* removed old refactored code

* wip - extracted k8s op method from workflowDAG part 1

* extracted artifact logic from workflowDAG

* extracted artifact logic from workflowDAG

* refactoring

* replaced slices -> k8s.io/utils/strings/slices import

* replaced slices -> k8s.io/utils/strings/slices import

* fix prod bug

* renamed PrePostStageTriggerService -> preStageTriggerService

* fix for rollback

* wip

* refactoring pre & post stage service

* updated PreCdTriggerService

* migrated AsyncTrigger consumer to eventProcessor

* review comments

* removed whitespaces

* migrated ci material topic to processor service

* migrated argo app status subsciption to common processor pkg

* migrated argo type pipeline publish and process to common pkg

* migrated appstore bulk deploy topic

* migrated cd bulk deploy topic

* migrated appstore helm install status topic

* migrated git webhook event publish

* minor changes in manifest creation service

* minor changes in cd trigger service

* minor changes in cd trigger service

* wip

* minor change in async helm install req handling

* wip

* updated common lib version(synced with main)

* safety check for concurrency in pipeline delete and asyn trigger

* updated common-lib version

* updated common-lib version to main 5807b130153800727ace993e98b24cb27b8fc1fa

---------

Co-authored-by: Nishant <58689354+nishant-d@users.noreply.github.com>
Co-authored-by: Ash-exp <asutosh2000ad@gmail.com>
Co-authored-by: nishant <nishant@devtron.ai>

* fix: hide ldap creds in in get req (#4788)

* hide ldap creds in in get req

* removed unnecessary code

* refactor

* revert

* refactor

* pointer binding (#4826)

* Adding script to calculate score for pagerduty issues

* Update github_pagerduty_issue_score_calculation.py

* feat: Branch Divergence Checker Plugin (#4806)

* Branch Divergence Checker Plugin

* Delete scripts/sql/229_GitHub_branch_divergence_checker_v1.0.down.sql

* Delete assets/GitHubBranchDivergenceCheckerlogo.png

* Adding assets

* Changing asset name

* Removing asset

* Update 229_github_branch_divergence_checker.up.sql

* feat: Added Apply job in k8s plugin (#4828)

* feat: Added Devtron CI Trigger Plugin v1.0.0

* feat: Added Apply JOB in k8s Plugin

* modified structure

* Added error handelling

* Removed CI trigger plugin

* Migration number changed

* logo changed

* feat: Information of Linked CI Pipelines on Parent CI Pipeline (#4786)

* Refactor ArgoUserService.go by adding a TODO

* wip

* updated API spec

* Update repo and service

* Add logging of errors

* modified API spec

* added total count in query

* updated service and rest handler layer

* Update router and resthandler

* set default req params at rest handler

* moved generics to global util

* Update resthandler

* added tracing for new queries

* fixed undefined ctx

* Update handler's rbac

* update error logs

* Update cipipeline repo

* added: comments for changes

* Change handler

* fixed typo and imports

* fixed adapter and constants

* updated test file errors and mock files

* updated API specs

* fixed API specs end points

* fixed: query errors

* fixed: linked cd condition

* fixed: typo

* fixed: search

* fixed nil check for runner query

* fixed: append in adapter

* fix searchkey to lowercase

* Fix duplicated env names

* fix pass env array as empty

* Fix error logs

* fix error

* fixed import

---------

Co-authored-by: komalreddy3 <koukuntlakomalreddy3@gmail.com>

* Update github_pagerduty_issue_score_calculation.py

* Update github_pagerduty_score_calculation.yml

* Update github_pagerduty_score_calculation.yml

* Update github_pagerduty_issue_score_calculation.py

* Update github_pagerduty_issue_score_calculation.py

* Update github_pagerduty_score_calculation.yml

* feat:Github Pull Request Closer (#4833)

* PR plugin script

* Update 230_Github_Pull_Request_Closer.up.sql

* Update 230_Github_Pull_Request_Closer.up.sql

* Update 230_Github_Pull_Request_Closer.up.sql

* Rename 230_Github_Pull_Request_Closer.down.sql to 231_Github_Pull_Request_Closer.down.sql

* Rename 230_Github_Pull_Request_Closer.up.sql to 231_Github_Pull_Request_Closer.up.sql

* Rename GithubReleasePR.png to GithubPullRequest-Plugin-logo.png

* Update 231_Github_Pull_Request_Closer.up.sql

---------

Co-authored-by: Prakarsh <71125043+prakarsh-dt@users.noreply.github.com>

* chore: Image scanning refactoring (#4802)

* added sql

* updated event type for job in workflow request

* added comments, handling for global cm/cs

* fixed json string global cmcs

* updated global cm/cs spec

* wip

* updated sql script number

* updated sql script number

* Update github_pagerduty_score_calculation.yml

* Update github_pagerduty_issue_score_calculation.py

* Update github_pagerduty_score_calculation.yml

* Update github_pagerduty_issue_score_calculation.py

* fix: helm deployements stucked in queued for devtron apps (#4842)

* ns not found in case ips is being injected in cluster (#4844)

* Delete .github/github_pagerduty_issue_score_calculation.py

---------

Co-authored-by: Shivam Nagar <124123645+Shivam-nagar23@users.noreply.github.com>
Co-authored-by: Prakash <prakash.kumar@devtron.ai>
Co-authored-by: Raunit Verma <155707586+raunit-verma@users.noreply.github.com>
Co-authored-by: kartik-579 <84493919+kartik-579@users.noreply.github.com>
Co-authored-by: Nishant <58689354+nishant-d@users.noreply.github.com>
Co-authored-by: Ash-exp <asutosh2000ad@gmail.com>
Co-authored-by: nishant <nishant@devtron.ai>
Co-authored-by: Kiran <155609672+kirandevtn@users.noreply.github.com>
Co-authored-by: komalreddy3 <koukuntlakomalreddy3@gmail.com>
Co-authored-by: Prakarsh <71125043+prakarsh-dt@users.noreply.github.com>
prakarsh-dt added a commit that referenced this pull request Mar 28, 2024
* misc: Pagerduty issue template change

Pagerduty issue template change to seek more information for validating the criticality of the issue.

* Update pager-duty.yaml

* Update pager-duty.yaml

* Update pager-duty.yaml

* feat: Adding script to calculate score for pagerduty issues (#4832)

* feat: active inactive user phase 3 (#4649)

* listing user and group changes

* default value chnage

* group listing optimisation

* wire_gen

* order by in group

* default values

* discard

* updated on

* case insensitive

* script number change

* specs

* review chnages

* sql update

* id for user

* script number chnage

* review comments-1

* review comments

* review comments

* rest handler remove methods

* validation for delete

* delete user handling

* review comments

* review comments

* bulk delete user and permission group

* legacy code fix from loop in query to bulk query

* export csv filters support

* Revert "export csv filters support"

This reverts commit 4448c9e.

* export csv filters

* bulk delete support with filters

* validation

* method break

* refactor the method

* condition

* open api  specs

* specs

* refactoring filters in bulk delete rolegroup

* comments

* comments

* script

* group listing

* review comments

* review comments

* review comments

* review comments

* user delete self-review

* error handling

* renaming helpers

* commets

* checks and renaming

* last login order

* script number change

* rolegroup migration

* name change

* backward compatibility handling

* self review name change

* chart-group -manager-fix

* userrolegroups

* change chnage operation

* specs update

* groups

* find by componet id

* job project id

* sql script chnage

* script number change

* fix: extra check added for mono-repo migraiton (#4764)

* extra check added for mono-repo migraiton

* comparison of repoNameWithoutPrefix and appstore name for mono repo condition

* refactor

* remove prefix

* fix

* remove-gitops-prefix-oss

* refactor

* refactor

* refactor

* added request method in audit logger (#4817)

* fix:  rolefilters correction with all applications and particular application selected. (#4820)

* merging issue resolved

* all namespace, cluster, kind remove as it is used in key

* removing unnecessary code

* fix: 5xx 4.0 iter (#4620)

* bugs revert

* 4th iter code changes

* delete dag exec

* initial poc (ongoing) of handling grpc errors

* grpc error handling at install helm chart and template services

* refactoring

* refactor

* code review changes with some refactoring

* removed unused files

* refactor

* chore: refactoring v4 (#4775)

* removed registerInArgo multiple impls

* extracted app metrics code

* migrated envLevel app metrics code to new service

* chore: Removed unused jira and migration  integration (#4498)

* removed unsued jira integration

* removed test-suite-code

* db migration conf removal

* chore: removed unused injection

* chore: removed dead code

* added: migration script

---------

Co-authored-by: Ash-exp <asutosh2000ad@gmail.com>

* chore: App store dead code cleanup and restructuring  (#4497)

* moved chart-group in seperate code

* removed unused dependency

* removed dead code

* extracted resource tree

* moved notes

* resource movement

* removed unused code

* removed unused dependency

* commit methods

* extracted status update

* chore: clean up unused dead code

* updated: EA mode docker file

* updated: migration number

---------

Co-authored-by: Ash-exp <asutosh2000ad@gmail.com>

* chart ref refactoring

* removed infra metrics db calls

* moved app metrics repositories from /internal to /pkg

* moved: const and types to bean

* removed: unused const

* review comments

* migrated some methods from chartService to chartRefService

* added dt validation service interface

* minor refactoring

* moved validation method - 1

* wip

* removed redundant appMetrics req obj

* moved app metrics bindings  to wireset

* removed multiple dead code

* remove redundant dependency

* moved ChartGroup router and rest handler to respective folder

* stage 1

* gitOps refactoring

* moved gitClient code to a common wrapper service

* chore: AppStoreDeployment Install flow refactoring

* review changes

* wip

* fix for unsupported charts

* refactoring: App Store deployment services

* minor cleanup

* renamed remote package to git

* renamed gitOpsRemoteOpService If and impl

* migrated usages of gitService to gitOperationService

* shifted git service and all gitOps clients to pkg

* gitops repository usages refactor

* refactored gitOpsRepository usages

* gitlab client creation refactoring

* renamed util/ChartService

* reverted renaming changes

* reverted renaming changes

* reverted renaming changes

* wip

* wip

* removed typo

* changes

* changes

* extracted trigger cd, nats subscriptions from wfDAGExec service

* removed gitOpsRepoName fetch logic duplicacy

* minor change for cd trigger method

* removed redundant imports

* extracted deployment bulk trigger publish event logic from workflowDag

* extracted manifest creation code from WorkflowDagExecutor

* moved WorkflowStatusUpdateHandler

* removed old refactored code

* wip - extracted k8s op method from workflowDAG part 1

* extracted artifact logic from workflowDAG

* extracted artifact logic from workflowDAG

* refactoring

* replaced slices -> k8s.io/utils/strings/slices import

* replaced slices -> k8s.io/utils/strings/slices import

* fix prod bug

* renamed PrePostStageTriggerService -> preStageTriggerService

* fix for rollback

* wip

* refactoring pre & post stage service

* updated PreCdTriggerService

* migrated AsyncTrigger consumer to eventProcessor

* review comments

* removed whitespaces

* migrated ci material topic to processor service

* migrated argo app status subsciption to common processor pkg

* migrated argo type pipeline publish and process to common pkg

* migrated appstore bulk deploy topic

* migrated cd bulk deploy topic

* migrated appstore helm install status topic

* migrated git webhook event publish

* minor changes in manifest creation service

* minor changes in cd trigger service

* minor changes in cd trigger service

* wip

* minor change in async helm install req handling

* wip

* updated common lib version(synced with main)

* safety check for concurrency in pipeline delete and asyn trigger

* updated common-lib version

* updated common-lib version to main 5807b130153800727ace993e98b24cb27b8fc1fa

---------

Co-authored-by: Nishant <58689354+nishant-d@users.noreply.github.com>
Co-authored-by: Ash-exp <asutosh2000ad@gmail.com>
Co-authored-by: nishant <nishant@devtron.ai>

* fix: hide ldap creds in in get req (#4788)

* hide ldap creds in in get req

* removed unnecessary code

* refactor

* revert

* refactor

* pointer binding (#4826)

* Adding script to calculate score for pagerduty issues

* Update github_pagerduty_issue_score_calculation.py

* feat: Branch Divergence Checker Plugin (#4806)

* Branch Divergence Checker Plugin

* Delete scripts/sql/229_GitHub_branch_divergence_checker_v1.0.down.sql

* Delete assets/GitHubBranchDivergenceCheckerlogo.png

* Adding assets

* Changing asset name

* Removing asset

* Update 229_github_branch_divergence_checker.up.sql

* feat: Added Apply job in k8s plugin (#4828)

* feat: Added Devtron CI Trigger Plugin v1.0.0

* feat: Added Apply JOB in k8s Plugin

* modified structure

* Added error handelling

* Removed CI trigger plugin

* Migration number changed

* logo changed

* feat: Information of Linked CI Pipelines on Parent CI Pipeline (#4786)

* Refactor ArgoUserService.go by adding a TODO

* wip

* updated API spec

* Update repo and service

* Add logging of errors

* modified API spec

* added total count in query

* updated service and rest handler layer

* Update router and resthandler

* set default req params at rest handler

* moved generics to global util

* Update resthandler

* added tracing for new queries

* fixed undefined ctx

* Update handler's rbac

* update error logs

* Update cipipeline repo

* added: comments for changes

* Change handler

* fixed typo and imports

* fixed adapter and constants

* updated test file errors and mock files

* updated API specs

* fixed API specs end points

* fixed: query errors

* fixed: linked cd condition

* fixed: typo

* fixed: search

* fixed nil check for runner query

* fixed: append in adapter

* fix searchkey to lowercase

* Fix duplicated env names

* fix pass env array as empty

* Fix error logs

* fix error

* fixed import

---------

Co-authored-by: komalreddy3 <koukuntlakomalreddy3@gmail.com>

* Update github_pagerduty_issue_score_calculation.py

* Update github_pagerduty_score_calculation.yml

* Update github_pagerduty_score_calculation.yml

* Update github_pagerduty_issue_score_calculation.py

* Update github_pagerduty_issue_score_calculation.py

* Update github_pagerduty_score_calculation.yml

* feat:Github Pull Request Closer (#4833)

* PR plugin script

* Update 230_Github_Pull_Request_Closer.up.sql

* Update 230_Github_Pull_Request_Closer.up.sql

* Update 230_Github_Pull_Request_Closer.up.sql

* Rename 230_Github_Pull_Request_Closer.down.sql to 231_Github_Pull_Request_Closer.down.sql

* Rename 230_Github_Pull_Request_Closer.up.sql to 231_Github_Pull_Request_Closer.up.sql

* Rename GithubReleasePR.png to GithubPullRequest-Plugin-logo.png

* Update 231_Github_Pull_Request_Closer.up.sql

---------

Co-authored-by: Prakarsh <71125043+prakarsh-dt@users.noreply.github.com>

* chore: Image scanning refactoring (#4802)

* added sql

* updated event type for job in workflow request

* added comments, handling for global cm/cs

* fixed json string global cmcs

* updated global cm/cs spec

* wip

* updated sql script number

* updated sql script number

* Update github_pagerduty_score_calculation.yml

* Update github_pagerduty_issue_score_calculation.py

* Update github_pagerduty_score_calculation.yml

* Update github_pagerduty_issue_score_calculation.py

* fix: helm deployements stucked in queued for devtron apps (#4842)

* ns not found in case ips is being injected in cluster (#4844)

* Delete .github/github_pagerduty_issue_score_calculation.py

---------

Co-authored-by: Shivam Nagar <124123645+Shivam-nagar23@users.noreply.github.com>
Co-authored-by: Prakash <prakash.kumar@devtron.ai>
Co-authored-by: Raunit Verma <155707586+raunit-verma@users.noreply.github.com>
Co-authored-by: kartik-579 <84493919+kartik-579@users.noreply.github.com>
Co-authored-by: Nishant <58689354+nishant-d@users.noreply.github.com>
Co-authored-by: Ash-exp <asutosh2000ad@gmail.com>
Co-authored-by: nishant <nishant@devtron.ai>
Co-authored-by: Kiran <155609672+kirandevtn@users.noreply.github.com>
Co-authored-by: komalreddy3 <koukuntlakomalreddy3@gmail.com>
Co-authored-by: Prakarsh <71125043+prakarsh-dt@users.noreply.github.com>

* Adding scritps to calculate score (#4848)

* Adding scritps to calculate score

* Update github_pagerduty_score_calculation.yml

---------

Co-authored-by: Prakarsh <71125043+prakarsh-dt@users.noreply.github.com>

---------

Co-authored-by: Yashasvi17 <155513200+YashasviDevtron@users.noreply.github.com>
Co-authored-by: Shivam Nagar <124123645+Shivam-nagar23@users.noreply.github.com>
Co-authored-by: Prakash <prakash.kumar@devtron.ai>
Co-authored-by: Raunit Verma <155707586+raunit-verma@users.noreply.github.com>
Co-authored-by: kartik-579 <84493919+kartik-579@users.noreply.github.com>
Co-authored-by: Nishant <58689354+nishant-d@users.noreply.github.com>
Co-authored-by: Ash-exp <asutosh2000ad@gmail.com>
Co-authored-by: nishant <nishant@devtron.ai>
Co-authored-by: Kiran <155609672+kirandevtn@users.noreply.github.com>
Co-authored-by: komalreddy3 <koukuntlakomalreddy3@gmail.com>
komalreddy3 added a commit that referenced this pull request May 28, 2024
* misc: Pagerduty issue template change

Pagerduty issue template change to seek more information for validating the criticality of the issue.

* Update pager-duty.yaml

* Update pager-duty.yaml

* Update pager-duty.yaml

* feat: Adding script to calculate score for pagerduty issues (#4832)

* feat: active inactive user phase 3 (#4649)

* listing user and group changes

* default value chnage

* group listing optimisation

* wire_gen

* order by in group

* default values

* discard

* updated on

* case insensitive

* script number change

* specs

* review chnages

* sql update

* id for user

* script number chnage

* review comments-1

* review comments

* review comments

* rest handler remove methods

* validation for delete

* delete user handling

* review comments

* review comments

* bulk delete user and permission group

* legacy code fix from loop in query to bulk query

* export csv filters support

* Revert "export csv filters support"

This reverts commit 4448c9e.

* export csv filters

* bulk delete support with filters

* validation

* method break

* refactor the method

* condition

* open api  specs

* specs

* refactoring filters in bulk delete rolegroup

* comments

* comments

* script

* group listing

* review comments

* review comments

* review comments

* review comments

* user delete self-review

* error handling

* renaming helpers

* commets

* checks and renaming

* last login order

* script number change

* rolegroup migration

* name change

* backward compatibility handling

* self review name change

* chart-group -manager-fix

* userrolegroups

* change chnage operation

* specs update

* groups

* find by componet id

* job project id

* sql script chnage

* script number change

* fix: extra check added for mono-repo migraiton (#4764)

* extra check added for mono-repo migraiton

* comparison of repoNameWithoutPrefix and appstore name for mono repo condition

* refactor

* remove prefix

* fix

* remove-gitops-prefix-oss

* refactor

* refactor

* refactor

* added request method in audit logger (#4817)

* fix:  rolefilters correction with all applications and particular application selected. (#4820)

* merging issue resolved

* all namespace, cluster, kind remove as it is used in key

* removing unnecessary code

* fix: 5xx 4.0 iter (#4620)

* bugs revert

* 4th iter code changes

* delete dag exec

* initial poc (ongoing) of handling grpc errors

* grpc error handling at install helm chart and template services

* refactoring

* refactor

* code review changes with some refactoring

* removed unused files

* refactor

* chore: refactoring v4 (#4775)

* removed registerInArgo multiple impls

* extracted app metrics code

* migrated envLevel app metrics code to new service

* chore: Removed unused jira and migration  integration (#4498)

* removed unsued jira integration

* removed test-suite-code

* db migration conf removal

* chore: removed unused injection

* chore: removed dead code

* added: migration script

---------

Co-authored-by: Ash-exp <asutosh2000ad@gmail.com>

* chore: App store dead code cleanup and restructuring  (#4497)

* moved chart-group in seperate code

* removed unused dependency

* removed dead code

* extracted resource tree

* moved notes

* resource movement

* removed unused code

* removed unused dependency

* commit methods

* extracted status update

* chore: clean up unused dead code

* updated: EA mode docker file

* updated: migration number

---------

Co-authored-by: Ash-exp <asutosh2000ad@gmail.com>

* chart ref refactoring

* removed infra metrics db calls

* moved app metrics repositories from /internal to /pkg

* moved: const and types to bean

* removed: unused const

* review comments

* migrated some methods from chartService to chartRefService

* added dt validation service interface

* minor refactoring

* moved validation method - 1

* wip

* removed redundant appMetrics req obj

* moved app metrics bindings  to wireset

* removed multiple dead code

* remove redundant dependency

* moved ChartGroup router and rest handler to respective folder

* stage 1

* gitOps refactoring

* moved gitClient code to a common wrapper service

* chore: AppStoreDeployment Install flow refactoring

* review changes

* wip

* fix for unsupported charts

* refactoring: App Store deployment services

* minor cleanup

* renamed remote package to git

* renamed gitOpsRemoteOpService If and impl

* migrated usages of gitService to gitOperationService

* shifted git service and all gitOps clients to pkg

* gitops repository usages refactor

* refactored gitOpsRepository usages

* gitlab client creation refactoring

* renamed util/ChartService

* reverted renaming changes

* reverted renaming changes

* reverted renaming changes

* wip

* wip

* removed typo

* changes

* changes

* extracted trigger cd, nats subscriptions from wfDAGExec service

* removed gitOpsRepoName fetch logic duplicacy

* minor change for cd trigger method

* removed redundant imports

* extracted deployment bulk trigger publish event logic from workflowDag

* extracted manifest creation code from WorkflowDagExecutor

* moved WorkflowStatusUpdateHandler

* removed old refactored code

* wip - extracted k8s op method from workflowDAG part 1

* extracted artifact logic from workflowDAG

* extracted artifact logic from workflowDAG

* refactoring

* replaced slices -> k8s.io/utils/strings/slices import

* replaced slices -> k8s.io/utils/strings/slices import

* fix prod bug

* renamed PrePostStageTriggerService -> preStageTriggerService

* fix for rollback

* wip

* refactoring pre & post stage service

* updated PreCdTriggerService

* migrated AsyncTrigger consumer to eventProcessor

* review comments

* removed whitespaces

* migrated ci material topic to processor service

* migrated argo app status subsciption to common processor pkg

* migrated argo type pipeline publish and process to common pkg

* migrated appstore bulk deploy topic

* migrated cd bulk deploy topic

* migrated appstore helm install status topic

* migrated git webhook event publish

* minor changes in manifest creation service

* minor changes in cd trigger service

* minor changes in cd trigger service

* wip

* minor change in async helm install req handling

* wip

* updated common lib version(synced with main)

* safety check for concurrency in pipeline delete and asyn trigger

* updated common-lib version

* updated common-lib version to main 5807b130153800727ace993e98b24cb27b8fc1fa

---------

Co-authored-by: Nishant <58689354+nishant-d@users.noreply.github.com>
Co-authored-by: Ash-exp <asutosh2000ad@gmail.com>
Co-authored-by: nishant <nishant@devtron.ai>

* fix: hide ldap creds in in get req (#4788)

* hide ldap creds in in get req

* removed unnecessary code

* refactor

* revert

* refactor

* pointer binding (#4826)

* Adding script to calculate score for pagerduty issues

* Update github_pagerduty_issue_score_calculation.py

* feat: Branch Divergence Checker Plugin (#4806)

* Branch Divergence Checker Plugin

* Delete scripts/sql/229_GitHub_branch_divergence_checker_v1.0.down.sql

* Delete assets/GitHubBranchDivergenceCheckerlogo.png

* Adding assets

* Changing asset name

* Removing asset

* Update 229_github_branch_divergence_checker.up.sql

* feat: Added Apply job in k8s plugin (#4828)

* feat: Added Devtron CI Trigger Plugin v1.0.0

* feat: Added Apply JOB in k8s Plugin

* modified structure

* Added error handelling

* Removed CI trigger plugin

* Migration number changed

* logo changed

* feat: Information of Linked CI Pipelines on Parent CI Pipeline (#4786)

* Refactor ArgoUserService.go by adding a TODO

* wip

* updated API spec

* Update repo and service

* Add logging of errors

* modified API spec

* added total count in query

* updated service and rest handler layer

* Update router and resthandler

* set default req params at rest handler

* moved generics to global util

* Update resthandler

* added tracing for new queries

* fixed undefined ctx

* Update handler's rbac

* update error logs

* Update cipipeline repo

* added: comments for changes

* Change handler

* fixed typo and imports

* fixed adapter and constants

* updated test file errors and mock files

* updated API specs

* fixed API specs end points

* fixed: query errors

* fixed: linked cd condition

* fixed: typo

* fixed: search

* fixed nil check for runner query

* fixed: append in adapter

* fix searchkey to lowercase

* Fix duplicated env names

* fix pass env array as empty

* Fix error logs

* fix error

* fixed import

---------

Co-authored-by: komalreddy3 <koukuntlakomalreddy3@gmail.com>

* Update github_pagerduty_issue_score_calculation.py

* Update github_pagerduty_score_calculation.yml

* Update github_pagerduty_score_calculation.yml

* Update github_pagerduty_issue_score_calculation.py

* Update github_pagerduty_issue_score_calculation.py

* Update github_pagerduty_score_calculation.yml

* feat:Github Pull Request Closer (#4833)

* PR plugin script

* Update 230_Github_Pull_Request_Closer.up.sql

* Update 230_Github_Pull_Request_Closer.up.sql

* Update 230_Github_Pull_Request_Closer.up.sql

* Rename 230_Github_Pull_Request_Closer.down.sql to 231_Github_Pull_Request_Closer.down.sql

* Rename 230_Github_Pull_Request_Closer.up.sql to 231_Github_Pull_Request_Closer.up.sql

* Rename GithubReleasePR.png to GithubPullRequest-Plugin-logo.png

* Update 231_Github_Pull_Request_Closer.up.sql

---------

Co-authored-by: Prakarsh <71125043+prakarsh-dt@users.noreply.github.com>

* chore: Image scanning refactoring (#4802)

* added sql

* updated event type for job in workflow request

* added comments, handling for global cm/cs

* fixed json string global cmcs

* updated global cm/cs spec

* wip

* updated sql script number

* updated sql script number

* Update github_pagerduty_score_calculation.yml

* Update github_pagerduty_issue_score_calculation.py

* Update github_pagerduty_score_calculation.yml

* Update github_pagerduty_issue_score_calculation.py

* fix: helm deployements stucked in queued for devtron apps (#4842)

* ns not found in case ips is being injected in cluster (#4844)

* Delete .github/github_pagerduty_issue_score_calculation.py

---------

Co-authored-by: Shivam Nagar <124123645+Shivam-nagar23@users.noreply.github.com>
Co-authored-by: Prakash <prakash.kumar@devtron.ai>
Co-authored-by: Raunit Verma <155707586+raunit-verma@users.noreply.github.com>
Co-authored-by: kartik-579 <84493919+kartik-579@users.noreply.github.com>
Co-authored-by: Nishant <58689354+nishant-d@users.noreply.github.com>
Co-authored-by: Ash-exp <asutosh2000ad@gmail.com>
Co-authored-by: nishant <nishant@devtron.ai>
Co-authored-by: Kiran <155609672+kirandevtn@users.noreply.github.com>
Co-authored-by: komalreddy3 <koukuntlakomalreddy3@gmail.com>
Co-authored-by: Prakarsh <71125043+prakarsh-dt@users.noreply.github.com>

* Adding scritps to calculate score (#4848)

* Adding scritps to calculate score

* Update github_pagerduty_score_calculation.yml

---------

Co-authored-by: Prakarsh <71125043+prakarsh-dt@users.noreply.github.com>

---------

Co-authored-by: Yashasvi17 <155513200+YashasviDevtron@users.noreply.github.com>
Co-authored-by: Shivam Nagar <124123645+Shivam-nagar23@users.noreply.github.com>
Co-authored-by: Prakash <prakash.kumar@devtron.ai>
Co-authored-by: Raunit Verma <155707586+raunit-verma@users.noreply.github.com>
Co-authored-by: kartik-579 <84493919+kartik-579@users.noreply.github.com>
Co-authored-by: Nishant <58689354+nishant-d@users.noreply.github.com>
Co-authored-by: Ash-exp <asutosh2000ad@gmail.com>
Co-authored-by: nishant <nishant@devtron.ai>
Co-authored-by: Kiran <155609672+kirandevtn@users.noreply.github.com>
Co-authored-by: komalreddy3 <koukuntlakomalreddy3@gmail.com>
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.

Enhancement: Hide the details configured in LDAP of SSO login service
2 participants