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-3064] reusable workflow #338

Merged
merged 13 commits into from
Dec 1, 2023
Merged

[DPE-3064] reusable workflow #338

merged 13 commits into from
Dec 1, 2023

Conversation

dragomirp
Copy link
Contributor

@dragomirp dragomirp commented Nov 28, 2023

Switch the charm to the GH hosted reusable workflow.

@dragomirp dragomirp force-pushed the dpe-3064-reusable-workflow branch from e286e78 to 2b7bde4 Compare November 28, 2023 23:04
@dragomirp dragomirp marked this pull request as ready for review November 29, 2023 10:18
tests/integration/ha_tests/test_upgrade.py Outdated Show resolved Hide resolved
pyproject.toml Outdated Show resolved Hide resolved
@dragomirp dragomirp force-pushed the dpe-3064-reusable-workflow branch from 4e28626 to 95bb4cf Compare November 29, 2023 10:45
marceloneppel
marceloneppel previously approved these changes Nov 29, 2023
marceloneppel
marceloneppel previously approved these changes Nov 30, 2023
@dragomirp dragomirp merged commit 5a21d70 into main Dec 1, 2023
39 checks passed
@dragomirp dragomirp deleted the dpe-3064-reusable-workflow branch December 1, 2023 10:37
Copy link
Contributor

@taurus-forever taurus-forever left a comment

Choose a reason for hiding this comment

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

LGTM postfactum. Tnx!

BON4 pushed a commit to BON4/postgresql-k8s-operator that referenced this pull request May 20, 2024
* Bump actions

* Adding marks

* Relative imports

* Switch to reusable workflow

* Add markers

* Skip password tests on juju

* Remove deps

* Borked import

* Bump workflow

* Missing import
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.

5 participants