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

Set the contour image version used by the contour-operator #12

Merged

Conversation

arjunrn
Copy link

@arjunrn arjunrn commented Sep 23, 2021

Currently the contour-operator uses the "latest" image to provision the resources for a Contour resource. However the not every contour image is guaranteed to work with every operator version. This change fixes the contour image version used by the operator. When updating the operator the contour image version will also have to be updated.

Currently the contour-operator uses the "latest" image to provision the
resources for a Contour resource. However the not every contour image is
guaranteed to work with every operator version. This change fixes the
contour image version used by the operator. When updating the operator
the contour image version will also have to be updated.
@arjunrn arjunrn changed the title Fix the contour image version used by the contour-operator Set the contour image version used by the contour-operator Sep 24, 2021
@@ -24,3 +24,4 @@ spec:
args:
- "--metrics-addr=127.0.0.1:8080"
- "--enable-leader-election"
- "--contour-image=docker.io/projectcontour/contour:v1.15.1"

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I noticed that there is a v1.16.0 is that not applicable ?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I haven't tested it with that particular version of contour. Once the CI is ready it should be easy to test the newer versions. Let's merge this.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm

@lmzuccarelli
Copy link

lmzuccarelli commented Sep 24, 2021

/approve

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Sep 24, 2021
@lmzuccarelli
Copy link

/approve

@openshift-ci
Copy link

openshift-ci bot commented Sep 24, 2021

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: arjunrn, lmzuccarelli

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

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Sep 24, 2021
@openshift-merge-robot openshift-merge-robot merged commit e07f706 into openshift:main Sep 24, 2021
@arjunrn arjunrn deleted the fix-contour-image-version branch September 24, 2021 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants