Bug 1838977 - No virtualHostedStyle option when upgrade from 4.4 to 4.5
Summary: No virtualHostedStyle option when upgrade from 4.4 to 4.5
Alias: None
Product: OpenShift Container Platform
Classification: Red Hat
Component: Image Registry
Version: 4.5
Hardware: Unspecified
OS: Unspecified
Target Milestone: ---
: 4.5.0
Assignee: Oleg Bulatov
QA Contact: Wenjing Zheng
Depends On:
TreeView+ depends on / blocked
Reported: 2020-05-22 08:36 UTC by Wenjing Zheng
Modified: 2020-07-13 17:41 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: No Doc Update
Doc Text:
Clone Of:
Last Closed: 2020-07-13 17:41:00 UTC
Target Upstream Version:

Attachments (Terms of Use)
Image registry operator log (45.50 KB, text/plain)
2020-05-22 08:40 UTC, Wenjing Zheng
no flags Details

System ID Private Priority Status Summary Last Updated
Github openshift cluster-image-registry-operator pull 558 0 None closed Bug 1838977: virtualHostedStyle should be false for public regions 2020-06-24 03:14:34 UTC
Red Hat Product Errata RHBA-2020:2409 0 None None None 2020-07-13 17:41:18 UTC

Description Wenjing Zheng 2020-05-22 08:36:47 UTC
Description of problem:
There is no "virtualHostedStyle" option when cluster is upgrade from 4.4 to 4.5 latest nightly build.

Version-Release number of selected component (if applicable):

How reproducible:

Steps to Reproduce:
1.Start a 4.4.z stable build 
2.Define regionEndpoint or not - optional
      bucket: wzhengup-cvzds-image-registry-us-east-2-xmodaxmojupwsiyqwnanwr
      encrypt: true
      region: us-east-2
      regionEndpoint: https://s3.us-east-2.amazonaws.com/
3.Upgrade to 4.5 latest nightly build

Actual results:
There is no option virtualHostedStyle  after upgrade.

Expected results:
There should be an option virtualHostedStyle  to indicate true/false

Additional info:

Comment 1 Wenjing Zheng 2020-05-22 08:40:09 UTC
Created attachment 1690989 [details]
Image registry operator log

Comment 2 Oleg Bulatov 2020-05-22 15:48:11 UTC
This parameter works only when regionEndpoint is set. When it's unset, virtualHostedStyle should have the backward compatible value, i.e. "false".

If someone has a script that patches regionEndpoint after the installer finishes its job, "false" won't break this script. "true" will break this script, i.e. the default "true" it's not backward compatible.

I'm going to update the default to be "false". Thanks for catching this.

Comment 5 Wenjing Zheng 2020-05-27 06:08:03 UTC
I can see below env var by accessing pod env list, but still no option in configs.imageregistry exposed by default after upgrade: 
$ oc set env pods/image-registry-564d4fdb7b-6vtkk --list | grep REGISTRY_STORAGE_S3_VIRTUALHOSTEDSTYLE

Whether the option "virtualHostedStyle" should display after upgrade in configs.imageregistry?

Comment 6 Oleg Bulatov 2020-05-27 09:15:10 UTC
No, it shouldn't. No value and false have the exact same meaning.

Comment 7 Wenjing Zheng 2020-05-27 10:01:05 UTC
OK, thanks, then this bug can be verified now.

Comment 8 errata-xmlrpc 2020-07-13 17:41:00 UTC
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.


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