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.

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

Add integration workflow for iosxr #526

Open
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

Ruchip16
Copy link
Contributor

@Ruchip16 Ruchip16 commented Jan 12, 2025

Summary: "Setup Cisco.IOSXR upstream integration test to run on CML"

details: |

  • Add upstream integration workflow for CML.
  • Enables test job for Github repo cisco.iosxr
  • Review the YAML files, node configuration file & workflow file
  • Run the job and verify whether the CI runs or not.
  • Update in comment about the status of the test failure if any.

ISSUE TYPE

COMPONENT NAME

integration-test.yml

Copy link

codecov bot commented Jan 12, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.69%. Comparing base (8f9341d) to head (87dd858).
Report is 40 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #526      +/-   ##
==========================================
+ Coverage   76.91%   78.69%   +1.78%     
==========================================
  Files         166      189      +23     
  Lines       11298    12116     +818     
==========================================
+ Hits         8690     9535     +845     
+ Misses       2608     2581      -27     

see 127 files with indirect coverage changes

@roverflow roverflow marked this pull request as draft January 15, 2025 09:47
Copy link
Member

@roverflow roverflow left a comment

Choose a reason for hiding this comment

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

LGTM

@Ruchip16 Ruchip16 removed the request for review from NilashishC January 21, 2025 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants