Skip to content

Fix wrong err return value in plugin reload status command#9348

Merged
sgmiller merged 5 commits into
masterfrom
fix-plugin-reload-cmd-error-handler
Jun 30, 2020
Merged

Fix wrong err return value in plugin reload status command#9348
sgmiller merged 5 commits into
masterfrom
fix-plugin-reload-cmd-error-handler

Conversation

@sgmiller
Copy link
Copy Markdown
Collaborator

No description provided.

@sgmiller sgmiller requested a review from alexanderbez June 30, 2020 15:34
@sgmiller sgmiller added this to the 1.5 milestone Jun 30, 2020
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

@sgmiller sgmiller requested a review from alexanderbez June 30, 2020 16:14
@alexanderbez
Copy link
Copy Markdown
Contributor

Looks like some TestSystemBackend_Plugin_reload tests are failing?

@sgmiller sgmiller merged commit 0be9220 into master Jun 30, 2020
@sgmiller sgmiller deleted the fix-plugin-reload-cmd-error-handler branch June 30, 2020 18:33
sgmiller added a commit that referenced this pull request Jun 30, 2020
* Fix wrong return value (discovered when merging to ENT)

* go.mod

* go mod vendor

* Add setup plugin reload hook

* All reloads return something now
sgmiller added a commit that referenced this pull request Jul 6, 2020
* Global Plugin Reload: OSS Changes Take II (#9347)

* 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 err return value in plugin reload status command (#9348)

* Fix wrong return value (discovered when merging to ENT)

* go.mod

* go mod vendor

* Add setup plugin reload hook

* All reloads return something now

* Address feedback on Plugin Reload: OSS Side (#9350)

* just use an error string

* Switch command to use new struct

* Don't setup plugin reload on perf standbys (#9352)
andaley pushed a commit that referenced this pull request Jul 17, 2020
* Fix wrong return value (discovered when merging to ENT)

* go.mod

* go mod vendor

* Add setup plugin reload hook

* All reloads return something now
eualin pushed a commit to eualin/vault that referenced this pull request Sep 15, 2025
…llment endpoint are routed as login requests. (hashicorp#9346) (hashicorp#9348)

Co-authored-by: Kuba Wieczorek <kuba.wieczorek@hashicorp.com>
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.

2 participants