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

Get rid of env var SERVICE_NAME #149

Merged
merged 2 commits into from
Nov 29, 2019

Conversation

cscetbon
Copy link
Contributor

No description provided.

Copy link

@allamand allamand left a comment

Choose a reason for hiding this comment

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

You can't remove this as it is used by the run.sh script from the bootstrap container:

 cat docker/bootstrap/files/run.sh | grep SERVICE_NAME                                                                    
    HOSTNAME=$POD_NAME.$SERVICE_NAME.$POD_NAMESPACE

@cscetbon
Copy link
Contributor Author

@allamand I see. I even wonder why we still remove the node from its seeds list. I need to check if it's still needed

@cscetbon cscetbon force-pushed the clean-code branch 2 times, most recently from f9b3ef0 to aff54a3 Compare November 26, 2019 23:39
@cscetbon cscetbon changed the title Remove env var SERVICE_NAME Get rid of env var SERVICE_NAME Nov 26, 2019
allamand
allamand previously approved these changes Nov 28, 2019
Copy link

@allamand allamand left a comment

Choose a reason for hiding this comment

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

LGTM , please trigger tests

@allamand allamand merged commit 37cee56 into Orange-OpenSource:master Nov 29, 2019
@cscetbon cscetbon deleted the clean-code branch November 29, 2019 17:22
cscetbon added a commit that referenced this pull request Aug 13, 2020
* clean-code - Remove env var SERVICE_NAME

* clean-code - Remove Service from bootstrap image
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.

2 participants