New commit detected on ManageIQ/manageiq/gaprindashvili: https://github.com/ManageIQ/manageiq/commit/fbb84f082699e1e6ede42aa25e372bf265ab2218 commit fbb84f082699e1e6ede42aa25e372bf265ab2218 Author: Chris Arcand <chrisarcand.github.com> AuthorDate: Mon Mar 12 21:16:47 2018 -0400 Commit: Chris Arcand <chrisarcand.github.com> CommitDate: Mon Mar 12 21:16:47 2018 -0400 Merge pull request #17139 from bdunne/only_update_zone_if_found Only update zone if found (cherry picked from commit 360af512f8c2bfbdc0c3478715805f2e19fdf68d) Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1599349 app/models/miq_server/configuration_management.rb | 4 +- spec/models/miq_server/configuration_management_spec.rb | 8 + 2 files changed, 10 insertions(+), 2 deletions(-)
On 5.9.4.2 it's possible to change zone settings to non-existent zone. Expected behavior is that only valid zone names are allowed. PATCH /api/servers/1/settings { "server": { "zone": "Non Existent" } } Response: ... "server": { ... "zone": "Non Existent"
Update to comment #3: The appliance keeps functioning after the zone is changed to invalid setting.
PR to fix this https://github.com/ManageIQ/manageiq/pull/17892
New commit detected on ManageIQ/manageiq/gaprindashvili: https://github.com/ManageIQ/manageiq/commit/1ee2efb07e813c581507acd08d1eec24f967479c commit 1ee2efb07e813c581507acd08d1eec24f967479c Author: Brandon Dunne <brandondunne> AuthorDate: Wed Aug 22 17:14:09 2018 -0400 Commit: Brandon Dunne <brandondunne> CommitDate: Wed Aug 22 17:14:09 2018 -0400 Merge pull request #17892 from juliancheal/fix_invalid_zone Ensure Zone data is Valid (cherry picked from commit 2b06c5daec30c7d00871b3d3503cb4475aee0948) Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1599349 lib/vmdb/config/validator.rb | 7 + 1 file changed, 7 insertions(+)
New commit detected on ManageIQ/manageiq/gaprindashvili: https://github.com/ManageIQ/manageiq/commit/f765ee0454abc0ded6ea169d8bb14bb726f503fa commit f765ee0454abc0ded6ea169d8bb14bb726f503fa Author: Jason Frey <fryguy9> AuthorDate: Tue Sep 18 13:42:42 2018 -0400 Commit: Jason Frey <fryguy9> CommitDate: Tue Sep 18 13:42:42 2018 -0400 Merge pull request #17994 from juliancheal/fix_spelling Fixed spelling error (cherry picked from commit 156cd6b949e5bc4837edd1d76e47e643d9250040) https://bugzilla.redhat.com/show_bug.cgi?id=1599349 lib/vmdb/config/validator.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
On 5.9.5.2: $ curl -k https://10.16.5.91/api/servers/1/settings -u admin:smartvm --data @/tmp/data -XPATCH {"error":{"kind":"internal_server_error","message":"configuration invalid","klass":"RuntimeError"}}% $ cat /tmp/data { "server": { "name": "Test Name", "zone": "de fault"}} In 5.9.4.7 I am allowed to change the zone.
Since the problem described in this bug report should be resolved in a recent advisory, it has been closed with a resolution of ERRATA. For information on the advisory, and where to find the updated files, follow the link below. If the solution does not work for you, open a new bug report. https://access.redhat.com/errata/RHSA-2018:3466
*** Bug 1648808 has been marked as a duplicate of this bug. ***