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

[NEXARL-306] Update prerm script to not disabling and stopping agents if it's an upgrade process #557

Merged
merged 3 commits into from
Oct 2, 2024

Conversation

yengliong93
Copy link
Contributor

@yengliong93 yengliong93 commented Oct 1, 2024

PULL DESCRIPTION

This PR updates the prerm script to prevent them from disabling and stopping if it's an upgrade process.

Impact Analysis

Info Please fill out this column
Root Cause Specifically for bugs, empty in case of no variants
Jira ticket NEXARL-306

CODE MAINTAINABILITY

  • Added required new tests relevant to the changes
  • Updated Documentation as relevant to the changes
  • PR change contains code related to security
  • PR introduces changes that break compatibility with other modules/services (If YES, please provide description)
  • Run go fmt or format-python.sh as applicable
  • Update Changelog
  • Integration tests are passing
  • If Cloudadapter changes, check Azure connectivity manually

Code must act as a teacher for future developers

it's an upgrade process

This PR updates the prerm script to prevent them from disabling and
stopping if it's an upgrade process.

Signed-off-by: yengliong <[email protected]>
@yengliong93
Copy link
Contributor Author

@gblewis1 please have a review, thanks.

@gblewis1
Copy link
Contributor

gblewis1 commented Oct 1, 2024

@yengliong93 looks good but please run through integration tests

@gblewis1
Copy link
Contributor

gblewis1 commented Oct 1, 2024

Integration tests pass.

@yengliong93 yengliong93 merged commit ac66a99 into develop Oct 2, 2024
7 checks passed
@yengliong93 yengliong93 deleted the not-restart-services branch October 2, 2024 02:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants