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

[JUJU-3878] - Fix/update secret backend #15849

Merged
merged 4 commits into from
Jul 4, 2023

Conversation

ycliuhw
Copy link
Member

@ycliuhw ycliuhw commented Jul 3, 2023

This PR ensures:

  • the secret backend endpoint read-only;
  • disallowed to change the backend name if the backend is in use for any model;

Checklist

  • Code style: imports ordered, good names, simple structure, etc
  • Comments saying why design decisions were made
  • Go unit tests, with comments saying what you're testing
  • Integration tests, with comments saying what you're testing
  • doc.go added or updated in changed packages

QA steps

juju add-secret-backend myvault vault --config ./vault.yaml

juju model-config secret-backend=myvault

juju deploy snappass-test

juju exec --unit snappass-test/0 -- secret-add owned-by=easyrsa-app
secret://025ca458-bb1c-4640-8ba4-15ba62fa7559/cih7fueffbas7akd0lng

juju update-secret-backend myvault name=myvault2
ERROR cannot rename a secret backend that is in use

juju update-secret-backend myvault endpoint=http://10.180.97.1:8201
ERROR invalid config for provider "vault": cannot change immutable field "endpoint"

Documentation changes

No

Bug reference

No

@ycliuhw ycliuhw force-pushed the fix/update-secret-backend branch from 4f7cd63 to ec7b6be Compare July 3, 2023 07:31
@ycliuhw ycliuhw changed the base branch from 3.3 to 3.1 July 3, 2023 07:32
@ycliuhw ycliuhw changed the title Fix/update secret backend [JUJU-3878] - Fix/update secret backend Jul 3, 2023
@ycliuhw ycliuhw requested a review from wallyworld July 3, 2023 07:34
@hpidcock hpidcock added the 3.1 label Jul 3, 2023
@ycliuhw
Copy link
Member Author

ycliuhw commented Jul 4, 2023

/merge

@jujubot jujubot merged commit 065ece1 into juju:3.1 Jul 4, 2023
21 of 22 checks passed
@jack-w-shaw jack-w-shaw mentioned this pull request Jul 6, 2023
jujubot added a commit that referenced this pull request Jul 6, 2023
#15866

Merges:
- #15852
- #15854
- #15849
- #15855
- #15857
- #15861
- #15825

One conflict in `cmd/juju/application/deployer/bundlehandler.go`
This was referenced Jul 6, 2023
jujubot added a commit that referenced this pull request Jul 7, 2023
#15881

Merges:
- #15845
- #15852
- #15854
- #15849
- #15855
- #15857
- #15861
- #15862
- #15825
- #15866
- #15863
- #15870
- #15871
- #15872
- #15873
- #15874
- #15875

One substantial conflict in `cmd/juju/application/deployer/bundlehandler.go` & `cmd/juju/application/deployer/bundlehandler_test.go`, resulting in failing unit tests.

The conflict relates to this PR:
- #15825

These were fixed by the two tops commits

### QA Steps

Ensure all unit tests pass
```
go test github.com/juju/juju/cmd/juju/application/deployer
go test github.com/juju/juju/core/charm
go test github.com/juju/juju/apiserver/facades/client/application
```

Ensure we can deploy a complex bundle
```
juju deploy kubeflow
```
@hpidcock hpidcock mentioned this pull request Jul 11, 2023
jujubot added a commit that referenced this pull request Jul 11, 2023
#15896

Forward ports:
- #15845
- #15725
- #15852
- #15854
- #15849
- #15855
- #15857
- #15861
- #15862
- #15860
- #15864
- #15825
- #15866
- #15863
- #15870
- #15871
- #15872
- #15873
- #15874
- #15876
- #15875
- #15881
- #15727
- #15883
- #15884
- #15880
- #15879
- #15886
- #15887
- #15877
- #15888
- #15893
- #15894

Conflicts:
- cmd/juju/ssh/debugcode_test.go
- cmd/juju/ssh/debughooks_test.go
- cmd/juju/ssh/scp_unix_test.go
- cmd/juju/ssh/ssh_machine.go
- cmd/juju/ssh/ssh_machine_test.go
- cmd/juju/ssh/ssh_unix_test.go
- worker/dbaccessor/worker.go
- caas/kubernetes/provider/k8s_test.go
- caas/kubernetes/provider/operator_test.go
- caas/kubernetes/provider/package_test.go
- caas/kubernetes/provider/rbac.go
- cmd/juju/application/deployer/bundlehandler.go
- cmd/juju/application/deployer/bundlehandler_test.go
- cmd/jujud/agent/model/manifolds.go
- core/bundle/changes/changes.go
- worker/caasapplicationprovisioner/application.go
- worker/caasapplicationprovisioner/application_test.go
- worker/caasapplicationprovisioner/mock_test.go
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
4 participants