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

Fix for LP:1925721. #12925

Merged
merged 1 commit into from
Apr 23, 2021
Merged

Fix for LP:1925721. #12925

merged 1 commit into from
Apr 23, 2021

Conversation

hmlanigan
Copy link
Member

Loosen restrictions on Volume secret names, they only need to include -token.

QA steps

# bootstrap microk8s
$ make make microk8s-operator-update
$ juju bootstrap microk8s testme

# download and build jnsgruk-kubernetes-dashboard
$ cd /tmp
$ git clone https://github.com/jnsgruk/charm-kubernetes-dashboard
# Build the charm
$ cd charm-kubernetes-dashboard
$ charmcraft build

# deploy
juju deploy ./jnsgruk-kubernetes-dashboard.charm --resource dashboard-image=kubernetesui/dashboard:v2.0.0

# verify no errors in model logs along the lines of "finding filesystem info for "
$ juju debug-log --level error

# deploy another k8s charm
juju deploy postgresql-charmers-postgresql-k8s

Bug reference

https://bugs.launchpad.net/juju/+bug/1925751

The secret name for a Volume must contain `-token`, the app name is not
required.
@wallyworld
Copy link
Member

$$merge$$

@jujubot jujubot merged commit 512e7a8 into juju:2.9 Apr 23, 2021
@wallyworld wallyworld mentioned this pull request Apr 27, 2021
jujubot added a commit that referenced this pull request Apr 28, 2021
#12930

Merge 2.9

#12911 improve bundle messaging output
#12914 Fix gosigma revision in go mod so it is a real revision
#12907 Charm with v2 format
#12918 juju trust on k8s also creates a cluster role
#12917 Do not panic on nil origin Channel.
#12920 Correct error message for add-unit on CAAS
#12921 Improve error message for invalid API result
#12922 Handle internal server errors from charmhub
#12923 Ensure we always have an application name
#12924 Handle k8s v1 charm inside a bundle
#12925 Fix for LP:1925721
#12912 Series-less charms
#12919 Tweak display of app version in k8s models
#12926 Better error for adding a k8s model with a used namespace
#12932 Remove "caas" from error messages
#12927 Add scope to trust CLI for k8s models
#12928 Suppress charm name in status oci image path
#12929 Adds support for missing Kubernetes vars to hook
#12933 Fix/lp 1924677
#12934 Fix ssh to a k8s unit in the controller model


```
# Conflicts:
# cmd/juju/application/config.go
# cmd/juju/application/deployer/deployer_test.go
# go.mod
# go.sum
# scripts/win-installer/setup.iss
# snap/snapcraft.yaml
# version/version.go
```

## QA steps

See PRs
@hmlanigan hmlanigan deleted the bug-1925751 branch September 26, 2022 17:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants