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

Bump ocp version for CI lane #713

Merged
merged 1 commit into from
Jun 17, 2024
Merged

Bump ocp version for CI lane #713

merged 1 commit into from
Jun 17, 2024

Conversation

SchSeba
Copy link
Collaborator

@SchSeba SchSeba commented Jun 13, 2024

this should make the ocp lane more stable

@github-actions GitHub Actions
Copy link

Thanks for your PR,
To run vendors CIs use one of:

  • /test-all: To run all tests for all vendors.
  • /test-e2e-all: To run all E2E tests for all vendors.
  • /test-e2e-nvidia-all: To run all E2E tests for NVIDIA vendor.

To skip the vendors CIs use one of:

  • /skip-all: To skip all tests for all vendors.
  • /skip-e2e-all: To skip all E2E tests for all vendors.
  • /skip-e2e-nvidia-all: To skip all E2E tests for NVIDIA vendor.
    Best regards.

@coveralls
Copy link

coveralls commented Jun 13, 2024 •
edited
Loading

Pull Request Test Coverage Report for Build 9501629157

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.04%) to 39.693%
Totals Coverage Status
Change from base Build 9402838972: 0.04%
Covered Lines: 5180
Relevant Lines: 13050
💛 - Coveralls

@bn222
Copy link
Collaborator

bn222 commented Jun 13, 2024

/lgtm

@github-actions github-actions bot added the lgtm label Jun 13, 2024
zeeke
zeeke approved these changes Jun 13, 2024
View reviewed changes
Copy link
Member

@zeeke zeeke left a comment

Choose a reason for hiding this comment

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

LGTM

Maybe after 4.16 GA we can stick to a GA version. WDYT?

@SchSeba SchSeba force-pushed the fix_ocp_ci_login branch from 4b1f5fb to 180f36a Compare June 13, 2024 18:11
@github-actions GitHub Actions
Copy link

Thanks for your PR,
To run vendors CIs use one of:

  • /test-all: To run all tests for all vendors.
  • /test-e2e-all: To run all E2E tests for all vendors.
  • /test-e2e-nvidia-all: To run all E2E tests for NVIDIA vendor.

To skip the vendors CIs use one of:

  • /skip-all: To skip all tests for all vendors.
  • /skip-e2e-all: To skip all E2E tests for all vendors.
  • /skip-e2e-nvidia-all: To skip all E2E tests for NVIDIA vendor.
    Best regards.

@coveralls
Copy link

coveralls commented Jun 13, 2024 •
edited
Loading

Pull Request Test Coverage Report for Build 9504663347

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • 3 unchanged lines in 1 file lost coverage.
  • Overall coverage increased (+0.02%) to 38.623%
Files with Coverage Reduction New Missed Lines %
controllers/drain_controller.go 3 67.1%
Totals Coverage Status
Change from base Build 9502015818: 0.02%
Covered Lines: 5177
Relevant Lines: 13404
💛 - Coveralls

@SchSeba SchSeba force-pushed the fix_ocp_ci_login branch from 180f36a to e9dda5e Compare June 13, 2024 18:55
@github-actions GitHub Actions
Copy link

Thanks for your PR,
To run vendors CIs use one of:

  • /test-all: To run all tests for all vendors.
  • /test-e2e-all: To run all E2E tests for all vendors.
  • /test-e2e-nvidia-all: To run all E2E tests for NVIDIA vendor.

To skip the vendors CIs use one of:

  • /skip-all: To skip all tests for all vendors.
  • /skip-e2e-all: To skip all E2E tests for all vendors.
  • /skip-e2e-nvidia-all: To skip all E2E tests for NVIDIA vendor.
    Best regards.

@SchSeba
Copy link
Collaborator Author

SchSeba commented Jun 13, 2024

/hold

@github-actions github-actions bot added the hold label Jun 13, 2024
@coveralls
Copy link

coveralls commented Jun 13, 2024 •
edited
Loading

Pull Request Test Coverage Report for Build 9505220855

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.04%) to 38.645%
Totals Coverage Status
Change from base Build 9502015818: 0.04%
Covered Lines: 5180
Relevant Lines: 13404
💛 - Coveralls

@SchSeba SchSeba force-pushed the fix_ocp_ci_login branch from e9dda5e to 319b480 Compare June 13, 2024 19:34
@github-actions GitHub Actions
Copy link

Thanks for your PR,
To run vendors CIs use one of:

  • /test-all: To run all tests for all vendors.
  • /test-e2e-all: To run all E2E tests for all vendors.
  • /test-e2e-nvidia-all: To run all E2E tests for NVIDIA vendor.

To skip the vendors CIs use one of:

  • /skip-all: To skip all tests for all vendors.
  • /skip-e2e-all: To skip all E2E tests for all vendors.
  • /skip-e2e-nvidia-all: To skip all E2E tests for NVIDIA vendor.
    Best regards.

@coveralls
Copy link

coveralls commented Jun 13, 2024 •
edited
Loading

