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 etcd and prometheus to latest operator versions #204

Merged

Conversation

ecordell
Copy link
Member

@ecordell ecordell commented Dec 7, 2017

No description provided.

@@ -27,7 +27,7 @@ spec:
**Backups included**
Coming soon, the ability to schedule backups to happen on or off cluster.
keywords: ['etcd', 'key value', 'database', 'coreos', 'open source']
version: 0.6.1
version: 0.7.1
Copy link
Contributor

Choose a reason for hiding this comment

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

Does this need a replaces?

Copy link
Member Author

Choose a reason for hiding this comment

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

It only needs a replaces when we want to upgrade existing installs, so for beta I don't think so

(it also seems like we should have a CSV for each version in between 0.6.1 and 0.7.1 if we are going to do that)

jakedt
jakedt previously approved these changes Dec 7, 2017
Copy link
Member

@alecmerdler alecmerdler left a comment

Choose a reason for hiding this comment

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

etcd changed their image to be hosted on GCR, so we can't merge this.

Failed to pull image "gcr.io/etcd-development/etcd:v3.1.4": rpc error: code = Unknown desc = unauthorized: authentication required

@alecmerdler alecmerdler dismissed jakedt’s stale review December 7, 2017 18:12

Does not work on an actual cluster because of etcd image on GCR

@ecordell ecordell force-pushed the ALM-268/latest-catalog-versions branch from f512283 to 609f888 Compare December 7, 2017 19:41
@ecordell ecordell force-pushed the ALM-268/latest-catalog-versions branch from 609f888 to 2fb7208 Compare December 7, 2017 19:58
Copy link
Member

@alecmerdler alecmerdler left a comment

Choose a reason for hiding this comment

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

LGTM

@ecordell ecordell merged commit 327e810 into operator-framework:master Dec 7, 2017
njhale pushed a commit to njhale/operator-lifecycle-manager that referenced this pull request Sep 10, 2018
…t-catalog-versions

Update etcd and prometheus to latest operator versions
ecordell added a commit to ecordell/operator-lifecycle-manager that referenced this pull request Mar 8, 2019
…t-catalog-versions

Update etcd and prometheus to latest operator versions
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants