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

Clarify jobname for app-server port #4029

Merged
merged 7 commits into from
Dec 9, 2024
Merged

Conversation

1000TurquoisePogs
Copy link
Member

The table that shows which jobs to associate with which ports for Zowe server networking is missing detail on a quirk of NodeJS+zOS by which the jobname of the app-server that uses the stated port may not be what's intended, depending on configuration.

I've updated the table to show 2 jobnames for app-server, and then added a section directly below to clarify when and how these jobnames take affect, and what the defaults are.

Copy link

github-actions bot commented Dec 5, 2024

😺 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 Dec 5, 2024

📁 The PR description is missing the file name(s) for the updated content. List all the files included in this PR so this information displays in our Zowe Docs GitHub Slack channel.

If you have addressed this issue already, refresh this page in your browser to remove this comment.

Copy link

github-actions bot commented Dec 5, 2024

@github-actions github-actions bot temporarily deployed to pull request December 5, 2024 14:03 Inactive
@1000TurquoisePogs 1000TurquoisePogs added area: install and config Issues related to z/OS install and config release: V2 Zowe V2 content release: V3 Zowe V3 content review: doc Needs review by Doc Squad member labels Dec 5, 2024
@github-actions github-actions bot temporarily deployed to pull request December 5, 2024 15:11 Inactive
@1000TurquoisePogs
Copy link
Member Author

Broken link checker reports
"* [520] https://testssl.net/openssl-iana.mapping.html | Failed: Network error: Unknown status code"

This just means the site is broken right now. I assume it's some downtime and will return tomorrow.

@github-actions github-actions bot temporarily deployed to pull request December 9, 2024 13:13 Inactive
Copy link
Collaborator

@ArooshLele ArooshLele left a comment

Choose a reason for hiding this comment

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

I approve of these updates and am signing off.

@ArooshLele ArooshLele merged commit 8087456 into master Dec 9, 2024
3 of 4 checks passed
@ArooshLele ArooshLele deleted the app-server-jobname-quirk branch December 9, 2024 13:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: install and config Issues related to z/OS install and config release: V2 Zowe V2 content release: V3 Zowe V3 content review: doc Needs review by Doc Squad member
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants