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

fetch-lib #156

Merged
merged 2 commits into from Nov 19, 2022
Merged

fetch-lib #156

merged 2 commits into from Nov 19, 2022

Conversation

sed-i
Copy link
Contributor

@sed-i sed-i commented Nov 17, 2022

Issue

Exception on ingress removal.

Solution

Fetch lib.

@sed-i
Copy link
Contributor Author

sed-i commented Nov 17, 2022

charmcarft says:

Library charms.grafana_auth.v0.grafana_auth has local changes, cannot be updated.
Library charms.grafana_k8s.v0.grafana_auth was already up to date in version 0.1.

charm code only uses

from charms.grafana_auth.v0.grafana_auth import AuthRequirer, AuthRequirerCharmEvents

Both files have the same LIBID.

Any cleanup to do here? @rbarry82 @saltiyazan

@sed-i sed-i force-pushed the feature/fetch-lib branch 2 times, most recently from 9dab358 to 49e7972 Compare November 18, 2022 16:26
@sed-i
Copy link
Contributor Author

sed-i commented Nov 19, 2022

CI fails for reasons documented in #137. Merging.

@sed-i sed-i merged commit c20ab40 into main Nov 19, 2022
@sed-i sed-i deleted the feature/fetch-lib branch November 19, 2022 02:44
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.

None yet

3 participants