Bug 1356823
Summary: | Can't push image to registry 2.4.1 with the native registry config by openshift-ansible | |||
---|---|---|---|---|
Product: | OpenShift Container Platform | Reporter: | zhou ying <yinzhou> | |
Component: | Installer | Assignee: | Scott Dodson <sdodson> | |
Status: | CLOSED ERRATA | QA Contact: | Johnny Liu <jialiu> | |
Severity: | high | Docs Contact: | ||
Priority: | high | |||
Version: | 3.3.0 | CC: | aos-bugs, ghuang, jokerman, mfojtik, miminar, mmccomas, wsun | |
Target Milestone: | --- | |||
Target Release: | --- | |||
Hardware: | Unspecified | |||
OS: | Unspecified | |||
Whiteboard: | ||||
Fixed In Version: | Doc Type: | Bug Fix | ||
Doc Text: |
Previously the installer did not correctly format the Docker Registry 2.4 configuration file when using S3 storage. We have corrected this formatting issue and the installer now correctly provisions S3 based registry components when configured to do so.
|
Story Points: | --- | |
Clone Of: | ||||
: | 1371187 (view as bug list) | Environment: | ||
Last Closed: | 2016-09-27 09:40:19 UTC | Type: | Bug | |
Regression: | --- | Mount Type: | --- | |
Documentation: | --- | CRM: | ||
Verified Versions: | Category: | --- | ||
oVirt Team: | --- | RHEL 7.3 requirements from Atomic Host: | ||
Cloudforms Team: | --- | Target Upstream Version: | ||
Embargoed: | ||||
Bug Depends On: | ||||
Bug Blocks: | 1371187 |
Description
zhou ying
2016-07-15 06:44:36 UTC
You're most probably missing some middleware sections from your registry config.yml file. Please update them according to our documentation: https://docs.openshift.org/latest/install_config/install/docker_registry.html#docker-registry-deploying-updated-configuration Please extend your config file at least for: middleware: registry: - name: openshift storage: - name: openshift Does it help? Yes, when extend config file, the issue has fixed,thanks. middleware: registry: - name: openshift storage: - name: openshift repository: - name: openshift options: pullthrough: true openshift version openshift v3.3.0.12 kubernetes v1.3.0+57fb9ac etcd 2.3.0+git When we use native s3 registry config by openshift-ansible, the docker-registry's middleware stanza should contain the following by default: registry: - name: openshift storage: - name: openshift Will close this, and open a new. Michal, Is that additional bit of config valid for 2.0 registry too? As in, should we add this for OSE 3.2 installs as well? (In reply to Scott Dodson from comment #6) > Is that additional bit of config valid for 2.0 registry too? As in, should > we add this for OSE 3.2 installs as well? No, it's not valid. I will be valid if we backport 2.4 registry though [1]. [1] https://github.com/openshift/ose/pull/314 So is there a document bug here? https://docs.openshift.com/enterprise/3.2/install_config/install/docker_registry.html#docker-registry-configuration-reference-middleware @ghunag there was :-). It's fixed now. See [1]. [1] https://github.com/openshift/openshift-docs/pull/2630 Confirmed on today's build env, the issue has fixed. openshift version openshift v3.3.0.18 kubernetes v1.3.0+507d3a7 etcd 2.3.0+git middleware: registry: - name: openshift repository: - name: openshift options: acceptschema2: false pullthrough: true enforcequota: false projectcachettl: 1m blobrepositorycachettl: 10m storage: - name: openshift > middleware: > registry: > - name: openshift > repository: > - name: openshift > options: > acceptschema2: false > pullthrough: true > enforcequota: false > projectcachettl: 1m > blobrepositorycachettl: 10m > storage: > - name: openshift This is a swift config file generated not by openshift-ansible. Currently openshift-ansible doesn't support swift backend storage for docker-registry, so we should check it on AWS. https://github.com/openshift/openshift-ansible/pull/2280 isn't in latest openshift-ansible rpm package. @Michal, we regard this bug as a installer bug. So I move to assigned first, and will check again once fixed in puddle. Gan, That PR should be in both 3.2.22-1 and the 3.3 version today. Marking ON_QA. Docker registry failed to deploy. "repository" is not in the right place as we expect. I'm not sure what's wrong.(it should be in the same level as "registry" and "storage") [root@ip-172-18-6-22 ~]# oc logs docker-registry-6-3j350 time="2016-08-15T06:42:43-04:00" level=fatal msg="Error parsing configuration file: yaml: line 28: did not find expected '-' indicator" Get registry-config: <--snip--> auth: openshift: realm: openshift middleware: registry: - name: openshift repository: - name: openshift options: pullthrough: True storage: - name: openshift Gan, repository section needs to be at the same level as registry and storage like this: auth: openshift: realm: openshift middleware: registry: - name: openshift repository: - name: openshift options: pullthrough: True storage: - name: openshift Oh I see, that's an ansible bug. Sorry for my useless comment. Moving back to ASSIGNED. I'm confused that I got a different config.yml today in ansible-2.2.0-0.5.prerelease.el7.noarch. Not sure the issue is related to ansible version. <--snip--> auth: openshift: realm: openshift middleware: repository: - name: openshift options: pullthrough: True https://github.com/openshift/openshift-ansible/pull/2314 fixes the padding. Part of the difficulty in testing this is that the secret is not overwritten on subsequent runs. We should decide if we should do that or not in the future. I tried it out using master branch. repository is at the right level. But looks like "openshift.common.version_gte_3_3_or_1_3" was set to false. Get the config.yml: <--snip--> auth: openshift: realm: openshift middleware: repository: - name: openshift options: pullthrough: True Remove the dependence of BZ#1368034 due to it's a edge case to set "openshift_image_tag=v3.3". Confirmed on aws, the issue has fixed: middleware: registry: - name: openshift repository: - name: openshift options: pullthrough: True storage: - name: openshift openshift version openshift v3.3.0.24-dirty kubernetes v1.3.0+507d3a7 etcd 2.3.0+git will verify it. 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/RHBA-2016:1933 |