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

6940_April_2_SafKeyringConfigSlashes #4277

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

ArooshLele
Copy link
Collaborator

Describe your pull request here:
The following is mentioned:
While migrating from Zowe v2.18, if you are using keyrings, verify that Zowe YAML references to safkeyring use 2 slashes, not 4, such as safkeyring:// instead of safkeyring:////

List the file(s) included in this PR:

  • zowe-v3-migration.md
  • release-notes/v2_18_0.md

After creating the PR, follow the instructions in the comments.

The following is mentioned:
While migrating from Zowe v2.18, if you are using keyrings, verify that Zowe YAML references to `safkeyring` use 2 slashes, not 4, such as `safkeyring://` instead of `safkeyring:////`
@ArooshLele ArooshLele added area: docs Documentation issue or issues that have documentation impact status: in-progress area: apiml This issue is related to Zowe API Mediation Layer area: misc This issue is a miscelleneous doc requirement not for a specific component or area for-review priority-medium release: V2 Zowe V2 content review: doc Needs review by Doc Squad member review: sme Needs review by an SME labels Apr 2, 2025
@ArooshLele ArooshLele self-assigned this Apr 2, 2025
Copy link

github-actions bot commented Apr 2, 2025

😺 Thank you for creating this PR! To publish your content to Zowe Docs, follow these required steps.

  • Add the label review: doc.
  • Identify your content topic with a label. (Examples: area: apiml, area: cli, area: install and config, etc.)
  • Specify the major Zowe release(s) for your content. (Examples: release: V1, release: V2, release: V3)
    • If adding content that needs to be removed from V3 documentation, add the V3 N/A tag.
  • Select the master branch if your PR updates content that is on the live site. Select docs-staging if your PR updates content for a future release.
  • Notify the Doc Squad about this PR. If you don't know whom should review your content, message the #zowe-doc Slack channel. If you know which Doc Squad writer should approve your content, add that person as a reviewer.

Need help? Contact the Doc Squad in the #zowe-doc Slack channel.

1 similar comment
Copy link

github-actions bot commented Apr 2, 2025

😺 Thank you for creating this PR! To publish your content to Zowe Docs, follow these required steps.

  • Add the label review: doc.
  • Identify your content topic with a label. (Examples: area: apiml, area: cli, area: install and config, etc.)
  • Specify the major Zowe release(s) for your content. (Examples: release: V1, release: V2, release: V3)
    • If adding content that needs to be removed from V3 documentation, add the V3 N/A tag.
  • Select the master branch if your PR updates content that is on the live site. Select docs-staging if your PR updates content for a future release.
  • Notify the Doc Squad about this PR. If you don't know whom should review your content, message the #zowe-doc Slack channel. If you know which Doc Squad writer should approve your content, add that person as a reviewer.

Need help? Contact the Doc Squad in the #zowe-doc Slack channel.

Copy link

github-actions bot commented Apr 2, 2025

@github-actions github-actions bot temporarily deployed to pull request April 2, 2025 14:47 Inactive
Copy link
Collaborator

@anaxceron anaxceron left a comment

Choose a reason for hiding this comment

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

Made a couple of comments re: style.

@@ -34,6 +34,7 @@ To migrate from Zowe **v2.16.0** or a lower version, perform the following tasks

2) Update your PROCLIB entries for Zowe, as enhancements and default parameters have changed throughout Zowe v2.
This can be performed with the unix command `zwe init stc`, by running the job ZWEISTC, or by copying the SZWESAMP members ZWESLSTC, ZWESISTC, and ZWESASTC into your desired PROCLIB.
3) If you are using keyrings, verify that Zowe YAML references to `safkeyring` use 2 slashes, not 4, such as `safkeyring://` instead of `safkeyring:////`.
Copy link
Collaborator

Choose a reason for hiding this comment

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

We should spell out 2 and 4. Typically, single-digit numbers are spelled out (when not used in measurements).

Copy link
Collaborator

Choose a reason for hiding this comment

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

That sentence was taken from the same article, it should be identical on both places.

@@ -28,6 +28,8 @@ System demos are typically held the week after a minor release becomes available

* It is now possibile to use custom logback.xml in all API ML services. Configuration for separate services can be performed by adding logging.config parameter under each API ML component in Zowe YAML file. ([#3669](https://github.com/zowe/api-layer/issues/3669))

* While migrating from Zowe v2.18, if you are using keyrings, verify that Zowe YAML references to `safkeyring` use 2 slashes, not 4, such as `safkeyring://` instead of `safkeyring:////`.
Copy link
Collaborator

Choose a reason for hiding this comment

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

  • We should spell out 2 and 4. Typically, single-digit numbers are spelled out (when not used in measurements).
  • To tighten up this sentence, I would include the examples in parentheses after the slash counts: " ... use two forward slashes (safkeyring://), not four (safkeyring:////).

@github-actions github-actions bot temporarily deployed to pull request April 3, 2025 05:13 Inactive
@github-actions github-actions bot temporarily deployed to pull request April 3, 2025 14:22 Inactive
@github-actions github-actions bot temporarily deployed to pull request April 7, 2025 11:28 Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: apiml This issue is related to Zowe API Mediation Layer area: docs Documentation issue or issues that have documentation impact area: misc This issue is a miscelleneous doc requirement not for a specific component or area for-review priority-medium release: V2 Zowe V2 content review: doc Needs review by Doc Squad member review: sme Needs review by an SME status: in-progress
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants