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

Fixed Settings hierarchy on Global region #20299

Merged
merged 6 commits into from Jun 29, 2020

Conversation

yrudman
Copy link
Contributor

@yrudman yrudman commented Jun 20, 2020

ISSUE:
On global region settings hierarchy MiqRegion->Zone->MiqServer was always using local region (which is global region), even for remote severs

Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1836158

@miq-bot add-label bug, core

app/models/miq_server.rb Outdated Show resolved Hide resolved
@yrudman yrudman force-pushed the global-region-settings-fix branch 4 times, most recently from 6b98690 to e82b446 Compare June 22, 2020 21:08
@yrudman yrudman changed the title [WIP] Fixed Settings hierarchy on Global region Fixed Settings hierarchy on Global region Jun 22, 2020
@yrudman
Copy link
Contributor Author

yrudman commented Jun 22, 2020

@miq-bot add-label ivanchuk/yes, jansa/yes?

@miq-bot miq-bot added ivanchuk/yes and removed wip labels Jun 22, 2020
@Fryguy Fryguy self-assigned this Jun 25, 2020
@yrudman yrudman force-pushed the global-region-settings-fix branch from f8dbc05 to 9c19aaf Compare June 25, 2020 18:47
@yrudman yrudman force-pushed the global-region-settings-fix branch from d5b9924 to 9d71f17 Compare June 25, 2020 21:20
@miq-bot
Copy link
Member

miq-bot commented Jun 25, 2020

Checked commits yrudman/manageiq@86f34c3~...9d71f17 with ruby 2.5.7, rubocop 0.69.0, haml-lint 0.28.0, and yamllint
3 files checked, 0 offenses detected
Everything looks fine. 🍰

@Fryguy Fryguy merged commit 68b086e into ManageIQ:master Jun 29, 2020
@yrudman yrudman deleted the global-region-settings-fix branch June 29, 2020 16:37
simaishi pushed a commit that referenced this pull request Jul 1, 2020
@simaishi
Copy link
Contributor

simaishi commented Jul 1, 2020

Ivanchuk backport details:

$ git log -1
commit a3251c7b642f777a716f179bd69f549f8946c34c
Author: Jason Frey <fryguy9@gmail.com>
Date:   Mon Jun 29 11:42:52 2020 -0400

    Merge pull request #20299 from yrudman/global-region-settings-fix

    Fixed Settings hierarchy on Global region

    (cherry picked from commit 68b086e235f11e3edd5a9826eb0a8f680a8da0b1)

    https://bugzilla.redhat.com/show_bug.cgi?id=1836158

simaishi pushed a commit that referenced this pull request Jul 1, 2020
Fixed Settings hierarchy on Global region

(cherry picked from commit 68b086e)
@simaishi
Copy link
Contributor

simaishi commented Jul 1, 2020

Jansa backport details:

$ git log -1
commit c729b7c442857b60c232238e1e4f841e3cb4b157
Author: Jason Frey <fryguy9@gmail.com>
Date:   Mon Jun 29 11:42:52 2020 -0400

    Merge pull request #20299 from yrudman/global-region-settings-fix

    Fixed Settings hierarchy on Global region

    (cherry picked from commit 68b086e235f11e3edd5a9826eb0a8f680a8da0b1)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants