-
-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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
JHipster control center migration plan #12167
Comments
That's a good idea, however, I would like to still generate the JHCC docker compose file for all apps, in case people want to test the feature. |
ya that should be fine, since we do that already for a lot of stuff |
Just like @PierreBesson I think we should add JHCC Docker compose anyway. |
OK for adding JHCC by default for all generated app -> so no question about adding JHCC, but instead, what do you think about an option for removing all Admin screen. It would be a light version of application. OK for changing/removing the question about Registry: but if I remember well, the Registry is used to scale monolith app with Hazelcast -> we need to check this behaviour before changing the question, can you @avdev4j ? |
It could be a mixed question like "Do you want to remove admin features in favor of JHipster control center?
We discussed about that with @PierreBesson this afternoon. It seems that this feature is not really used and we though to remove it totally. (Only keep discovery choice for microservices). |
Regarding future migrations (after initial release), how do you plan to ensure compatibility between apps and JHCC? Wouldn't it be nice if the JHCC could request the managed apps to check whether their management API version is compatible with the JHCC version? |
There are few things to consider here
|
I've started something on a separated branch, this is what I suggest according to different elements we have here:
WDYT? |
Hey,
I’m worried this will break all applications using the registry to scale.
Then, the current system is easy but not perfect -> it’s nearly impossible
to do this well at our level, and it’s not good to provide a production
option which isn’t working 100% fine.
So this would remove a great option, and push the scalability burden to our
users, but in the end it might be better.
I just want to make sure we are all aware of this before removing this,
then I’m ok with it.
Julien
Le jeu. 30 juil. 2020 à 17:17, Avdev4J <[email protected]> a écrit :
I've started something on a separated branch, this is what I suggest
according to different elements we have here:
- Delete registry question when we choose monolith application
- add question to enable admin module (screens, yes by default)
- If no, remove admin screens and add default logging file config to
allow JHCC to display it
WDYT?
—
You are receiving this because you are on a team that was mentioned.
Reply to this email directly, view it on GitHub
<#12167 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AACNLI5KQ5TAG2I2CGZC4KLR6GFHVANCNFSM4PKNQABA>
.
--
Julien Dubois
Twitter: @julienDubois <http://twitter.com/#!/juliendubois>
|
After thinking about it, we can keep this question. We don't have any issue on this since months/years, so I suppose it works well or not used at all :) |
This issue is stale because it has been open 30 days with no activity. |
@jhipster/developers since JHCC and jhipster-registry are deprecated (nobody is maintaining those projects), I will close this issue. |
With JHipster 7 we need to provide a migration path for JHCC. I'm not in favor of removing all the admin screens as it will force people to use JHCC which is a separate app thus complicating situations for simple monoliths, which is still the majority of the apps created with JHipster. It will also make infra more complicated for them, not everyone needs microservices or complicated registries and stuff. People shouldn't be forced to use JHCC just to get some useful admin screens (Those screens are pretty popular)
So let's try the below approach
@jhipster/developers WDYT?
The text was updated successfully, but these errors were encountered: