Skip to content

Backport 1.5.x: Global plugin reload#9354

Merged
sgmiller merged 5 commits into
release/1.5.xfrom
backport-pr-9347-1.5.x
Jul 6, 2020
Merged

Backport 1.5.x: Global plugin reload#9354
sgmiller merged 5 commits into
release/1.5.xfrom
backport-pr-9347-1.5.x

Conversation

@sgmiller
Copy link
Copy Markdown
Collaborator

No description provided.

sgmiller added 4 commits June 30, 2020 18:34
* Carefully move changes from the plugin-cluster-reload branch into this clean branch off master.

* Don't test this at this level, adequately covered in the api level tests

* Change PR link

* go.mod

* Vendoring

* Vendor api/sys_plugins.go
* Fix wrong return value (discovered when merging to ENT)

* go.mod

* go mod vendor

* Add setup plugin reload hook

* All reloads return something now
* just use an error string

* Switch command to use new struct
@sgmiller sgmiller changed the title Backport plugin reload changes to 1.5.x release branch Backport 1.5.x: Global plugin reload Jun 30, 2020
@sgmiller sgmiller added this to the 1.5 milestone Jun 30, 2020
@sgmiller sgmiller requested review from alexanderbez and calvn July 1, 2020 11:59
Copy link
Copy Markdown
Contributor

@alexanderbez alexanderbez left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK

Comment thread vault/core.go
}

if !c.ReplicationState().HasState(consts.ReplicationPerformanceSecondary | consts.ReplicationDRSecondary) {
//Cannot do this above, as we need other resources like mounts to be setup
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
//Cannot do this above, as we need other resources like mounts to be setup
// Cannot do this above, as we need other resources like mounts to be setup

@sgmiller sgmiller merged commit 539a54f into release/1.5.x Jul 6, 2020
@sgmiller sgmiller deleted the backport-pr-9347-1.5.x branch July 6, 2020 16:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants