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.

Sign up for GitHub

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

Jump to bottom

Add post-upgrade hook to helm installer job in case of upgrade #312

Merged
merged 42 commits into from
Jul 15, 2021
Merged

Add post-upgrade hook to helm installer job in case of upgrade #312

merged 42 commits into from
Jul 15, 2021

Conversation

amansharma00a
Copy link
Contributor

added post upgrade hook in case of upgrade

@amansharma00a
Update _pega-installer-job.tpl
132bc42 
added post upgrade hook
@amansharma00a amansharma00a requested review from dcasavant and a team as code owners June 20, 2021 18:42
@MadhuriArugula MadhuriArugula added the integ-eks Label that triggers automation testing against EKS label Jun 21, 2021
@pegaautomationuser
Copy link
Collaborator

Starting PR-312 validation on -> integ-eks

1 similar comment
@pegaautomationuser
Copy link
Collaborator

Starting PR-312 validation on -> integ-eks

@amansharma00a amansharma00a changed the title Added post-upgrade hook Add post-upgrade hook to helm installer job in case of upgrade Jun 21, 2021
@kishorv10 kishorv10 added the build Related to the build process label Jun 21, 2021
@amansharma00a
Update pega-installer-config.yaml
6873acd 
config changes in case of upgrade
@pegaautomationuser
Copy link
Collaborator

Invalid label - build. valid labels are [integ-all, integ-eks, integ-gke, integ-aks, configs]

1 similar comment
@pegaautomationuser
Copy link
Collaborator

Invalid label - build. valid labels are [integ-all, integ-eks, integ-gke, integ-aks, configs]

@amansharma00a
Update _helpers.tpl
1905e61 
added pega-upgrade-config
@pegaautomationuser
Copy link
Collaborator

Invalid label - build. valid labels are [integ-all, integ-eks, integ-gke, integ-aks, configs]

1 similar comment
@pegaautomationuser
Copy link
Collaborator

Invalid label - build. valid labels are [integ-all, integ-eks, integ-gke, integ-aks, configs]

@MadhuriArugula MadhuriArugula removed the build Related to the build process label Jun 23, 2021
MadhuriArugula
MadhuriArugula requested changes Jun 23, 2021
View reviewed changes
Copy link
Collaborator

@MadhuriArugula MadhuriArugula left a comment

Choose a reason for hiding this comment

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

Added review comments which needs to be addressed.

@pegaautomationuser
Copy link
Collaborator

Starting PR-312 validation on -> integ-eks

@pegaautomationuser
Copy link
Collaborator

Starting PR-312 validation on -> integ-eks

1 similar comment
@pegaautomationuser
Copy link
Collaborator

Starting PR-312 validation on -> integ-eks

@amansharma00a
Update _helpers.tpl
21075b5 
pega-installer-config -> pega-install-config
@pegaautomationuser
Copy link
Collaborator

Starting PR-312 validation on -> integ-eks

1 similar comment
@pegaautomationuser
Copy link
Collaborator

Starting PR-312 validation on -> integ-eks

@amansharma00a
Update _pega-installer-job.tpl
d4adf1f 
config map changes for pegaUpgradeConfig in case of upgrade
@pegaautomationuser
Copy link
Collaborator

Starting PR-312 validation on -> integ-eks

2 similar comments
@pegaautomationuser
Copy link
Collaborator

Starting PR-312 validation on -> integ-eks

@pegaautomationuser
Copy link
Collaborator

Starting PR-312 validation on -> integ-eks

@pegaautomationuser
Copy link
Collaborator

Starting PR-312 validation on -> integ-eks

1 similar comment
@pegaautomationuser
Copy link
Collaborator

Starting PR-312 validation on -> integ-eks

@amansharma00a
Update _helpers.tpl
a96b80d 
pegaDBCustomUpgrade
@pegaautomationuser
Copy link
Collaborator

Starting PR-312 validation on -> integ-eks

@pegaautomationuser
Copy link
Collaborator

Starting PR-312 validation on -> integ-eks

2 similar comments
@pegaautomationuser
Copy link
Collaborator

Starting PR-312 validation on -> integ-eks

@pegaautomationuser
Copy link
Collaborator

Starting PR-312 validation on -> integ-eks

Saurabh-16
Saurabh-16 approved these changes Jul 9, 2021
View reviewed changes
Copy link
Contributor

@Saurabh-16 Saurabh-16 left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Changes looks good. Please ensure they are working as expected.

@pegaautomationuser
Copy link
Collaborator

Starting PR-312 validation on -> integ-eks

1 similar comment
@pegaautomationuser
Copy link
Collaborator

Starting PR-312 validation on -> integ-eks

@pegaautomationuser
Copy link
Collaborator

Starting PR-312 validation on -> integ-eks

@pegaautomationuser
Copy link
Collaborator

Starting PR-312 validation on -> integ-eks

@pegaautomationuser
Copy link
Collaborator

Starting PR-312 validation on -> integ-eks

@amansharma00a
Create pega-custom-upgrade_test.go
f1a3b72 
test case for upgradeType -> custom
@pegaautomationuser
Copy link
Collaborator

Starting PR-312 validation on -> integ-eks

@pegaautomationuser
Copy link
Collaborator

Starting PR-312 validation on -> integ-eks

@MadhuriArugula MadhuriArugula merged commit a39c99f into pegasystems:master Jul 15, 2021
@MadhuriArugula MadhuriArugula mentioned this pull request Oct 25, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pega-chikv pega-chikv pega-chikv approved these changes

@arvasrikanth arvasrikanth arvasrikanth approved these changes

@Saurabh-16 Saurabh-16 Saurabh-16 approved these changes

@MadhuriArugula MadhuriArugula MadhuriArugula approved these changes

@kishorv10 kishorv10 kishorv10 approved these changes

@dcasavant dcasavant Awaiting requested review from dcasavant

Assignees
No one assigned
Labels
integ-eks Label that triggers automation testing against EKS
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@amansharma00a @pegaautomationuser @kishorv10 @MadhuriArugula @Saurabh-16 @arvasrikanth @pega-chikv