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

[DPE-4157] Add upgrade basics #408

Merged
merged 27 commits into from
May 27, 2024

Conversation

MiaAltieri
Copy link
Contributor

@MiaAltieri MiaAltieri commented Apr 23, 2024

Summary

Before implementing Upgrades add some of the boiler plate that is necessary for them. (i.e. Carl's event handlers + small changes for them to work with MongoDB)

Future PRs

  1. Adding in Carls fixes for upgrade code
  2. Support Upgrades
  • Implementing the actual upgrade procedure in machine_upgrades.py
  • Block other events from executing when an upgrade is in process
  • Set unit state to healthy in the correct places
  1. Adding in unit + integration tests

@@ -58,3 +58,14 @@ set-tls-private-key:

pre-upgrade-check:
description: Run necessary pre-upgrade checks before executing a charm upgrade.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

copied from OpenSearch

UPGRADE_RELATION = "upgrade"


class MongoDBUpgrade(Object):
Copy link
Contributor Author

Choose a reason for hiding this comment

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

most of this is copied from OpenSearch with some changes for MongoDB

@MiaAltieri MiaAltieri changed the title Add upgrade basics [DPE-4157] Add upgrade basics Apr 24, 2024
actions.yaml Outdated Show resolved Hide resolved
charm_version Outdated Show resolved Hide resolved
metadata.yaml Outdated Show resolved Hide resolved
workload_version Outdated Show resolved Hide resolved
src/config.py Outdated Show resolved Hide resolved
src/upgrades/machine_upgrade.py Outdated Show resolved Hide resolved
src/upgrades/mongodb_upgrade.py Outdated Show resolved Hide resolved
src/upgrades/mongodb_upgrade.py Show resolved Hide resolved
src/upgrades/mongodb_upgrade.py Outdated Show resolved Hide resolved
src/upgrades/mongodb_upgrade.py Outdated Show resolved Hide resolved
Co-authored-by: Carl Csaposs <carl.csaposs@canonical.com>
metadata.yaml Outdated Show resolved Hide resolved
from config import Config
from upgrades import upgrade
Copy link
Contributor Author

Choose a reason for hiding this comment

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

copied from opensearch

@@ -9,6 +9,7 @@

import abc
import copy
import enum
Copy link
Contributor Author

Choose a reason for hiding this comment

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

copied from opensearch

@MiaAltieri MiaAltieri changed the title [DPE-4157] Add upgrade basics [DPE-4157] Add upgrade basics + bring branch up to date with 6/edge Apr 30, 2024
@MiaAltieri MiaAltieri changed the title [DPE-4157] Add upgrade basics + bring branch up to date with 6/edge [DPE-4157] Add upgrade basics May 8, 2024
@MiaAltieri MiaAltieri merged commit e126eca into new-upgrade-implementation May 27, 2024
34 checks passed
@MiaAltieri MiaAltieri deleted the add-upgrade-basics branch May 27, 2024 09:23
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.

None yet

3 participants