Skip to content

createDomain create LogFilter, XMLEntity Cache, and XML Registry earlier #872

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

Merged
merged 3 commits into from
Apr 19, 2021

Conversation

ddsharpe
Copy link
Member

Create LogFilter, XMLEntity Cache, and XML Registry earlier in the flow. Server/Log depends on LogFilter. Fixes #871

Copy link
Member

@rakillen rakillen left a comment

Choose a reason for hiding this comment

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

This covers the "select template" create, but needs to be implemented for __extend_domain (older) create.

Copy link
Contributor

@CarolynRountree CarolynRountree left a comment

Choose a reason for hiding this comment

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

update javadoc to reflect removal of location

@@ -663,16 +665,21 @@ def __set_core_domain_params(self):
self.logger.exiting(class_name=self.__class_name, method_name=_method_name)
return

def __create_mbeans_used_by_topology_mbeans(self, location, topology_folder_list):
def __create_mbeans_used_by_topology_mbeans(self, topology_folder_list):
"""
Create the entities that are referenced by domain, machine, server and server template attributes.
:param location: current location
Copy link
Contributor

Choose a reason for hiding this comment

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

this javadoc is out of date.

@ddsharpe ddsharpe merged commit 7f9c97f into master Apr 19, 2021
@ddsharpe ddsharpe deleted the move-topology-mbeans-create branch April 19, 2021 14:43
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.

LogFilter not found
3 participants