Add admin client requirements to the process #600
Merged
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
bstansberry
requested changes
Sep 16, 2024
darranl
force-pushed
the
admin_clients
branch
from
9d8f4e5 to
3728a73
Compare
bstansberry
approved these changes
Sep 16, 2024
There was a problem hiding this comment.
Thanks, @darranl. I'll leave merging this to you as you know what other feedback you want.
jfdenise
reviewed
Sep 17, 2024
FEATURE_PROCESS.adoc
Outdated
| | *Admin Clients* | ||
| (HAL / JBoss CLI) | ||
|
|
||
| | Management model changes must be compatible with the lowe level model |
There was a problem hiding this comment.
Thank you, I have fixed this.
jfdenise
approved these changes
Sep 17, 2024
hpehl
approved these changes
Sep 18, 2024
… moving away from embedded comments.
darranl
force-pushed
the
admin_clients
branch
from
3728a73 to
c5a443f
Compare
|
I just sent a message to the dev list, I think will give it one more day then merge. Being merged does not set it in stone so we can still evolve as feedback comes in / we try and work with these contraints. |
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 updates I am proposing here relate to how in general the admin clients should be considered when new features are added to the application server.
The admin clients are maintained by smaller teams within the overall WildFly team, for this reason it is not going to scale if every feature requires complete admin client integration at every stability level so the general premise behind this proposal is that features can be added at Experimental or Preview level with a reduced set of admin client requirements, the admin clients can then continue to evolve to add these features and then the underlying feature can be promoted to Community / Default.
Note: This is not so much about features being added directly to the admin clients at the stability levels as in that case the admin client developer is already in control of compatibility etc...