Bug 1650177 - During engine-setup (as past of engine update), ImageProxyAddress doesn't get updated with the engine FQDN
Summary: During engine-setup (as past of engine update), ImageProxyAddress doesn't get...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: ovirt-imageio
Classification: oVirt
Component: Proxy
Version: 1.4.6
Hardware: x86_64
OS: Unspecified
high
high
Target Milestone: ovirt-4.3.2
: 1.5.1
Assignee: Yedidyah Bar David
QA Contact: Petr Matyáš
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2018-11-15 14:17 UTC by Elad
Modified: 2019-03-26 07:20 UTC (History)
13 users (show)

Fixed In Version: ovirt-imageio-1.5.1
Doc Type: Bug Fix
Doc Text:
This release ensures the upgrade process in Red Hat Virtualization Manager sets the configuration value ImageProxyAddress to point to the Red Hat Virtualization Manager FQDN if the configuration value was set to "localhost".
Clone Of:
Environment:
Last Closed: 2019-03-26 07:20:49 UTC
oVirt Team: Integration
Embargoed:
rule-engine: ovirt-4.3+
lleistne: testing_ack+


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 98011 0 master MERGED packaging: setup: set vdc option ImageProxyAddress on upgrade 2020-04-26 08:45:39 UTC

Description Elad 2018-11-15 14:17:44 UTC
Description of problem:
For an engine that was upgraded from 4.2.6 to 4.2.7 and was running also as earlier versions (starting 3.1), the ImageProxyAddress key in vdc_options doesn't get updated with the engine FQDN.
We guess that ImageProxyAddress key never existed while the engine was initlially installed (aroud RHV 3.1) so when it was updated to a version that added this key, the key was set with a default value - 'localhost:DEFAULT_IMAGEIO_PORT'.


Version-Release number of selected component (if applicable):
Current version - rhvm-4.2.7.4-0.1.el7ev.noarch
This engine was updated many times, starting somewhere around RHV 3.1


Not sure which info is relevant here, please raise a need-info if any additional info is required.

Comment 1 Tal Nisan 2018-11-15 15:04:31 UTC
Daniel, is this our realm or integration?

Comment 2 Daniel Erez 2018-11-19 08:34:24 UTC
(In reply to Tal Nisan from comment #1)
> Daniel, is this our realm or integration?

@Asaf - could it be related to bug 1519194? is it a similar code perhaps?

Comment 3 Sandro Bonazzola 2018-11-19 08:57:24 UTC
Didi, can you please have a look?

Comment 4 Yedidyah Bar David 2018-11-26 09:11:49 UTC
Please attach sosreport from the engine vm. Thanks. Can mark as private if it's a production system.

Comment 6 Yedidyah Bar David 2018-11-27 10:29:02 UTC
The guess in comment 0 seems accurate. The code updating vdc_options indeed runs only on new setups, not on upgrade. It was added to fix bug 1342322.

The code was mostly copied from similar code for websocket-proxy, so we likely have the same bug about vdc option 'WebSocketProxy'.

Fixing is trivial - just removing the condition on NEW_DATABASE. But is this what we want? In particular: What if the user manually changed it? This can happen due to several reasons, not only rename - e.g. they might have several IP addresses on the machine and want to use a specific one for the imageio proxy, so set to a name that resolves to this address. I might agree to changing it if it's set to 'localhost', but not certain this is a good idea either.

Comment 7 Sandro Bonazzola 2018-11-28 09:57:00 UTC
Can we compare engine FQDN with content of the vdc option during configuration step and ask if we want it updated or not in next stages?
So if user changed it on purpose we won't overwrite the change and if it comes from previous versions that missed the key user can fix it as part of the upgrade.

Comment 8 Sandro Bonazzola 2019-01-28 09:40:14 UTC
This bug has not been marked as blocker for oVirt 4.3.0.
Since we are releasing it tomorrow, January 29th, this bug has been re-targeted to 4.3.1.

Comment 10 Yedidyah Bar David 2019-03-18 06:58:32 UTC
(In reply to Avihai from comment #9)
> Hi Didi, 
> 
> As we do not have the original setup with multiple upgrades I did the
> following on 4.3.2.1(with ovirt-imageio-1.5.1) engine:

Please start from a 4.1 engine and upgrade it to 4.2 then 4.3.

If not done already, please open a bug/ticket in QE to have this
and similar upgrade flows checked automatically. I realize this
will require quite a lot of work, but IMO will be worth it.

> 
> 1) I set via engine-config 'ImageProxyAddress' to 'localhost'

This is incorrect. The code (both that inserts this key and that
uses this key) uses host:port, not only host.

Specifically, the upgrade code (with the fix for current bug)
will change the key only if it's exactly 'localhost:54323'.

But please do not rely on this, and start from 4.1 as suggested
above. Verify that in the upgrade to 4.2 it's not updated (that's
current bug), then that upgrade to 4.3.2 updates it (so it's
fixed). Thanks.

Comment 11 Petr Matyáš 2019-03-21 10:08:42 UTC
Verified on ovirt-imageio-proxy-setup-1.5.1-0.el7ev.noarch

Comment 12 Sandro Bonazzola 2019-03-26 07:20:49 UTC
This bugzilla is included in oVirt 4.3.2 release, published on March 19th 2019.

Since the problem described in this bug report should be
resolved in oVirt 4.3.2 release, it has been closed with a resolution of CURRENT RELEASE.

If the solution does not work for you, please open a new bug report.


Note You need to log in before you can comment on or make changes to this bug.