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: added deployment_strategy_ref_mapping_removal #3362

Merged
merged 1 commit into from
May 9, 2023

Conversation

pawan-59
Copy link
Contributor

@pawan-59 pawan-59 commented May 9, 2023

Description

@pawan-59 pawan-59 changed the title Fix: added deployment_strategy_ref_mapping_removal fix: added deployment_strategy_ref_mapping_removal May 9, 2023
@@ -0,0 +1 @@
UPDATE global_strategy_metadata_chart_ref_mapping SET active=true WHERE chart_ref_id in(SELECT id FROM chart_ref WHERE location LIKE '%deployment-chart_%') AND global_strategy_metadata_id=(SELECT id FROM global_strategy_metadata WHERE name='RECREATE');
Copy link
Contributor

Choose a reason for hiding this comment

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

why like ? if we know the exact chart ref

@sonarqubecloud
Copy link

sonarqubecloud bot commented May 9, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@pawan-59 pawan-59 merged commit e8d72f1 into main May 9, 2023
@pawan-59 pawan-59 deleted the deployment_strategy_fix branch May 9, 2023 13:22
gireesh-naidu pushed a commit that referenced this pull request Jun 16, 2023
* feat: Approval node scripts (#3345)

* approval node scripts

* go template with missing key handling

* approver param added

* reverted TPrintf func

* approver param type change

* script number updated

* script version update

* script number updated

* enhancement: made revision history limit configurable for helm upgrade (#3353)

* made revision history limit configurable for helm upgrade

* review changes

* updated fieldName

* renamed constS

* added deployment_strategy_ref_mapping_removal (#3362)

* task: changes for kubelink update application requests (#3359)

* changes for kubelink - update application requests

* added bean.go

---------

Co-authored-by: kripanshdevtron <107392309+kripanshdevtron@users.noreply.github.com>
Co-authored-by: Pawan Kumar <85476803+pawan-59@users.noreply.github.com>
gireesh-naidu pushed a commit that referenced this pull request Jun 16, 2023
* feat: Approval node scripts (#3345)

* approval node scripts

* go template with missing key handling

* approver param added

* reverted TPrintf func

* approver param type change

* script number updated

* script version update

* script number updated

* enhancement: made revision history limit configurable for helm upgrade (#3353)

* made revision history limit configurable for helm upgrade

* review changes

* updated fieldName

* renamed constS

* added deployment_strategy_ref_mapping_removal (#3362)

* task: changes for kubelink update application requests (#3359)

* changes for kubelink - update application requests

* added bean.go

* fix: updated rbac for fetching all default role (#3363)

* updated rbac for fetching all default role

* moved method to get rbac team names to enforcerUtil

---------

Co-authored-by: kripanshdevtron <107392309+kripanshdevtron@users.noreply.github.com>
Co-authored-by: Pawan Kumar <85476803+pawan-59@users.noreply.github.com>
gireesh-naidu pushed a commit that referenced this pull request Jun 16, 2023
* feat: Approval node scripts (#3345)

* approval node scripts

* go template with missing key handling

* approver param added

* reverted TPrintf func

* approver param type change

* script number updated

* script version update

* script number updated

* enhancement: made revision history limit configurable for helm upgrade (#3353)

* made revision history limit configurable for helm upgrade

* review changes

* updated fieldName

* renamed constS

* added deployment_strategy_ref_mapping_removal (#3362)

* task: changes for kubelink update application requests (#3359)

* changes for kubelink - update application requests

* added bean.go

* fix: updated rbac for fetching all default role (#3363)

* updated rbac for fetching all default role

* moved method to get rbac team names to enforcerUtil

* enhancement: made pipeline status crons cascading (#3372)

* made pipeline status crons cascading, added metrics for cron process duration

* updated defer calls

* updated label for cron status metric

---------

Co-authored-by: kripanshdevtron <107392309+kripanshdevtron@users.noreply.github.com>
Co-authored-by: Pawan Kumar <85476803+pawan-59@users.noreply.github.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.

2 participants