-
Notifications
You must be signed in to change notification settings - Fork 414
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
feat(api,sdk,ui): deprecated worker models #2458
Conversation
Signed-off-by: François Samin <francois.samin@corp.ovh.com>
ui/src/assets/i18n/fr.json
Outdated
"worker_model_is_deprecated": "Modèle de worker déprécié", | ||
"worker_model_deprecated_help": "Indique que ce modèle de worker est déprécié", | ||
"worker_model_deprecated_tooltip": "Ce modèle de worker est déprécié", | ||
"worker_model_official_tooltip": "Modèle de worker official maintenu par les administrateurs de CDS", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
:%s/official/officiel
<span *ngIf="v.disabled==='true'"> {{ 'worker_model_disabled' | translate }}</span> | ||
<div class="ui"> | ||
{{v.name}} | ||
<span *ngIf="v.disabled"> {{ 'worker_model_disabled' | translate }}</span> | ||
<span *ngIf="v.nb_spawn_err > 0"> <i class="warning sign icon orange" title="{{ 'worker_model_warning' | translate }}"></i> </span> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can you use smDirTooltip on v.nb_spawn_err > 0 too? This icon (warning sign) does not have a great icon (not same size) nor a great tooltip.
</div> | ||
<div class="ui two wide column valign"> | ||
<span *ngIf="getWorkerModel(r.value)?.is_official" [smDirTooltip]="'worker_model_official_tooltip' | translate" smDirPosition="top center"><i class="check circle outline icon large green"></i></span> | ||
<span *ngIf="getWorkerModel(r.value)?.is_deprecated" [smDirTooltip]="'worker_model_deprecated_tooltip' | translate" smDirPosition="top center"><i class="exclamation circle icon large orange"></i></span> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if too many error with a worker model (5) -> status is disabled. It would be useful to user to see that the selected worker model is disabled in job requirements. So, it should be great to have an icon for the status disabled or not, as deprecated and official.
@@ -18,37 +18,30 @@ <h2>{{ 'worker_model_list_title' | translate }}</h2> | |||
<table class="ui fixed celled table" *ngIf="getDataForCurrentPage().length > 0"> | |||
<thead> | |||
<tr> | |||
<th class="three wide">{{ 'worker_model_name' | translate }}</th> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not your fault but sum of column wide must be equal to 16
ui/src/assets/i18n/fr.json
Outdated
@@ -641,7 +641,8 @@ | |||
"worker_model_is_deprecated": "Modèle de worker déprécié", | |||
"worker_model_deprecated_help": "Indique que ce modèle de worker est déprécié", | |||
"worker_model_deprecated_tooltip": "Ce modèle de worker est déprécié", | |||
"worker_model_official_tooltip": "Modèle de worker official maintenu par les administrateurs de CDS", | |||
"worker_model_official_tooltip": "Modèle de worker officiel maintenu par les administrateurs de CDS", | |||
"worker_model_spawn_error_tooltip": "Modèle est erreur", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
"Modèle en erreur"
Signed-off-by: François Samin francois.samin@corp.ovh.com
Description
Add ability to set worker model as deprecated ans display this information while selecting model"
Related issues
n/a
About tests
done manually
Mentions
@ovh/cds