Skip to content
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

Update nginx.conf to be compatible with nginx imagestreams on OCP 4.6 + #161

Merged
merged 3 commits into from
Apr 25, 2021

Conversation

fmenesesg
Copy link
Contributor

@fmenesesg fmenesesg commented Apr 20, 2021

What does this PR do?

Update nginx.conf to be compatible with latest nginx image stream (nginx:1.18-ubi8) and be able to use this app with pelorus examples

How should this be tested?

  1. On an OCP 4.6 o later, create a new project

oc new-project nginx-test

  1. Create the nginx app using images stream nginx:1.18-ubi8

oc new-app nginx:1.16-el8~https://github.com/fmenesesg/container-pipelines.git --context-dir=/basic-nginx

Is there a relevant Issue open for this?

resolves #160

Who would you like to review this?

cc: @redhat-cop/containers-approvers

@varodrig
Copy link

please assign @oybed

@garethahealy
Copy link
Contributor

/assign @oybed

@varodrig
Copy link

QA raised an issue. Francisco will be reviewing.

@fmenesesg
Copy link
Contributor Author

QA raised an issue. Francisco will be reviewing.

Fixed, I changed the image stream to nginx:1.16-el8 because nginx:1.18-ubi8 doesn't exist on OCP 4.6

@varodrig
Copy link

varodrig commented Apr 22, 2021

It's working now in OCP 4.6
image

@varodrig
Copy link

varodrig commented Apr 22, 2021

Can you please approve? thank you!
@etsauer
@garethahealy
@oybed
@pcarney8
@pabrahamsson
@sabre1041

@garethahealy
Copy link
Contributor

garethahealy commented Apr 23, 2021

have tried this on the opentlc shared cluster (https://console-openshift-console.apps.shared-na46.openshift.opentlc.com/), but that imagestream doesn't exist:

oc new-app nginx:1.18-ubi8~https://github.com/redhat-cop/container-pipelines.git --context-dir=/basic-nginx
error: multiple images or templates matched "nginx:1.18-ubi8"

The argument "nginx:1.18-ubi8" could apply to the following Docker images, OpenShift image streams, or templates:

* Image stream "nginx" (tag "1.14-el8") in project "openshift"
  Use --image-stream="openshift/nginx:1.14-el8" to specify this image or template

* Image stream "nginx" (tag "1.16-el7") in project "openshift"
  Use --image-stream="openshift/nginx:1.16-el7" to specify this image or template

* Image stream "nginx" (tag "1.16-el8") in project "openshift"
  Use --image-stream="openshift/nginx:1.16-el8" to specify this image or template

* Image stream "nginx" (tag "latest") in project "openshift"
  Use --image-stream="openshift/nginx:latest" to specify this image or template

@fmenesesg
Copy link
Contributor Author

fmenesesg commented Apr 23, 2021

have tried this on the opentlc shared cluster (https://console-openshift-console.apps.shared-na46.openshift.opentlc.com/), but that imagestream doesn't exist:

oc new-app nginx:1.18-ubi8~https://github.com/redhat-cop/container-pipelines.git --context-dir=/basic-nginx
error: multiple images or templates matched "nginx:1.18-ubi8"

The argument "nginx:1.18-ubi8" could apply to the following Docker images, OpenShift image streams, or templates:

* Image stream "nginx" (tag "1.14-el8") in project "openshift"
  Use --image-stream="openshift/nginx:1.14-el8" to specify this image or template

* Image stream "nginx" (tag "1.16-el7") in project "openshift"
  Use --image-stream="openshift/nginx:1.16-el7" to specify this image or template

* Image stream "nginx" (tag "1.16-el8") in project "openshift"
  Use --image-stream="openshift/nginx:1.16-el8" to specify this image or template

* Image stream "nginx" (tag "latest") in project "openshift"
  Use --image-stream="openshift/nginx:latest" to specify this image or template

I forget to update the test instructions, now they are, please execute:

oc new-app nginx:1.16-el8~https://github.com/fmenesesg/container-pipelines.git --context-dir=/basic-nginx

@garethahealy
Copy link
Contributor

/approve
/lgtm

@redhat-cop-ci-bot
Copy link

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: garethahealy

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@redhat-cop-ci-bot redhat-cop-ci-bot merged commit 15dedb6 into redhat-cop:master Apr 25, 2021
@varodrig
Copy link

thank you so much!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Image stream not found during build process - base nginx image
5 participants