-
Notifications
You must be signed in to change notification settings - Fork 143
Updated configuring-at-tls-for-zowe-server.md #4732
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
base: master
Are you sure you want to change the base?
Conversation
Signed-off-by: arooshlele <[email protected]>
|
😺 Thank you for creating this PR! To publish your content to Zowe Docs, follow these required steps.
Need help? Contact the Doc Squad in the #zowe-doc Slack channel. |
| # Enabling AT-TLS | ||
|
|
||
| :::caution | ||
| Do not use AT-TLS in high availability until you install Zowe on z/OS. First, ensure that the components of Zowe are functional and run as expected. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
'in high availability'? I don't see how HA is related.
My opinion is people should not try an AT-TLS setup until they have a non-ATTLS setup working first. There are situations in which non-ATTLS is not possible and therefore the recommendation doesn't make sense, but for most people it is.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@1000TurquoisePogs Thanks for your review. I have removed HA from the Note.
Based on your feedback, here's the Pull Request: #4744
This Pull Request is now merged.
The updated Note is as follows:
Do not use an AT-TLS setup until you have a non-AT-TLS setup that works.
For more information about how to install Zowe, refer to Installing Zowe.
For more information about how to verify whether Zowe components are functional, refer to Verifying Zowe installation on z/OS.
CC: @ArooshLele @gauravs-20
|
|
||
| :::caution | ||
| Do not use AT-TLS in high availability until you install Zowe on z/OS. First, ensure that the components of Zowe are functional and run as expected. | ||
| For more information about how to install Zowe, refer to [Installing Zowe](../user-guide/install-overview.md) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add punctuation after links
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@janan07 Thanks for your review. I have added the punctuation marks after the links.
Here's the Pull Request: #4744
This Pull Request (containing the punctuation marks) is now merged.
CC: @ArooshLele @gauravs-20
|
Updated w/ https://github.com/zowe/docs-site/pull/4744/files + Aroosh and I will make minor correction to 4744 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you @ArooshLele!
| # Enabling AT-TLS | ||
|
|
||
| :::caution | ||
| Do not use an AT-TLS setup until you have a non-AT-TLS setup that works. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Similarly, do not attempt a Zowe HA setup until you have a non-HA Zowe setup that works.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add both sentences into here too: https://docs.zowe.org/stable/user-guide/zowe-ha-overview/
(or just the HA one)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is tricky recommendation as up until at least 3.4, maybe 3.5 the hardware based ICSF Keyrings requires usage of AT-TLS and it won't work without it.
Otherwise I agree.
| # Enabling AT-TLS | ||
|
|
||
| :::caution | ||
| Do not use an AT-TLS setup until you have a non-AT-TLS setup that works. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add both sentences into here too: https://docs.zowe.org/stable/user-guide/zowe-ha-overview/
(or just the HA one)
List the file(s) included in this PR:
configuring-at-tls-for-zowe-server.md
Describe your pull request here:
Based on my discussion with @1000TurquoisePogs , I added a cautionary note at the beginning of the "Enabling AT-TLS" topic.
After creating the PR, follow the instructions in the comments.
CC: @ArooshLele @gauravs-20