Skip to content
This repository has been archived by the owner on Oct 10, 2023. It is now read-only.

Skip running cc based pipelines if PR is from a fork #3313

Merged
merged 1 commit into from
Sep 12, 2022

Conversation

saji-pivotal
Copy link
Contributor

@saji-pivotal saji-pivotal commented Sep 12, 2022

What this PR does / why we need it

Skips the CC based integration test workflow if the PR is submitted from a fork

Which issue(s) this PR fixes

Fixes #

Describe testing done for PR

  • Github CI pipelines pass

Release note

Skips cc based integration tests if the PR is from a fork

Additional information

Special notes for your reviewer

@codecov
Copy link

codecov bot commented Sep 12, 2022

Codecov Report

Merging #3313 (7cd87ed) into main (48a7073) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #3313   +/-   ##
=======================================
  Coverage   45.58%   45.58%           
=======================================
  Files         324      324           
  Lines       34126    34126           
=======================================
  Hits        15558    15558           
  Misses      17102    17102           
  Partials     1466     1466           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Copy link
Contributor

@anujc25 anujc25 left a comment

Choose a reason for hiding this comment

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

LGTM

@anujc25 anujc25 added kind/failing-test Categorizes issue or PR as related to a failing test ok-to-merge PRs should be labelled with this before merging labels Sep 12, 2022
Copy link
Contributor

@anujc25 anujc25 left a comment

Choose a reason for hiding this comment

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

We should keep the package build and publish part and only skip the tests

.github/workflows/tkg_integration_tests_clusterclass.yaml Outdated Show resolved Hide resolved
.github/workflows/tkg_integration_tests_clusterclass.yaml Outdated Show resolved Hide resolved
Copy link
Contributor

@vuil vuil left a comment

Choose a reason for hiding this comment

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

lgtm, thanks

@saji-pivotal saji-pivotal merged commit f470b6c into vmware-tanzu:main Sep 12, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla-not-required kind/failing-test Categorizes issue or PR as related to a failing test ok-to-merge PRs should be labelled with this before merging
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants