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

Added more configs to dev ispn configuration for console test needs. #11003

Merged
merged 1 commit into from
Jun 13, 2023

Conversation

andyuk1986
Copy link
Contributor

@andyuk1986 andyuk1986 commented May 30, 2023

backport #10999

karesti
karesti previously approved these changes Jun 1, 2023
@karesti
Copy link
Contributor

karesti commented Jun 1, 2023

@tristantarrant what do you think on this one?

@tristantarrant
Copy link
Member

I'm ok with the metrics. I'm not ok with the default cache and the configuration template. Those do not exist in a real server, and building against them would create a disalignment.

<transport cluster="${infinispan.cluster.name:cluster}" stack="${infinispan.cluster.stack:tcp}" node-name="${infinispan.node.name:}"/>
<security>
<authorization/>
</security>
<local-cache name="default"/>
Copy link
Member

Choose a reason for hiding this comment

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

Replace with a distributed cache that inherits from the template defined below.

Copy link
Contributor

Choose a reason for hiding this comment

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

I pushed a change

@tristantarrant tristantarrant merged commit 7601fec into infinispan:main Jun 13, 2023
3 of 4 checks passed
@tristantarrant
Copy link
Member

Merged, thanks

@andyuk1986
Copy link
Contributor Author

Hi team, sorry - seems that I have missed the email notifications from this PR. Thanks for the changes and merging. Anna

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants