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

Automated backport of #1268: Output init containers in subctl gather #1269

Open
wants to merge 1 commit into
base: release-0.18
Choose a base branch
from
Open

Automated backport of #1268: Output init containers in subctl gather #1269

wants to merge 1 commit into from

Conversation

tpantelis
Copy link
Contributor

@tpantelis tpantelis commented Nov 26, 2024 •
edited
Loading

Backport of #1268 on release-0.18.

#1268: Output init containers in subctl gather

For details on the backport process, see the backport requests page.

Depends on submariner-io/submariner-operator#3276

@tpantelis
Output init containers in subctl gather
e624cb4 
Refactored "gatherPodLogs" to take the desired container names. If none
specified, gather logs for all init and normal containers confgiured in
the pod.

Also, if a pod fails, it's useful to see the pod details so gather the
YAML for pod resources for daemonsets and deployments.

Signed-off-by: Tom Pantelis <tompantelis@gmail.com>
@submariner-bot
Copy link
Contributor

🤖 Created branch: z_pr1269/tpantelis/automated-backport-of-#1268-upstream-release-0.18
🚀 Full E2E won't run until the "ready-to-test" label is applied. I will add it automatically once the PR has 2 approvals, or you can add it manually.

@github-actions GitHub Actions
Copy link

This PR/issue depends on:

@tpantelis tpantelis mentioned this pull request Nov 26, 2024
Open
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Jaanki Jaanki Awaiting requested review from Jaanki Jaanki is a code owner

@mkolesnik mkolesnik Awaiting requested review from mkolesnik mkolesnik is a code owner

@Oats87 Oats87 Awaiting requested review from Oats87 Oats87 is a code owner

@skitt skitt Awaiting requested review from skitt skitt is a code owner

@sridhargaddam sridhargaddam Awaiting requested review from sridhargaddam sridhargaddam is a code owner

@vthapar vthapar Awaiting requested review from vthapar vthapar is a code owner

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
automated-backport dependent
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tpantelis @submariner-bot