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.

Sign up for GitHub

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

Jump to bottom

[8.13](backport #39534) Remove unused env vars from pipelines #39557

Closed
wants to merge 1 commit into from
Closed

[8.13](backport #39534) Remove unused env vars from pipelines #39557

wants to merge 1 commit into from

Conversation

mergify[bot]
Copy link
Contributor

@mergify mergify bot commented May 14, 2024 •
edited by zube bot
Loading

Proposed commit message

This PR performs some cleanup on some environment variables used in the dockerlogbeat and osquerybeat pipelines.

This is an automatic backport of pull request #39534 done by Mergify.

@mergify mergify bot requested a review from a team as a code owner May 14, 2024 13:03
@mergify mergify bot added the backport label May 14, 2024
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label May 14, 2024
@botelastic
Copy link

botelastic bot commented May 14, 2024

This pull request doesn't have a Team:<team> label.

@mergify Mergify
Copy link
Contributor Author

mergify bot commented May 15, 2024

This pull request is now in conflicts. Could you fix it? 🙏
To fixup this pull request, you can check out it locally. See documentation: https://help.github.com/articles/checking-out-pull-requests-locally/

git fetch upstream
git checkout -b mergify/bp/8.13/pr-39534 upstream/mergify/bp/8.13/pr-39534
git merge upstream/8.13
git push upstream mergify/bp/8.13/pr-39534

@alexsapran
Remove unused env vars from pipelines (#39534)
e8f5ceb 
Signed-off-by: Alexandros Sapranidis <[email protected]>
(cherry picked from commit 30e7239)
@alexsapran
Copy link
Contributor

closing, since the GH PR view is saying that there are conflicts, which I cannot see when checking out locally.

dliappis
dliappis approved these changes May 15, 2024
View reviewed changes
Copy link
Contributor

@dliappis dliappis left a comment

Choose a reason for hiding this comment

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

LGTM

@alexsapran alexsapran closed this May 15, 2024
@mergify mergify bot deleted the mergify/bp/8.13/pr-39534 branch May 15, 2024 11:30
@alexsapran alexsapran mentioned this pull request May 15, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dliappis dliappis dliappis approved these changes

Assignees

@alexsapran alexsapran

Labels
backport needs_team Indicates that the issue/PR needs a Team:* label
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@alexsapran @dliappis