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

[NC | NSFS] RPM Upgrades #7493

Merged
merged 1 commit into from
Sep 28, 2023

Conversation

tangledbytes
Copy link
Member

@tangledbytes tangledbytes commented Sep 14, 2023

Explain the changes

This PR updates the RPM upgrades to add support for:

  1. Auto upgrade trigger for both nsfs and non-nsfs deployments (determines the deployment type based on the processes running).
  2. Enhances the upgrade_manager to support NSFS deployments where it does not rely on the presence of a DB.
  3. Adds the construct system.json under /etc/noobaa.conf.d which holds the following data right now { "current_version": "...", "upgrade_history": "[ ... ]" } which is created by the nsfs.js during its first boot.

Testing Instructions:

  1. Build and install the RPM.
  2. Update the version in package.json and build the RPM again.
  3. Upgrade the RPM by running rpm -U <name> and you should see additional logs flowing in, ultimately the upgrade_manager should exit with zero exit code.
  • Doc added/updated
  • Tests added

@tangledbytes tangledbytes force-pushed the utkarsh/rpm/nsfs-upgrade branch 2 times, most recently from 6840428 to f1a0a8d Compare September 26, 2023 07:32
@tangledbytes tangledbytes changed the title [NC | NSFS] Upgrades [NC | NSFS] RPM Upgrades Sep 26, 2023
@tangledbytes tangledbytes marked this pull request as ready for review September 26, 2023 08:25
src/cmd/nsfs.js Outdated Show resolved Hide resolved
src/cmd/nsfs.js Outdated Show resolved Hide resolved
src/upgrade/upgrade_manager.js Outdated Show resolved Hide resolved
src/upgrade/upgrade_manager.js Outdated Show resolved Hide resolved
src/upgrade/upgrade_manager.js Outdated Show resolved Hide resolved
@tangledbytes
Copy link
Member Author

Will update the PR once #7509 gets in.

@romayalon
Copy link
Contributor

@tangledbytes I merged 7509

Signed-off-by: Utkarsh Srivastava <srivastavautkarsh8097@gmail.com>

fix init upgrade

Signed-off-by: Utkarsh Srivastava <srivastavautkarsh8097@gmail.com>

fix pgrep command output

Signed-off-by: Utkarsh Srivastava <srivastavautkarsh8097@gmail.com>

address PR comments

Signed-off-by: Utkarsh Srivastava <srivastavautkarsh8097@gmail.com>
@tangledbytes tangledbytes merged commit 765527f into noobaa:master Sep 28, 2023
8 checks passed
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.

None yet

2 participants