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

DNM: Run the e2e regression at mutiple nodes in parallel to check sta… #182

Closed
wants to merge 1 commit into from
Closed

DNM: Run the e2e regression at mutiple nodes in parallel to check sta… #182

wants to merge 1 commit into from

Conversation

qinqon
Copy link
Member

@qinqon qinqon commented Sep 26, 2019

…bility

@qinqon qinqon force-pushed the e2e_stability branch 2 times, most recently from 08776ba to 1534da3 Compare September 26, 2019 09:51
@qinqon qinqon changed the title WIP: Run the e2e regression at mutiple nodes in parallel to check sta… DNM: Run the e2e regression at mutiple nodes in parallel to check sta… Sep 26, 2019
@qinqon qinqon force-pushed the e2e_stability branch 3 times, most recently from c07446c to 7d56f18 Compare September 26, 2019 14:31
@qinqon
Copy link
Member Author

qinqon commented Sep 26, 2019 •
edited
Loading

About the vlan_filtering issue, looks like br1 is already there and nmstate fails, attached is the error with multiline fixed.
error.log

The timeout at "node" resource Get looks like we have to increase timeout at .Get or reduce it and retry at Eventually

@phoracek
Copy link
Member

@qinqon indeed looks like you were just lucky :)

@qinqon qinqon force-pushed the e2e_stability branch 4 times, most recently from eed60cd to 261b0f4 Compare September 27, 2019 09:42
@qinqon
Copy link
Member Author

qinqon commented Sep 27, 2019

ci test please

@qinqon qinqon force-pushed the e2e_stability branch 2 times, most recently from fc08e47 to f02313c Compare September 27, 2019 13:01
@qinqon
Copy link
Member Author

qinqon commented Sep 27, 2019

Ther were a bug at bridgeDescription function, Instead of ReadInteval it was using ReadTimeout.

@phoracek
Copy link
Member

@qinqon please cherry-pick #187 to this PR, I think it will solve the leftover bridge issue :)

@phoracek
Copy link
Member

@qinqon tested with my PR on top #188, all lanes have passed

@phoracek
Copy link
Member

@qinqon and I tested to enable the default bridged network test on top of my PR #189 and it is still raceful.

@qinqon qinqon force-pushed the e2e_stability branch 6 times, most recently from 3512741 to e15a6a3 Compare October 3, 2019 11:03
@qinqon qinqon force-pushed the e2e_stability branch 2 times, most recently from 46943ec to efed7d6 Compare October 14, 2019 08:41
@qinqon
Copy link
Member Author

qinqon commented Oct 14, 2019 •
edited
Loading

Executions summary:

First pass:

Second pass:

@qinqon
Copy link
Member Author

qinqon commented Oct 14, 2019

ci test please

@qinqon qinqon closed this Nov 28, 2019
@qinqon qinqon deleted the e2e_stability branch November 28, 2019 08:30
bcrochet pushed a commit to bcrochet/kubernetes-nmstate that referenced this pull request Feb 9, 2021
@openshift-merge-robot
Merge pull request nmstate#182 from openshift-bot/art-consistency-ope…
d92a1f5 
…nshift-4.7-openshift-kubernetes-nmstate-operator

Updating openshift-kubernetes-nmstate-operator builder & base images to be consistent with ART
@kubevirt-bot kubevirt-bot mentioned this pull request Apr 21, 2021
Closed
@kubevirt-bot kubevirt-bot mentioned this pull request Aug 27, 2021
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@qinqon @phoracek