Skip to content

Remove unnecessary GEM_HOME/GEM_PATH settings#752

Merged
kenhys merged 1 commit intomasterfrom
service-file-remove-unnecessary-env-about-gem
Dec 5, 2024
Merged

Remove unnecessary GEM_HOME/GEM_PATH settings#752
kenhys merged 1 commit intomasterfrom
service-file-remove-unnecessary-env-about-gem

Conversation

@daipom
Copy link
Contributor

@daipom daipom commented Dec 4, 2024

Remove environment variables that prevent zero-downtime-restart feature of Fluentd v1.18.0. This is because an upgrade with a Ruby version update will not work correctly due to environment variables for the old Ruby.

Note: Cherry-picked 9762e1f on feature-nodowntime branch.

Remove environment variables that prevent zero-downtime-restart
feature of Fluentd v1.18.0. This is because an upgrade with a Ruby
version update will not work correctly due to environment variables
for the old Ruby.

Signed-off-by: Shizuo Fujita <[email protected]>
Signed-off-by: Daijiro Fukuda <[email protected]>
@daipom daipom force-pushed the service-file-remove-unnecessary-env-about-gem branch from 22b20d8 to 251bd3e Compare December 4, 2024 09:50
Copy link
Contributor

@Watson1978 Watson1978 left a comment

Choose a reason for hiding this comment

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

👍🏻

Copy link
Contributor

@kenhys kenhys left a comment

Choose a reason for hiding this comment

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

LGTM.

@kenhys kenhys merged commit c8419c4 into master Dec 5, 2024
@kenhys kenhys deleted the service-file-remove-unnecessary-env-about-gem branch December 5, 2024 02:22
@kenhys kenhys added this to the 5.2.0 milestone Dec 12, 2024
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