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

add k8s log collector to CI #553

Merged
merged 1 commit into from
Sep 12, 2024

Conversation

willcl-ark
Copy link
Contributor

Currently, it's very painful to debug inconsistencies in the CI vs a local environment. Additionally, when things go wrong, we don't get any kubernetes output at all.

Fix this by adding a job which uploads artefacts to GH for each job.

stern allows tailing of all pods (in all namespaces), and can dump to json (I chose default mode here).

See an example of the artefacts here: https://github.com/willcl-ark/warnet/actions/runs/10763746456

@josibake
Copy link
Collaborator

josibake commented Sep 9, 2024

ACK, +1 for making CI failures easier to debug. I don't have any expertise in the GA area, but this seems reasonable to me.

(I chose default mode here

why not founder mode?

@willcl-ark
Copy link
Contributor Author

ACK, +1 for making CI failures easier to debug. I don't have any expertise in the GA area, but this seems reasonable to me.

(I chose default mode here

why not founder mode?

Ok, I'm going full founder mode. Just ordered one of these in each colour for each of us: https://levelsio.com/products/founder-mode-hat-100-cotton

@bdp-DrahtBot
Copy link
Collaborator

bdp-DrahtBot commented Sep 9, 2024

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Conflicts

No conflicts as of last run.

@josibake
Copy link
Collaborator

needs a rebase and would be great to get someone else's opinion who is more familiar with GA / our CI, cc @pinheadmz

@josibake
Copy link
Collaborator

LGTM and seems useful, if this ends up being problematic for GA we can always revert.

@josibake josibake merged commit 6afb540 into bitcoin-dev-project:main Sep 12, 2024
11 checks passed
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