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

ci: move hubble connectivity tests to nightly pipeline #2310

Merged
merged 31 commits into from
Oct 20, 2023
Merged

ci: move hubble connectivity tests to nightly pipeline #2310

merged 31 commits into from
Oct 20, 2023

Conversation

jshr-w
Copy link
Contributor

@jshr-w jshr-w commented Oct 19, 2023

Reason for Change:

PR moves Hubble Connectivity tests to Cilium Nightly Pipeline by enabling Hubble Metrics.
Same as #2286, because of inability to merge from fork currently.

Issue Fixed:

Requirements:

Notes:

matmerr and others added 26 commits October 11, 2023 09:51
@matmerr
[test] add hubble system test to CI (#2265)
3c43d7e 
* cilium configmap

* update hubble configs and add metrics test

* update pipeline yaml

* separate cilium+hubble config
@jshr-w
ci: move hubble connectivity test to nightly pipeline
8d1816b
@jshr-w
fix: move to correct file
2b11ae9
@jshr-w
style: indentation change
07073fc
@jshr-w
ci: update configmap to enable Hubble
cc5cffa
@jshr-w
fix: move hubble test
e684c32
@jshr-w
fix: move connectivity test before delete
f0a6bd7
@jshr-w
fix: add daemonset namespace
a7d4b1b
@jshr-w
fix: update command for configmap replace
b7766e5
@jshr-w
test no restart after replace
838340f
@jshr-w
fix: try apply instead of replace
0b5bb19
@jshr-w
fix: add back restart ds
050d422
@jshr-w
add longer timeout after ds restart
481af23
@jshr-w
adjust timeout setup
8ebb5ad
@jshr-w
extend timeout, add logging
daa576d
@jshr-w
add logging, change cm command
ac77e11
@jshr-w
update hubble configmap
0c79537
@jshr-w
clean up sleep statements
281de47
@jshr-w
Merge branch 'master' into shjayaraman/hubble-ci
c9afdd5
@jshr-w
remove hubble connectivity test from PR pipeline
4350e5d
@jshr-w
use kubernetes utils
9770a9f
@jshr-w
fix style
2850e5f 
Signed-off-by: jshr-w <[email protected]>
@jshr-w
update ds restart wait
d3bca63
@jshr-w
enable Hubble on nightly, disable on PR
5397746
@jshr-w
tag networkobservability test
3b570cb
@jshr-w
fix test call
5c44984
@jshr-w jshr-w added cni Related to CNI. ci Infra or tooling. labels Oct 19, 2023
@jshr-w jshr-w self-assigned this Oct 19, 2023
@jshr-w jshr-w requested a review from a team as a code owner October 19, 2023 16:46
jpayne3506
jpayne3506 approved these changes Oct 19, 2023
View reviewed changes
Copy link
Contributor

@jpayne3506 jpayne3506 left a comment

Choose a reason for hiding this comment

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

lgtm

matmerr
matmerr approved these changes Oct 19, 2023
View reviewed changes
Copy link
Member

@matmerr matmerr left a comment

Choose a reason for hiding this comment

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

🚀

@jshr-w jshr-w enabled auto-merge (squash) October 19, 2023 22:55
@jshr-w jshr-w merged commit 996c3fe into master Oct 20, 2023
78 checks passed
@jshr-w jshr-w deleted the shjayaraman/hubble-ci branch October 20, 2023 05:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jpayne3506 jpayne3506 jpayne3506 approved these changes

@matmerr matmerr matmerr approved these changes

@tamilmani1989 tamilmani1989 Awaiting requested review from tamilmani1989

@vakalapa vakalapa Awaiting requested review from vakalapa

Assignees

@jshr-w jshr-w

Labels
ci Infra or tooling. cni Related to CNI.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jshr-w @matmerr @jpayne3506