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

WINDUP-2261: Updated OS details to indicate shared storage. #247

Merged
merged 1 commit into from Jan 25, 2019

Conversation

huffmanca
Copy link
Contributor

Updated the OpenShift section of the Web Console guide to indicate a shared persistent volume.

This PR should not be merged until the image has been updated; the current one is a placeholder until one can be received from the graphics design team.

Copy link
Member

@mrizzi mrizzi left a comment

Choose a reason for hiding this comment

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

Thanks @huffmanca, the changes look very good: clear and precise 👍

Reading the whole "Understanding the Web Console OpenShift Image" paragraph, i think we could do some more little changes to describe the images we provide.
I write here my proposed changes:

Red Hat offers a some containerized image images for RHAMT ’s web console that is are designed for use with OpenShift. Using this these image images, developers can quickly and easily manage migration projects and analyze applications.

The OpenShift image images includes include three separate pods:

  • The executor, responsible for generating running the analysis and generating the reports
  • The web console interface and SSO, which provide access to the web console itself
  • The PostgreSQL database, which stores the project’s configuraton and analysis tracking information

@huffmanca wdyt?

@huffmanca
Copy link
Contributor Author

@mrizzi,

I think these changes sound good. I made a couple of minor modifications to your proposal, but the singular/plural of this section should be adjusted to correctly represent that we ship multiple images!

Copy link
Member

@mrizzi mrizzi left a comment

Choose a reason for hiding this comment

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

Thanks @huffmanca for the latest changes: for me it's perfect to be merged 👍

@huffmanca huffmanca changed the title [WIP] WINDUP-2261: Updated OS details to indicate shared storage. WINDUP-2261: Updated OS details to indicate shared storage. Jan 25, 2019
@huffmanca huffmanca merged commit 98c76ae into windup:master Jan 25, 2019
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

2 participants