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

Fix duplicate headless services - #473

Open
qu1queee wants to merge 1 commit into
cloudfoundry-incubator:developfrom
qu1queee:remove_extra_headless_svc
Open

Fix duplicate headless services #473
qu1queee wants to merge 1 commit into
cloudfoundry-incubator:developfrom
qu1queee:remove_extra_headless_svc

Conversation

@qu1queee

@qu1queee qu1queee commented Feb 19, 2019

Copy link
Copy Markdown
Contributor

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
Copy Markdown

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

@jandubois

Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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