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/lp 1936253 #13168

Merged
merged 2 commits into from Jul 15, 2021
Merged

Fix/lp 1936253 #13168

merged 2 commits into from Jul 15, 2021

Conversation

ycliuhw
Copy link
Member

@ycliuhw ycliuhw commented Jul 14, 2021

Fix Preconditions for cluster role binding deletion;

Checklist

  • Requires a pylibjuju change
  • Added integration tests for the PR
  • Added or updated doc.go related to packages changed
  • Comments answer the question of why design decisions were made

QA steps

$ juju deploy snappass-test

$ juju remove-application snappass-test --destroy-storage --force
removing application snappass-test

Documentation changes

No

Bug reference

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

@ycliuhw
Copy link
Member Author

ycliuhw commented Jul 15, 2021

$$merge$$

@jujubot jujubot merged commit e797352 into juju:2.9 Jul 15, 2021
@ycliuhw ycliuhw deleted the fix/lp-1936253 branch July 15, 2021 03:53
@ycliuhw
Copy link
Member Author

ycliuhw commented Jul 15, 2021

@wallyworld wallyworld mentioned this pull request Jul 16, 2021
jujubot added a commit that referenced this pull request Jul 16, 2021
#13173

Merge 2.9

#13172 Fix azure instance queries to account for old completed deployments
#13166 Add support for bespoke service principals in Azure credentials
#13169 SpaceAddress conversion from single candidates instead of a slice
#13162 enhance test-network-health
#13168 Fix/lp 1936253
#13164 Fixes unit logging appearing on machine log
#13160 State logs: index label fields
#13163 CLI Deploy: Fix panic

The conflicts were mainly due to removing legacy storage from Azure in develop and changing an api version.

```
Conflicts:
worker/uniter/runner/jujuc/mocks/context_mock.go
version/version.go
tests/suites/network/network_health.sh
snap/snapcraft.yaml
scripts/win-installer/setup.iss
provider/azure/environ_test.go
provider/azure/environ.go
provider/azure/config_test.go
```

## QA steps

See PRs
jujubot added a commit that referenced this pull request Jul 22, 2021
#13187

Merge from 2.9 to bring forward:
- #13184 from manadart/2.9-net-info-iaas-addr-results
- #13176 from SimonRichardson/juju-inspect
- #13185 from SimonRichardson/logging-unmarshalling-fix
- #13181 from SimonRichardson/debug-log-space-labels
- #13182 from achilleasa/2.9-fix-error-in-equinix-credential-detection
- #13179 from SimonRichardson/update-charm-error-messages
- #13178 from manadart/2.9-k9s-no-bootstrap-series
- #13175 from manadart/2.9-net-info-ingress-sort
- #13174 from SimonRichardson/export-bundle-local-charms
- #13170 from manadart/2.9-address-sorting
- #13171 from SimonRichardson/deploy-v1-kubernetes
- #13172 from wallyworld/azure-instance-lookup
- #13167 from jujubot/increment-to-2.9.10
- #13169 from manadart/2.9-space-address-conversion
- #13162 from hmlanigan/fix-test-network
- #13166 from wallyworld/azure-bespoke-serviceprincipal
- #13168 from ycliuhw/fix/lp-1936253
- #13164 from tlm/lp1933548-unit-logging
- #13163 from SimonRichardson/deploy-panic-fix
- #13160 from SimonRichardson/index-label-logs

Conflicts resolved in:
- apiserver/facades/agent/uniter/networkinfoiaas.go
- core/charm/format.go
- core/charm/format_test.go
- go.mod
- go.sum
- provider/azure/config_test.go
- provider/azure/environ.go
- provider/azure/environ_test.go
- provider/azure/environprovider.go
- scripts/win-installer/setup.iss
- snap/snapcraft.yaml
- tests/suites/network/network_health.sh
- version/version.go
- worker/uniter/runner/jujuc/mocks/context_mock.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