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

Story #12067: Support Debian 12 #1650

Merged
merged 9 commits into from
Feb 20, 2024
Merged

Conversation

Ro3034
Copy link
Contributor

@Ro3034 Ro3034 commented Feb 14, 2024

Description

Description des modifications

Type de changement:

Indiquer le ou les types de changements

  • Build

  • PKI

  • Ansiblerie

  • Nouveau Code

  • Correction

  • Refactorisation de code

  • Autre

Documentation:

Indiquer la documentation mise à jour

[ ] Quels sont les nouvelles documentations ?

[ ] Quels sont les modifications existantes ?

[ ] Quels sont les documentations ou sections de documentations supprimés ?

Tests:

Indiquer comment le code à été testé (manuel, environnement, TU, etc)

manuel

environnement

TU

Migration:

Indiquer si les modifications apportées impliquent une migration sur l'existant et comment la faire

Checklist:

Sélectionner les éléments de la checklist

[ ] Mon code suit le style de code de ce projet.

[ ] J'ai commenté mon code, en particulier dans les classes et les méthodes difficile à comprendre.

[ ] J'ai fait les changements correspondant dans la documentation RAML.

[ ] J'ai fait les changements correspondant dans la documentation Métier.

[ ] J'ai fait les changements correspondant dans la documentation Technique.

[ ] J'ai rajouté les tests unitaires vérifiant mes fonctionnalités.

[ ] J'ai rajouté les tests de non régression vérifiant mes fonctionnalités.

[ ] Les tests unitaires nouveaux et existants passent avec succès localement.

[ ] Toutes les dépendances ont été mergées en priorité

Contributeur

Indiquer qui a développé cette fonctionnalité

VAS (Vitam Accessible en Service)

CEA (Commissariat à l'énergie atomique et aux énergies alternatives)

@Ro3034 Ro3034 added the OPS REVIEW Mandatory if deployment/ directory is modified. label Feb 14, 2024
@Ro3034 Ro3034 added this to the IT 131 milestone Feb 14, 2024
@Ro3034 Ro3034 self-assigned this Feb 14, 2024
@vitam-devops
Copy link
Collaborator

vitam-devops commented Feb 14, 2024

Logo
Checkmarx One – Scan Summary & Detailsf980dab6-a7ee-4154-bbcf-72d87798cfd8

No New Or Fixed Issues Found

@GiooDev GiooDev changed the title Story#12067 support debian12 Story #12067: Support Debian 12 Feb 14, 2024
@Ro3034 Ro3034 requested review from GiooDev and bbenaissa February 19, 2024 15:42
@@ -43,6 +43,14 @@

when: force_vitamui_version is defined

- name: Ensure mongo-express conf subdir exists
Copy link
Contributor

Choose a reason for hiding this comment

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

Ça fonctionnait pas sans ce sous répertoire ?

Copy link
Contributor

Choose a reason for hiding this comment

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

À corriger sur les branches de maintenance si nécessaire.

- ansible_distribution_version | int < 12


- name: "install openjdk 17 only when Debian 12"
Copy link
Contributor

Choose a reason for hiding this comment

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

Pas besoin d'installer la jdk sur toutes les VMs de vitamui. À déplacer après avoir intégrer l'API GW et les nouvelles UI en angular (pas besoin sur les UI de la JDK).

@Ro3034 Ro3034 force-pushed the story_12067_support_debian12 branch from bc5fbb0 to 424abe2 Compare February 20, 2024 09:43
@GiooDev GiooDev merged commit 12978c1 into develop Feb 20, 2024
1 check was pending
@GiooDev GiooDev deleted the story_12067_support_debian12 branch February 20, 2024 09:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OPS REVIEW Mandatory if deployment/ directory is modified.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants