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

Fix hostname env. #901

Merged
merged 1 commit into from
Sep 10, 2018
Merged

Conversation

Random-Liu
Copy link
Member

@Random-Liu Random-Liu commented Sep 10, 2018

Ref #898 (comment)

Add HOSTNAME back. /cc @estesp

Signed-off-by: Lantao Liu lantaol@google.com

@estesp
Copy link
Member

estesp commented Sep 10, 2018

Nice; thanks @Random-Liu

@Random-Liu Random-Liu added this to the v1.0 milestone Sep 10, 2018
@Random-Liu
Copy link
Member Author

We should cherrypick this into all supported branches.

Signed-off-by: Lantao Liu <lantaol@google.com>
@yujuhong
Copy link
Member

/lgtm

Copy link
Member

@mikebrow mikebrow left a comment

Choose a reason for hiding this comment

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

/LGTM

@Random-Liu Random-Liu merged commit ed68cfd into containerd:master Sep 10, 2018
@Random-Liu Random-Liu deleted the fix-hostname-env branch September 10, 2018 20:57
@Random-Liu
Copy link
Member Author

I'll cherrypick this one after it is proved that this PR doesn't break e2e test.

Random-Liu added a commit that referenced this pull request Sep 11, 2018
Random-Liu added a commit that referenced this pull request Sep 11, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants