Skip to content
This repository has been archived by the owner on Oct 22, 2021. It is now read-only.

Fix duplicate headless services #473

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Fix duplicate headless services #473

wants to merge 1 commit into from

Conversation

qu1queee
Copy link
Contributor

@qu1queee qu1queee commented Feb 19, 2019

Current setup will generate two headless svc´s, these are <role>-<job>-set and <role>-set.
When using the service_name key in an specific job, in order to rename the svc´s, having both headless svc´s will lead to duplicate svc´s .

A more precise example is the nats role. Without using the service_name key in an instance_group job, the headless svcs will be:

nats-nats-set
nats-set

When using the service_name key, to enforce the name of uaa. the headless svcs will be:

nats-set
nats-set

which will lead to the svc´s duplication.

After speaking with @viovanov about this, he mentioned that the the <role>-<job>-set svc, is not widely used, and only a reference to it is found in:

Therefore this PR to remove the <role>-<job>-set svc .

These are <role>-<job>-set and <role>-set.
When using the `service_name` key in an specific job, in order to
rename the svc´s, having both headless svc´s will lead to duplicate
svc´s .
Also, the <role>-<job>-set is not widely used, and only a reference
to it is found in:
 - https://github.com/SUSE/scf/blob/develop/container-host-files/etc/scf/config/role-manifest.yml#L2241
@cfdreddbot
Copy link

✅ Hey qu1queee! The commit authors and yourself have already signed the CLA.

@jandubois
Copy link
Member

I would like to see a CI build of SCF that passes tests with this change to fissile before merging it into develop.

@qu1queee
Copy link
Contributor Author

This will not work when renaming the services for nats. Reason is the following line: https://github.com/SUSE/scf/blob/2.14.5/container-host-files/etc/scf/config/scripts/configure-HA-hosts.sh#L41

@jandubois
Copy link
Member

Yes, this will require additional work.

This name conflict is the reason we renamed some roles, e.g. api to api-group, to make the original service name api available again as an alternate service name for backwards compatibility. Same reason for renaming diego-access to diego-ssh.

@viovanov
Copy link
Member

The -set is still not required.
Renaming for nats may not work but it's not because of this.
This should be merged afaict.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants