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

Adds remaining authorization objects to ignore. #13214

Merged
merged 1 commit into from
Aug 4, 2021

Conversation

tlm
Copy link
Member

@tlm tlm commented Aug 4, 2021

It's important that the model admission operator ignores certain types
of Kubernetes objects that don't persist in the etcd state. This change
further adds remainder of objects that should be ignored.

QA steps

Run Kubeflow locally on microk8s. Access the dashboard and add a volume from the Kubeflow dashboard.

Also confirmed from Dom that this fixes the issue.

Bug reference

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

It's important that the model admission operator ignores certain types
of Kubernetes objects that don't perists in the etcd state. This change
further adds remainder of objects that should be ignored.
@tlm
Copy link
Member Author

tlm commented Aug 4, 2021

$$merge$$

@jujubot jujubot merged commit e11999e into juju:2.9 Aug 4, 2021
@wallyworld wallyworld mentioned this pull request Aug 6, 2021
jujubot added a commit that referenced this pull request Aug 8, 2021
#13220

Merge 2.9

#13211 Fix data race (accessing charmInfo) in TestV2CharmExitsApplicationWorker
#13212 Fix retry-provisioning command
#13210 Add note and todo
#13209 Improve readablity of 'no prdesc not found' error message
#13214 Adds remaining authorization objects to ignore
#13215 Use a key-based mutex lock to download resources/agent bins once
#13218 Show application data set by the remote app in show-unit output
#13219 Add impish to the ubuntu series list

Trivial conflicts in imports/code comments.

```
Conflicts:
# apiserver/facades/client/client/client.go
# apiserver/facades/schema.json
# resource/resourceadapters/opener.go
```
## QA steps

See PRs
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