Skip to content

Added web console steps to create subscription for logging#13451

Merged
mburke5678 merged 1 commit intoopenshift:enterprise-4.0from
mburke5678:logging-deploy-subscription
Feb 15, 2019
Merged

Added web console steps to create subscription for logging#13451
mburke5678 merged 1 commit intoopenshift:enterprise-4.0from
mburke5678:logging-deploy-subscription

Conversation

@mburke5678
Copy link
Contributor

@mburke5678 mburke5678 commented Jan 25, 2019

No description provided.

@openshift-ci-robot openshift-ci-robot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Jan 25, 2019
@mburke5678
Copy link
Contributor Author

mburke5678 commented Jan 25, 2019

@jcantrill PTAL I added the steps to use the OpenShift web console to create the cluster logging subscription. What else is required to install cluster logging? What is no longer needed.

@mburke5678 mburke5678 added this to the Future Release milestone Jan 25, 2019
@openshift-ci-robot openshift-ci-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Feb 9, 2019
@mburke5678 mburke5678 force-pushed the logging-deploy-subscription branch from 6f440a5 to 6ba735b Compare February 11, 2019 14:53
@openshift-ci-robot openshift-ci-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Feb 11, 2019
@openshift-docs-preview-bot
Copy link

openshift-docs-preview-bot commented Feb 11, 2019

The preview will be availble shortly at:

@mburke5678
Copy link
Contributor Author

mburke5678 commented Feb 11, 2019

@jcantrill PTAL I added the steps to use the OpenShift web console to create the cluster logging subscription. What else is required to install cluster logging? What is no longer needed.

jcantrill
jcantrill reviewed Feb 15, 2019
View reviewed changes
modules/efk-logging-deploy-subscription.adoc Outdated
Copy link

@jcantrill jcantrill Feb 15, 2019

Choose a reason for hiding this comment

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

We'll need to revisit this once various changes to OLM merge. The workflow is different.

@openshift-ci-robot openshift-ci-robot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. labels Feb 15, 2019
@mburke5678
Copy link
Contributor Author

mburke5678 commented Feb 15, 2019

@jcantrill Are all these tasks still required to install cluster logging?

https://logging-deploy-subscription--ocpdocs.netlify.com/openshift-enterprise/latest/logging/efk-logging-deploy.html

@jcantrill
Copy link

jcantrill commented Feb 15, 2019

@jcantrill Are all these tasks still required to install cluster logging?

https://logging-deploy-subscription--ocpdocs.netlify.com/openshift-enterprise/latest/logging/efk-logging-deploy.html

No. The primary goal of operators is to make it easy. We'll need to revisit this to clarify. I suspect there will be a lot of "gutting" of the documentation.

@mburke5678 mburke5678 force-pushed the logging-deploy-subscription branch from d6bf099 to 87dc2d3 Compare February 15, 2019 23:22
@mburke5678
Copy link
Contributor Author

mburke5678 commented Feb 15, 2019

Merging at the request of Jeff Cantrill to facilitate logging docs review.

@mburke5678 mburke5678 merged commit 6e144ab into openshift:enterprise-4.0 Feb 15, 2019
@mburke5678 mburke5678 deleted the logging-deploy-subscription branch February 15, 2019 23:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jcantrill jcantrill jcantrill left review comments

Assignees

No one assigned

Labels

branch/enterprise-4.1 size/M Denotes a PR that changes 30-99 lines, ignoring generated files.

Projects

None yet

Milestone

OCP 4.1 GA

Development

Successfully merging this pull request may close these issues.

5 participants

@mburke5678 @openshift-docs-preview-bot @jcantrill @vikram-redhat @openshift-ci-robot