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

fix: update defaults for infinispan in v3 #4158

Merged
merged 2 commits into from
Jan 29, 2025

Conversation

pablocarle
Copy link
Contributor

  • Tests for the changes have been added (for bug fixes / features)
  • Necessary documentation (if appropriate) have been added / updated
  • DCO signoffs have been added to all commits, including this PR

PR type

  • Bugfix

Changes proposed in this PR

  • API ML assumes port 7098 for JGroups and 7099 for the key exchange port. The documentation establishes that if infinispan is configured, it should be set in zowe.yaml. The current defaults are in conflict.

Does this PR introduce a breaking change?

  • Yes
  • No

Copy link

github-actions bot commented Jan 29, 2025

build 6871 SUCCEEDED.
Link to workflow run: https://github.com/zowe/zowe-install-packaging/actions/runs/13027557499

Copy link

github-actions bot commented Jan 29, 2025

Test workflow 6037 is started.
Running install test: Convenience Pax
The zowe artifact being used by this test workflow: libs-snapshot-local/org/zowe/3.1.0-PR-4158/zowe-3.1.0-pr-4158-6871-20250129084727.pax
Running on machine: zzow09
Result: SUCCESS
Link to workflow run: https://github.com/zowe/zowe-install-packaging/actions/runs/13027628145

@MarkAckert MarkAckert added the Build: None Used in CI label Jan 29, 2025
@1000TurquoisePogs
Copy link
Member

The current defaults match the documentation: https://docs.zowe.org/stable/user-guide/address-network-requirements
Is there an associated doc update?

@MarkAckert MarkAckert merged commit 2b50022 into v3.x/staging Jan 29, 2025
9 checks passed
@MarkAckert MarkAckert deleted the reboot/fix/defaults-infinispan branch January 29, 2025 16:46
@pj892031
Copy link
Contributor

The current defaults match the documentation: https://docs.zowe.org/stable/user-guide/address-network-requirements Is there an associated doc update?

It would be fixed by zowe/docs-site#4101

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Build: None Used in CI
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

5 participants