Skip to content
This repository has been archived by the owner on Jul 11, 2023. It is now read-only.

Collect bug report for e2e failures #4504

Conversation

nshankar13
Copy link
Contributor

@nshankar13 nshankar13 commented Feb 3, 2022

Signed-off-by: nshankar13 [email protected]

Description: Use the osm support bug-report command to collect mesh related logs for e2e CI Failures.

Testing done: Ensured that the osm support bug-report command was not run for successful tests. Then, I deliberately changed the test condition in one of the tests to fail, and saw that the command was run:

Collecting logs, using "/home/nshankar13/documents/new_osm_fork/osm/scripts/get-osm-namespace-logs.sh /tmp/test-1376025377" Running locally '/home/nshankar13/documents/new_osm_fork/osm/scripts/get-osm-namespace-logs.sh /tmp/test-1376025377' Running locally 'kubectl get events -A' Running locally '../../bin/osm proxy get config_dump client --namespace client -f /tmp/test-1376025377/envoy_configs/client_client/config_dump.txt' Running locally '../../bin/osm proxy get clusters client --namespace client -f /tmp/test-1376025377/envoy_configs/client_client/clusters.txt' Running locally '../../bin/osm proxy get certs client --namespace client -f /tmp/test-1376025377/envoy_configs/client_client/certs.txt' Running locally '../../bin/osm proxy get listeners client --namespace client -f /tmp/test-1376025377/envoy_configs/client_client/listeners.txt' Running locally '../../bin/osm proxy get ready client --namespace client -f /tmp/test-1376025377/envoy_configs/client_client/ready.txt' Running locally '../../bin/osm proxy get stats client --namespace client -f /tmp/test-1376025377/envoy_configs/client_client/stats.txt'

Running locally '../../bin/osm support bug-report --all -o=/tmp/test-1376025377/osm_bug_report.tar.gz'
stdout:
[+] Created staging dir /tmp/osm_bug_report_805321390 to generate bug report

[+] Collecting information about meshed namespaces
✓ Finished generating report for meshed namespaces

[+] Collecting information from individual app namespaces
✓ Collected report from Namespace "client"
✓ Finished generating report for individual app namespaces

[+] Collecting information from individual app pods
✓ Collected report for Pod "client/client"
✓ Finished generating report for individual app pods

[+] Collecting information from control plane pods
✓ Collected report for Pod osm-system/osm-bootstrap-bd8d4d478-xtbng
✓ Collected report for Pod osm-system/osm-controller-844549fd77-gmzkq
✓ Collected report for Pod osm-system/osm-injector-675bd48898-9tmb2
✓ Finished generating report for control plane pods

[+] Collecting ingress information
✓ Finished generating report for ingress

[+] Collecting information from individual app namespaces
✓ Bug report successfully archived to /tmp/test-1376025377/osm_bug_report.tar.gz

Output file was successfully generated in /tmp/test-1376025377 folder:

nshankar13@DESKTOP-U63CGKH:/tmp/test-1376025377$ ls
envoy_configs events osm_bug_report.tar.gz

Affected area:

Functional Area
New Functionality [ ]
CI System [X]
CLI Tool [X]
Certificate Management [ ]
Control Plane [ ]
Demo [ ]
Documentation [ ]
Egress [ ]
Ingress [ ]
Install [ ]
Networking [ ]
Observability [ ]
Performance [ ]
SMI Policy [ ]
Security [ ]
Sidecar Injection [ ]
Tests [X]
Upgrade [ ]
Other [ ]

Please answer the following questions with yes/no.

  1. Does this change contain code from or inspired by another project?

    • Did you notify the maintainers and provide attribution?
  2. Is this a breaking change?

  3. Has documentation corresponding to this change been updated in the osm-docs repo (if applicable)?

@codecov-commenter
Copy link

Codecov Report

Merging #4504 (2fee28e) into main (3367f49) will decrease coverage by 0.02%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #4504      +/-   ##
==========================================
- Coverage   69.25%   69.22%   -0.03%     
==========================================
  Files         213      213              
  Lines       14625    14625              
==========================================
- Hits        10128    10124       -4     
- Misses       4446     4450       +4     
  Partials       51       51              
Flag Coverage Δ
unittests 69.22% <ø> (-0.03%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
pkg/messaging/workqueue.go 89.28% <0.00%> (-10.72%) ⬇️
pkg/certificate/rotor/rotor.go 84.78% <0.00%> (-2.18%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 3367f49...2fee28e. Read the comment docs.

@nshankar13 nshankar13 force-pushed the collect_bug_report_e2e_failures branch from 2fee28e to 4938246 Compare February 3, 2022 03:49
@nshankar13 nshankar13 marked this pull request as ready for review February 3, 2022 04:04
tests/framework/common.go Outdated Show resolved Hide resolved
@nshankar13 nshankar13 force-pushed the collect_bug_report_e2e_failures branch 2 times, most recently from 9458b79 to a766b1f Compare February 3, 2022 17:26
tests/framework/common.go Outdated Show resolved Hide resolved
tests/framework/common.go Outdated Show resolved Hide resolved
@nshankar13 nshankar13 force-pushed the collect_bug_report_e2e_failures branch 2 times, most recently from 913cd0b to 91cfcad Compare February 3, 2022 18:48
@snehachhabria snehachhabria merged commit 47db020 into openservicemesh:main Feb 3, 2022
schristoff pushed a commit to schristoff/osm that referenced this pull request Feb 22, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants