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

Collect full Sonobuoy results for cloud CI jobs #1034

Merged
merged 1 commit into from
Aug 7, 2020
Merged

Collect full Sonobuoy results for cloud CI jobs #1034

merged 1 commit into from
Aug 7, 2020

Conversation

Dyanngg
Copy link
Contributor

@Dyanngg Dyanngg commented Aug 4, 2020 •
edited
Loading

This is the first step in attempt to solve #1032. Dump the sonobouy results for easier debugging of flaky failures.

@antrea-bot
Copy link
Collaborator

Thanks for your PR.
Unit tests and code linters are run automatically every time the PR is updated.
E2e, conformance and network policy tests can only be triggered by a member of the vmware-tanzu organization. Regular contributors to the project should join the org.

The following commands are available:

  • /test-e2e: to trigger e2e tests.
  • /skip-e2e: to skip e2e tests.
  • /test-conformance: to trigger conformance tests.
  • /skip-conformance: to skip conformance tests.
  • /test-whole-conformance: to trigger all conformance tests on linux.
  • /skip-whole-conformance: to skip all conformance tests on linux.
  • /test-networkpolicy: to trigger networkpolicy tests.
  • /skip-networkpolicy: to skip networkpolicy tests.
  • /test-windows-conformance: to trigger windows conformance tests.
  • /skip-windows-conformance: to skip windows conformance tests.
  • /test-windows-networkpolicy: to trigger windows networkpolicy tests.
  • /skip-windows-networkpolicy: to skip windows networkpolicy tests.
  • /test-hw-offload: to trigger ovs hardware offload test.
  • /skip-hw-offload: to skip ovs hardware offload test.
  • /test-all: to trigger all tests (except whole conformance).
  • /skip-all: to skip all tests (except whole conformance).

These commands can only be run by members of the vmware-tanzu organization.

@Dyanngg
Copy link
Contributor Author

Dyanngg commented Aug 4, 2020

/skip-all

@Dyanngg Dyanngg force-pushed the cloud-ci-full-log branch from 2b81d0e to 6d32cd0 Compare August 4, 2020 22:11
@Dyanngg
Copy link
Contributor Author

Dyanngg commented Aug 4, 2020

/skip-all

antoninbas
antoninbas reviewed Aug 4, 2020
View reviewed changes
Copy link
Contributor

@antoninbas antoninbas left a comment

Choose a reason for hiding this comment

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

with the current commit message / PR description, issue #1032 will be automatically closed when we merge this. I don't think that's what you want.

ci/run-k8s-e2e-tests.sh Outdated Show resolved Hide resolved
ci/run-k8s-e2e-tests.sh Outdated Show resolved Hide resolved
ci/test-conformance-gke.sh Show resolved Hide resolved
@Dyanngg Dyanngg force-pushed the cloud-ci-full-log branch from 6d32cd0 to a8d55f2 Compare August 4, 2020 23:05
@Dyanngg
Copy link
Contributor Author

Dyanngg commented Aug 4, 2020

/skip-all

@Dyanngg Dyanngg force-pushed the cloud-ci-full-log branch 2 times, most recently from 64dd4ef to b417d50 Compare August 4, 2020 23:11
@Dyanngg
Copy link
Contributor Author

Dyanngg commented Aug 4, 2020

/skip-all

@Dyanngg Dyanngg requested a review from antoninbas August 4, 2020 23:14
antoninbas
antoninbas previously approved these changes Aug 5, 2020
View reviewed changes
Copy link
Contributor

@antoninbas antoninbas left a comment

Choose a reason for hiding this comment

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

LGTM

@Dyanngg Dyanngg force-pushed the cloud-ci-full-log branch 2 times, most recently from 9a8a970 to 0eb7986 Compare August 5, 2020 07:25
@Dyanngg
Copy link
Contributor Author

Dyanngg commented Aug 5, 2020

/skip-all

antoninbas
antoninbas previously approved these changes Aug 5, 2020
View reviewed changes
Copy link
Contributor

@antoninbas antoninbas left a comment

Choose a reason for hiding this comment

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

LGTM

@Dyanngg Dyanngg changed the title Set "dump" level logging for cloud CI jobs Collect full Sonobuoy results for cloud CI jobs Aug 5, 2020
@Dyanngg
Copy link
Contributor Author

Dyanngg commented Aug 5, 2020

/skip-all

@Dyanngg Dyanngg force-pushed the cloud-ci-full-log branch from 32da026 to 2d5d99d Compare August 5, 2020 23:44
@Dyanngg
Copy link
Contributor Author

Dyanngg commented Aug 5, 2020

/skip-all

@Dyanngg Dyanngg requested a review from antoninbas August 6, 2020 20:26
@Dyanngg
Copy link
Contributor Author

Dyanngg commented Aug 6, 2020

Tested on Jenkins that full log bundle can be collected

antoninbas
antoninbas approved these changes Aug 6, 2020
View reviewed changes
ci/jenkins/jobs/projects.yaml
triggers:
- timed: H H */2 * *
publishers:
- archive:
allow-empty: true
artifacts: 'eks-test.log'
artifacts: 'eks-test.log, ci/*.tar.gz'
Copy link
Contributor

Choose a reason for hiding this comment

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

just want to make sure that this will be a single tarball per job run (i.e. each run is a new checkout and we don't accumulate artifacts indefinitely under this ci/ directory)?

Copy link
Contributor Author

@Dyanngg Dyanngg Aug 6, 2020 •
edited
Loading

Choose a reason for hiding this comment

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

Yes the work directory will be cleared and pulled for each job run

@antoninbas antoninbas merged commit bb99292 into antrea-io:master Aug 7, 2020
@Dyanngg Dyanngg deleted the cloud-ci-full-log branch September 30, 2020 23:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@antoninbas antoninbas antoninbas approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@Dyanngg @antrea-bot @antoninbas @vmwclabot