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 #12096: Fix service registration when colocalisation with consul server. #1582

Merged
merged 1 commit into from
Dec 13, 2023

Conversation

GiooDev
Copy link
Contributor

@GiooDev GiooDev commented Dec 12, 2023

Description

Consul server is not bind to localhost.

Type de changement:

  • Ansiblerie

Contributeur

  • VAS (Vitam Accessible en Service)

@GiooDev GiooDev added bug Something isn't working small pr embarquant peu de changements et à review rapide, ne nécessitant qu'un reviewer VAS VAS contribution OPS REVIEW Mandatory if deployment/ directory is modified. labels Dec 12, 2023
@GiooDev GiooDev added this to the IT 129 milestone Dec 12, 2023
@GiooDev GiooDev self-assigned this Dec 12, 2023
@vitam-devops
Copy link
Collaborator

vitam-devops commented Dec 12, 2023

Logo
Checkmarx One – Scan Summary & Details41496056-23d0-4315-bcc0-5dd83aae0ee8

No New Or Fixed Issues Found

@@ -3,6 +3,7 @@ spring:
cloud:
consul:
enabled: true
host: {{ hostvars[inventory_hostname][consul.network] if inventory_hostname in groups['hosts_vitamui_consul_server'] else 'localhost' }}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Je ne suis pas sûr de comprendre ni le soucis ni le fix 😄

La conf consul ne semble pas écouter sur localhost
https://github.com/ProgrammeVitam/vitam-ui/blob/story_12096_fix_consul_host_register/deployment/roles/consul/templates/consul.json.j2

Aussi, je ne sais pas si on est censé supporté le mode où vitamui est déployé sur une instance hébergeant également vitam : quel consul est utilisé / déployé dans ce cas?

@GiooDev GiooDev force-pushed the story_12096_fix_consul_host_register branch from 9753e18 to ad7d9a7 Compare December 13, 2023 10:02
…l server.

Consul server is not bind to localhost.
@GiooDev GiooDev force-pushed the story_12096_fix_consul_host_register branch from ad7d9a7 to e454008 Compare December 13, 2023 13:29
@GiooDev GiooDev merged commit adcb149 into develop Dec 13, 2023
1 check was pending
@GiooDev GiooDev deleted the story_12096_fix_consul_host_register branch December 13, 2023 13:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working OPS REVIEW Mandatory if deployment/ directory is modified. small pr embarquant peu de changements et à review rapide, ne nécessitant qu'un reviewer VAS VAS contribution
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants