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

[promtail] Add enable service links for Promtail #1863

Merged
merged 8 commits into from
Nov 12, 2022

Conversation

TAYTS
Copy link
Contributor

@TAYTS TAYTS commented Oct 1, 2022

This is to allow disable enableServiceLinks so that the service environments are not injected into the pods where in my scenario Promtail is deployed in the same namespace as the Knative services resulted the Promtail pod unable to start up due to too many service environment being injected.

Reference issue for pod unable to startup due to too many services:

@CLAassistant
Copy link

CLAassistant commented Oct 1, 2022

CLA assistant check
All committers have signed the CLA.

@TAYTS TAYTS force-pushed the feat/add-enable-service-links branch 2 times, most recently from 7cc51d7 to cf94022 Compare October 1, 2022 13:14
@zanhsieh zanhsieh changed the title Add enable service links for Promtail [promtail] Add enable service links for Promtail Oct 2, 2022
@TAYTS TAYTS force-pushed the feat/add-enable-service-links branch 3 times, most recently from 881ba90 to 8276d8b Compare October 22, 2022 22:41
Signed-off-by: TAYTS <taytzushieh@gmail.com>
Signed-off-by: TAYTS <taytzushieh@gmail.com>
Signed-off-by: TAYTS <taytzushieh@gmail.com>
Signed-off-by: TAYTS <taytzushieh@gmail.com>
@TAYTS TAYTS force-pushed the feat/add-enable-service-links branch from 8276d8b to 682c522 Compare October 22, 2022 22:42
@zanhsieh zanhsieh merged commit 1c63a49 into grafana:main Nov 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants