WFLY-2527 Don't require a generic (foo=*) ManagementResourceRegistration to execute wildcard ops #5572
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Build 1860 is now running using a merge of 481ba950b15e3c6253a050945809d9b9cb3b9ecf |
|
Build 1860 outcome was FAILURE using a merge of 481ba950b15e3c6253a050945809d9b9cb3b9ecf Build problems:Failed tests detected Failed tests |
|
retest this please |
|
Build 1874 is now running using a merge of 2885c7887f9107702512d86bdf6a545a8990255c |
|
Build 1874 outcome was FAILURE using a merge of 2885c7887f9107702512d86bdf6a545a8990255c Build problems:Failed tests detected Failed tests |
|
retest this please |
|
Build 1882 is now running using a merge of 2885c7887f9107702512d86bdf6a545a8990255c |
|
Build 1882 outcome was SUCCESS using a merge of 2885c7887f9107702512d86bdf6a545a8990255c |
…stration during wildcard queries
…ontext for each resolved address
…card registration
…lid wildcard addresses
|
Build 1890 is now running using a merge of af17a5d |
|
Build 1890 outcome was FAILURE using a merge of af17a5d Build problems:Failed tests detected Failed tests |
|
Build 1894 is now running using a merge of af17a5d |
|
Build 1894 outcome was SUCCESS using a merge of af17a5d |
n1hility
added a commit
that referenced
this pull request
Dec 17, 2013
WFLY-2527 Don't require a generic (foo=*) ManagementResourceRegistration to execute wildcard ops
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
The read-resource-description op supports wildcard results (with the result being a list with one element per match) but this fails if the target resource type does not have a generic MRR registered. This patch fixes that.
The last commit also makes read-operation-description more lenient when the user specifies a wildcard address but there is no wildcard registration. The OSH now checks if all registered non-wildcard MRRs have the op and with the same description and flags; if so that description/flags are used to prepare the response.
The more lenient read-operation-description behavior is necessary to make the read-resource-description fix useful in the CLI. Without it the CLI's background validation of the user's read-resource-description input will fail. The validation uses read-operation-description.