-
Notifications
You must be signed in to change notification settings - Fork 1
testing unverified commit #1
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
base: main
Are you sure you want to change the base?
Conversation
* Updated testbed * Update cleanConfig.yml Added new line * Update cleanConfig_sit.yml
* QUAGGA_CONFIG update QUAGGA_CONFIG there is no policy (route-map) defined for peer-group IXIA. Due to that it is not able to receive routes In/Out. If we add “no bgp ebgp-requires-policy”, it advertises and receives the routes from other peers and traffic flows fine. * print ixia app error log added code to print ixia error log when start_protocols stop_protocols --------- Co-authored-by: vkuma82 <kumarvinod82@yahoo.com>
Signed-off-by: hubogee <hubogee@gmail.com>
Signed-off-by: hubogee <hubogee@gmail.com>
Fixed stage result settings
| self.log.info('Verify if cloned repo test branch has files in it') | ||
| output = Utilities.runLinuxCmd('ls', cwd=self.testIdTestingBranch, logObj=self.log) | ||
| if len(output) > 0: | ||
| self.log.info('Verified cloned files!') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In my opinion that could make it harder for PR owner to go over the code comments when addressing them as they'd have to open them one by one.
On the other hand, as a reviewer I might want to collapse all comments to focus on reading the code and only open comments to see if someone already commented the same thing at the same code location before adding a comment.
No description provided.