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

[WFLY-19651]: Restore configuration CLI commands produce a warning messages for remote-helloworld-mdb QS. #950

Merged
merged 1 commit into from
Sep 24, 2024

Conversation

ehsavoie
Copy link
Contributor

  • Adding the server with all it required in a batch.

Jira: https://issues.redhat.com/browse/WFLY-19651

…ssages for remote-helloworld-mdb QS.

* Adding the server with all it required in a batch.

Jira: https://issues.redhat.com/browse/WFLY-19651

Signed-off-by: Emmanuel Hugonnet <[email protected]>
@emmartins
Copy link
Contributor

@kstekovi can you please review this one?

/subsystem=ee/service=default-bindings:write-attribute(name=jms-connection-factory, value=java:jboss/RemoteConnectionFactory)
reload
Copy link
Contributor

Choose a reason for hiding this comment

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

I think you need to remove this reload, cause such command doesn't work well with any provisioning

Copy link
Contributor

Choose a reason for hiding this comment

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

Nevermind, false alarm, this script is not executed when provisioning.

Copy link
Contributor

Choose a reason for hiding this comment

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

The reload is not required. In The README is manual step for stop and review the configuration. Then it is followed by a start command.

@kstekovi
Copy link
Contributor

Hi @ehsavoie The reported warning message is away. Thank you very much for fix.

@emmartins emmartins merged commit ade117c into wildfly:main Sep 24, 2024
17 checks passed
@emmartins
Copy link
Contributor

thanks both

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.

3 participants