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 README Management Center Hello World command #916

Merged

Conversation

JackPGreen
Copy link
Collaborator

@JackPGreen JackPGreen commented Mar 11, 2025

When the current command is run:

/opt/hazelcast/management-center/bin/hz-mc: line 97: ./mc-conf.sh: No such file or directory

Updated to use new location, and avoids deprecated mc-conf.sh as well (#909).

Example successful output:

Executing command specified by MC_INIT_CMD.
Successfully added Cluster Config.
+ exec java -server -Dloader.path=/opt/hazelcast/management-center/bin/user-lib --add-opens java.base/java.lang=ALL-UNNAMED --add-opens java.base/sun.nio.ch=ALL-UNNAMED -Dhazelcast.mc.home=/data -Djava.net.preferIPv4Stack=true -XX:+UseContainerSupport -XX:MaxRAMPercentage=80 -cp /opt/hazelcast/management-center/bin/../hazelcast-management-center-5.7.1.jar org.springframework.boot.loader.launch.PropertiesLauncher

Fixes: #909

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
When the current command is run:
```
/opt/hazelcast/management-center/bin/hz-mc: line 97: ./mc-conf.sh: No such file or directory
```

Updated to use new location, and avoids deprecated `hz-mc` as well (#909).

Example successful output:
```
Executing command specified by MC_INIT_CMD.
Successfully added Cluster Config.
+ exec java -server -Dloader.path=/opt/hazelcast/management-center/bin/user-lib --add-opens java.base/java.lang=ALL-UNNAMED --add-opens java.base/sun.nio.ch=ALL-UNNAMED -Dhazelcast.mc.home=/data -Djava.net.preferIPv4Stack=true -XX:+UseContainerSupport -XX:MaxRAMPercentage=80 -cp /opt/hazelcast/management-center/bin/../hazelcast-management-center-5.7.1.jar org.springframework.boot.loader.launch.PropertiesLauncher
```

Fixes: #909

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@JackPGreen JackPGreen self-assigned this Mar 11, 2025
@JackPGreen JackPGreen requested a review from a team as a code owner March 11, 2025 15:05
@JackPGreen JackPGreen enabled auto-merge (squash) March 11, 2025 15:05
@nishaatr
Copy link
Contributor

Updated to use new location, and avoids deprecated hz-mc as well (#909).

You mean mc-conf.sh? please correct description if thats the case

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
…d`-command
@JackPGreen JackPGreen merged commit ee27594 into master Mar 14, 2025
17 checks passed
@JackPGreen JackPGreen deleted the Fix-`README`-`Management-Center-Hello-World`-command branch March 14, 2025 13:05
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.

README uses deprecated MC mc-conf.sh
3 participants