Skip to content

Commit

Permalink
Merge pull request juju#18548 from ycliuhw/remove-environ-upgrader
Browse files Browse the repository at this point in the history
juju#18548

This PR removes the IAAS and CAAS environ upgrader workers and the related facades.
We will redesign and implement the environ upgrades later.

## 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
- [ ] ~[Integration tests](https://github.com/juju/juju/tree/main/tests), with comments saying what you're testing~
- [ ] ~[doc.go](https://discourse.charmhub.io/t/readme-in-packages/451) added or updated in changed packages~

## QA steps

```
juju deploy ubuntu

juju destroy-controller k1 --destroy-all-models

```

## Documentation changes

No

## Links

**Jira card:** [JUJU-7232](https://warthogs.atlassian.net/browse/JUJU-7232)



[JUJU-7232]: https://warthogs.atlassian.net/browse/JUJU-7232?atlOrigin=eyJpIjoiNWRkNTljNzYxNjVmNDY3MDlhMDU5Y2ZhYzA5YTRkZjUiLCJwIjoiZ2l0aHViLWNvbS1KU1cifQ
  • Loading branch information
jujubot authored Dec 18, 2024
2 parents be55fea + 3a37b2f commit 17a4ce1
Show file tree
Hide file tree
Showing 24 changed files with 7 additions and 2,166 deletions.
14 changes: 0 additions & 14 deletions api/controller/environupgrader/package_test.go

This file was deleted.

98 changes: 0 additions & 98 deletions api/controller/environupgrader/upgrader.go

This file was deleted.

168 changes: 0 additions & 168 deletions api/controller/environupgrader/upgrader_test.go

This file was deleted.

1 change: 0 additions & 1 deletion api/facadeversions.go
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,6 @@ var facadeVersions = facades.FacadeVersions{
"Deployer": {1},
"DiskManager": {2},
"EntityWatcher": {2},
"EnvironUpgrader": {1},
"ExternalControllerUpdater": {1},
"FilesystemAttachmentsWatcher": {2},
"Firewaller": {7},
Expand Down
2 changes: 0 additions & 2 deletions apiserver/allfacades.go
Original file line number Diff line number Diff line change
Expand Up @@ -79,7 +79,6 @@ import (
"github.com/juju/juju/apiserver/facades/controller/crosscontroller"
"github.com/juju/juju/apiserver/facades/controller/crossmodelrelations"
"github.com/juju/juju/apiserver/facades/controller/crossmodelsecrets"
"github.com/juju/juju/apiserver/facades/controller/environupgrader"
"github.com/juju/juju/apiserver/facades/controller/externalcontrollerupdater"
"github.com/juju/juju/apiserver/facades/controller/firewaller"
"github.com/juju/juju/apiserver/facades/controller/imagemetadata"
Expand Down Expand Up @@ -201,7 +200,6 @@ func AllFacades() *facade.Registry {
externalcontrollerupdater.Register(registry)
deployer.Register(registry)
diskmanager.Register(registry)
environupgrader.Register(registry)
firewaller.Register(registry)
highavailability.Register(registry)
hostkeyreporter.Register(registry)
Expand Down
44 changes: 0 additions & 44 deletions apiserver/facades/controller/environupgrader/backend.go

This file was deleted.

14 changes: 0 additions & 14 deletions apiserver/facades/controller/environupgrader/package_test.go

This file was deleted.

Loading

0 comments on commit 17a4ce1

Please sign in to comment.