Skip to content

Add docs on how to upgrade the metrics components between versions. #2122

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

Merged
merged 1 commit into from
Jun 2, 2016

Conversation

mwringe
Copy link

@mwringe mwringe commented May 18, 2016

No description provided.

@mwringe
Copy link
Author

mwringe commented Jun 2, 2016

Bump, any news on this?

@tnguyen-rh
Copy link

@mwringe This is for 3.2 only, right?

@mwringe
Copy link
Author

mwringe commented Jun 2, 2016

@tnguyen-rh yes, its for when people are moving between 3.1 and 3.2 (since that is the only main version update they have so far). But it will also be required for going between 3.2 and 3.3, etc

@tnguyen-rh
Copy link

@mwringe Thanks. For future reference, feel free to add the label branch/enterprise-3.2 directly, and tag me or @openshift/team-documentation to call (more) timely attention to the PR.

@tnguyen-rh tnguyen-rh merged commit 3626081 into openshift:master Jun 2, 2016
@tnguyen-rh tnguyen-rh self-assigned this Jun 2, 2016
@adellape adellape added this to the Next Release milestone Jun 7, 2016
@adellape
Copy link
Contributor

adellape commented Jun 7, 2016

@tnguyen-rh Added a milestone.

@mwringe
Copy link
Author

mwringe commented Jun 7, 2016

Any idea when this will appear in the 3.2 docs? I can see it in the origin docs but not the 3.2 ones.

@bfallonf
Copy link

bfallonf commented Jun 7, 2016

@mwringe Once a PR is merged it's in Origin fairly quickly, but isn't released until the followup label (to double-check if it hits all the CCS standards) is removed and after our weekly async release.

Does this need to be in the Enterprise docs quickly?

@tnguyen-rh
Copy link

@mwringe To expand on what @bfallonf said, the to_followup label will be removed once #2234 is merged.

You will now need to run:
----
$ oc new-app -f metrics-deployer.yaml \
-p HAWKULAR_METRICS_HOSTNAME=hawkular-metrics.example.com,MODE=redeploy

Choose a reason for hiding this comment

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

@mwringe
Hmm, there is a discrepency between this example (which uses redeploy) and the introductory text (which uses refresh). Which is correct? (I will make the change in #2234.)

Copy link
Author

Choose a reason for hiding this comment

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

Oops, it should be 'refresh' not 'redeploy' here. Thanks for the catch

tnguyen-rh added a commit to tnguyen-rh/openshift-docs that referenced this pull request Jun 22, 2016
- Use table for post-reboot "next step".
- Simplify "Upgrading Cluster Metrics" (automatic).
- Simplify "Upgrading Cluster Metrics" (manual).
@tnguyen-rh
Copy link

PR #2234 has rev history and has merged. Removing to_followup label.

@adellape adellape modified the milestones: Next Release, Staging Jul 11, 2016
@mwringe
Copy link
Author

mwringe commented Jul 12, 2016

@adellape I still don't see this in the OSE 3.2 docs, this should have been a high priority fix as its essential for anyone updating to OSE 3.2 who had metrics deployed before.

@adellape
Copy link
Contributor

@mwringe It's in the Staging milestone currently, meaning we're currently working on cherry-picking and publishing to OSE docs. I'll bump you when it's live.

adellape pushed a commit to adellape/openshift-docs that referenced this pull request Jul 12, 2016
- Use table for post-reboot "next step".
- Simplify "Upgrading Cluster Metrics" (automatic).
- Simplify "Upgrading Cluster Metrics" (manual).
@adellape adellape modified the milestones: Staging, Staging (Picked) Jul 12, 2016
@adellape
Copy link
Contributor

@mwringe
Copy link
Author

mwringe commented Jul 15, 2016

@adellape awesome, thanks so much!

@adellape adellape modified the milestones: Staging (Picked), Staging, Published - 7/14/16 - 7/21/16 Jul 18, 2016
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.

4 participants