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-2145] Refactoring: refactor charm events handlers #166

Merged
merged 2 commits into from
Jun 23, 2023

Conversation

dmitry-ratushnyy
Copy link
Contributor

@dmitry-ratushnyy dmitry-ratushnyy commented Jun 22, 2023

About

This PR is a part of our efforts con converge code between MongoDB VM charm and MongoDB K8s charm
This PR introduces refactoring of charm events handlers to make them similar to MongoDB VM charm.

Related issue

DPE-2145

@dmitry-ratushnyy dmitry-ratushnyy force-pushed the dmitry.ratushnyy/refactoring__event_handlers branch 2 times, most recently from e1db271 to ce5502b Compare June 22, 2023 08:24
@dmitry-ratushnyy dmitry-ratushnyy changed the title [WIP] Refactoring: refactor charm events handlers [DPE-2145] Refactoring: refactor charm events handlers Jun 22, 2023
@dmitry-ratushnyy dmitry-ratushnyy force-pushed the dmitry.ratushnyy/refactoring__event_handlers branch 2 times, most recently from b021062 to 01aebe9 Compare June 22, 2023 08:30
@dmitry-ratushnyy dmitry-ratushnyy force-pushed the dmitry.ratushnyy/refactoring__event_handlers branch 2 times, most recently from 4311704 to 5779ce5 Compare June 23, 2023 09:56
Copy link
Contributor

@juditnovak juditnovak left a comment

Choose a reason for hiding this comment

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

LGTM (assuming a reliable test coverage ensuring that nothing broke underneath).
Will just be a nightmare to rebase on I'm afraid 😱 :-)

@dmitry-ratushnyy dmitry-ratushnyy force-pushed the dmitry.ratushnyy/refactoring__event_handlers branch from 5779ce5 to c309f02 Compare June 23, 2023 15:15
src/charm.py Outdated Show resolved Hide resolved
@dmitry-ratushnyy dmitry-ratushnyy force-pushed the dmitry.ratushnyy/refactoring__event_handlers branch from c309f02 to 91cd829 Compare June 23, 2023 15:50
@dmitry-ratushnyy dmitry-ratushnyy force-pushed the dmitry.ratushnyy/refactoring__event_handlers branch from 91cd829 to 4681787 Compare June 23, 2023 16:24
@dmitry-ratushnyy dmitry-ratushnyy merged commit 41a167f into main Jun 23, 2023
@dmitry-ratushnyy dmitry-ratushnyy deleted the dmitry.ratushnyy/refactoring__event_handlers branch June 23, 2023 18:15
src/charm.py Show resolved Hide resolved
src/charm.py Show resolved Hide resolved
src/charm.py Show resolved Hide resolved
src/charm.py Show resolved Hide resolved
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.

3 participants