Pull Request Test Coverage Report for Build 9505697056

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.04%) to 38.645%
Totals Coverage Status
Change from base Build 9502015818: 0.04%
Covered Lines: 5180
Relevant Lines: 13404
💛 - Coveralls

@SchSeba SchSeba force-pushed the fix_ocp_ci_login branch from 319b480 to 114099d Compare June 13, 2024 20:26
@github-actions GitHub Actions
Copy link

Thanks for your PR,
To run vendors CIs use one of:

  • /test-all: To run all tests for all vendors.
  • /test-e2e-all: To run all E2E tests for all vendors.
  • /test-e2e-nvidia-all: To run all E2E tests for NVIDIA vendor.

To skip the vendors CIs use one of:

  • /skip-all: To skip all tests for all vendors.
  • /skip-e2e-all: To skip all E2E tests for all vendors.
  • /skip-e2e-nvidia-all: To skip all E2E tests for NVIDIA vendor.
    Best regards.

@coveralls
Copy link

coveralls commented Jun 13, 2024 •
edited
Loading

Pull Request Test Coverage Report for Build 9506309484

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.05%) to 38.653%
Totals Coverage Status
Change from base Build 9502015818: 0.05%
Covered Lines: 5181
Relevant Lines: 13404
💛 - Coveralls

@SchSeba SchSeba force-pushed the fix_ocp_ci_login branch from 114099d to 13d4bec Compare June 16, 2024 09:22
@github-actions GitHub Actions
Copy link

Thanks for your PR,
To run vendors CIs use one of:

  • /test-all: To run all tests for all vendors.
  • /test-e2e-all: To run all E2E tests for all vendors.
  • /test-e2e-nvidia-all: To run all E2E tests for NVIDIA vendor.

To skip the vendors CIs use one of:

  • /skip-all: To skip all tests for all vendors.
  • /skip-e2e-all: To skip all E2E tests for all vendors.
  • /skip-e2e-nvidia-all: To skip all E2E tests for NVIDIA vendor.
    Best regards.

@SchSeba SchSeba force-pushed the fix_ocp_ci_login branch from 13d4bec to 6e40f36 Compare June 16, 2024 09:23
@github-actions GitHub Actions
Copy link

Thanks for your PR,
To run vendors CIs use one of:

  • /test-all: To run all tests for all vendors.
  • /test-e2e-all: To run all E2E tests for all vendors.
  • /test-e2e-nvidia-all: To run all E2E tests for NVIDIA vendor.

To skip the vendors CIs use one of:

  • /skip-all: To skip all tests for all vendors.
  • /skip-e2e-all: To skip all E2E tests for all vendors.
  • /skip-e2e-nvidia-all: To skip all E2E tests for NVIDIA vendor.
    Best regards.

@coveralls
Copy link

coveralls commented Jun 16, 2024 •
edited
Loading

Pull Request Test Coverage Report for Build 9534932450

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.04%) to 38.645%
Totals Coverage Status
Change from base Build 9502015818: 0.04%
Covered Lines: 5180
Relevant Lines: 13404
💛 - Coveralls

@coveralls
Copy link

coveralls commented Jun 16, 2024 •
edited
Loading

Pull Request Test Coverage Report for Build 9534932450

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.04%) to 38.645%
Totals Coverage Status
Change from base Build 9502015818: 0.04%
Covered Lines: 5180
Relevant Lines: 13404
💛 - Coveralls

@SchSeba SchSeba force-pushed the fix_ocp_ci_login branch from 6e40f36 to bdd8327 Compare June 16, 2024 17:07
@github-actions GitHub Actions
Copy link

Thanks for your PR,
To run vendors CIs use one of:

  • /test-all: To run all tests for all vendors.
  • /test-e2e-all: To run all E2E tests for all vendors.
  • /test-e2e-nvidia-all: To run all E2E tests for NVIDIA vendor.

To skip the vendors CIs use one of:

  • /skip-all: To skip all tests for all vendors.
  • /skip-e2e-all: To skip all E2E tests for all vendors.
  • /skip-e2e-nvidia-all: To skip all E2E tests for NVIDIA vendor.
    Best regards.

@SchSeba
Bump ocp version for CI lane
bdd8327 
this should make the ocp lane more stable

Signed-off-by: Sebastian Sch <[email protected]>
@coveralls
Copy link

coveralls commented Jun 16, 2024 •
edited
Loading

Pull Request Test Coverage Report for Build 9537600546

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.05%) to 38.653%
Totals Coverage Status
Change from base Build 9537529562: 0.05%
Covered Lines: 5181
Relevant Lines: 13404
💛 - Coveralls

@adrianchiris adrianchiris merged commit c542716 into k8snetworkplumbingwg:master Jun 17, 2024
12 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@adrianchiris adrianchiris adrianchiris approved these changes

@zeeke zeeke zeeke approved these changes

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

Successfully merging this pull request may close these issues.
5 participants
@SchSeba @coveralls @bn222 @zeeke @adrianchiris