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-12759 Extract microprofile-config galleon layer from observability #12776

Merged
merged 2 commits into from Nov 7, 2019

Conversation

pferraro
Copy link
Contributor

@pferraro pferraro commented Nov 5, 2019

https://issues.jboss.org/browse/WFLY-12759

Microprofile specs like OpenAPI don't require metrics nor health, suggesting the microprofile layers should be more granular.

@wildfly-ci wildfly-ci added the deps-ok Dependencies have been checked, and there are no significant changes label Nov 5, 2019
Copy link
Contributor

@bstansberry bstansberry left a comment

Choose a reason for hiding this comment

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

This will need an entry in the testsuite/layers/pom.xml. All layers should have an entry there as that is the basic boot validation for the layer.

It also needs an update to https://github.com/wildfly/wildfly/blob/master/docs/src/main/asciidoc/_admin-guide/Galleon_provisioning.adoc

@pferraro
Copy link
Contributor Author

pferraro commented Nov 6, 2019

@bstansberry Fixed.

@bstansberry bstansberry added ready-for-merge Only for use by those with merge permissions! and removed ready-for-merge Only for use by those with merge permissions! labels Nov 6, 2019
@bstansberry
Copy link
Contributor

@jmesnil OK with you?

Copy link
Member

@rhusar rhusar left a comment

Choose a reason for hiding this comment

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

Thanks for this. Looks good.

@rhusar
Copy link
Member

rhusar commented Nov 7, 2019

Actually, would it make any sense to document this in docs/src/main/asciidoc/_admin-guide/Galleon_provisioning.adoc? Seems unnecessary as its basically not useful by itself.

@bstansberry
Copy link
Contributor

@rhusar MicroProfile Config is usable directly by applications, so any layer at all could possibly be decorated with this one.

It seems simpler to document them all too.

@bstansberry bstansberry mentioned this pull request Nov 7, 2019
@bstansberry bstansberry merged commit 65d2355 into wildfly:master Nov 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deps-ok Dependencies have been checked, and there are no significant changes
Projects
None yet
4 participants