Skip to content

Commit

Permalink
Merge pull request juju#16105 from SimonRichardson/inject-ctrl-config…
Browse files Browse the repository at this point in the history
…-upgrade-steps

juju#16105

The following injects the controller config getter into the upgrade steps facade. This is a nice easy example of swapping over to the controller config service.

## Checklist

- [x] Code style: imports ordered, good names, simple structure, etc
- [x] Comments saying why design decisions were made
- [x] Go unit tests, with comments saying what you're testing

## QA steps

```sh
$ juju bootstrap lxd test --build-agent
$ juju add-model ubuntu
$ juju deploy ubuntu -n 3
$ juju upgrade-controller --build-agent
$ juju upgrade-model
```
  • Loading branch information
jujubot authored Aug 22, 2023
2 parents 9473545 + 10fed8a commit 257c615
Show file tree
Hide file tree
Showing 9 changed files with 483 additions and 63 deletions.
377 changes: 377 additions & 0 deletions apiserver/facades/agent/upgradesteps/facade_mock_test.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading

0 comments on commit 257c615

Please sign in to comment.