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
Made changes to save server name only if it is not blank #4730
Made changes to save server name only if it is not blank #4730
Conversation
320203a
to
db267f4
Compare
@h-kataria can you try to fix the CC issue? |
db267f4
to
975f5dd
Compare
Also fixed the check that compares previously saved value against the value from miq_server record instead of checking `@update.config[:server][:name]` server/name key did not exist in settings.yml so the condition always returned true. Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1634518
975f5dd
to
9680941
Compare
Checked commit h-kataria@9680941 with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0 app/controllers/ops_controller/settings/common.rb
|
@dclarizio fixed cc issue. |
…me_for_server Made changes to save server name only if it is not blank (cherry picked from commit 5078295) Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1634518
Hammer backport details:
|
backporting this PR to Gaprindashvili, should fix https://bugzilla.redhat.com/show_bug.cgi?id=1635178 |
@h-kataria Both files conflict backporting to Gaprindashvili branch. Please create a separate PR |
Backported to Gaprindashvili via #4876 |
Also fixed the check that compares previously saved value against the value from miq_server record instead of checking
@update.config[:server][:name]
server/name key did not exist in settings.yml so the condition always returned true.Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1634518
Fixes https://bugzilla.redhat.com/show_bug.cgi?id=1635178
@yrudman please test