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 secret ci #15855

Merged
merged 3 commits into from
Jul 4, 2023
Merged

Fix secret ci #15855

merged 3 commits into from
Jul 4, 2023

Conversation

ycliuhw
Copy link
Member

@ycliuhw ycliuhw commented Jul 4, 2023

A couple of secret CI fixes.
Drive-by: we should ignore vault non-reachable network error during model teardown.

@ycliuhw ycliuhw requested a review from wallyworld July 4, 2023 06:30
@wallyworld
Copy link
Member

/merge

@jujubot jujubot merged commit 99a7b7c 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
None yet
Projects
None yet
3 participants