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-2804] Split off lib-check #298

Merged
merged 3 commits into from
Oct 24, 2023
Merged

Conversation

dragomirp
Copy link
Contributor

@dragomirp dragomirp commented Oct 20, 2023

Issue

Checking charm libs requires credentials that are not available to contributors outside of the DPE team.

Solution

Split the checking action in a separate workflow that is not run on forked PRs.

@dragomirp dragomirp force-pushed the dpe-2804-conditional-lib-check branch from 838e10e to 681fcc0 Compare October 20, 2023 12:43
@dragomirp dragomirp changed the title Split off lib-check [DPE-2804] Split off lib-check Oct 20, 2023
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.

2022 is only outdated info here...

LGTM as a phase 1 to unblock external contributions (plugins).
The next step will be: automatical PR creation (daily?) for charm-libs updates available.

Tnx!

@dragomirp dragomirp marked this pull request as ready for review October 23, 2023 09:22
name: Check libraries
runs-on: ubuntu-latest
timeout-minutes: 5
if: ${{ github.event.pull_request.head.repo.full_name == 'canonical/postgresql-k8s-operator' }}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

If the PR is not from the main repo, skip the check.

Copy link
Member

@marceloneppel marceloneppel left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks Dragomir!

@dragomirp dragomirp merged commit 5c308a5 into main Oct 24, 2023
31 checks passed
@dragomirp dragomirp deleted the dpe-2804-conditional-lib-check branch October 24, 2023 08:48
BON4 pushed a commit to BON4/postgresql-k8s-operator that referenced this pull request May 20, 2024
* Split off lib-check

* Don't release lib-check changes

* Bump libs
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.

3 participants