kubeflow / pipelines

Machine Learning Pipelines for Kubeflow
https://www.kubeflow.org/docs/components/pipelines/
Apache License 2.0
3.6k stars 1.62k forks source link

Migrate postsubmit tests to GH Actions #10747

Closed rimolive closed 1 week ago

rimolive commented 5 months ago

Description

As part of the main tracker issue, we need to migrate postsubmit tests to GH Actions.

Acceptance Criteria

Impacted by this bug? Give it a 👍.

rimolive commented 5 months ago

/remove-kind bug /kind feature /good-first-issue

google-oss-prow[bot] commented 5 months ago

@rimolive: This request has been marked as suitable for new contributors.

Please ensure the request meets the requirements listed here.

If this request no longer meets these requirements, the label can be removed by commenting with the /remove-good-first-issue command.

In response to [this](https://github.com/kubeflow/pipelines/issues/10747): >/remove-kind bug >/kind feature >/good-first-issue Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.
rimolive commented 5 months ago

/remove-area sdk

VDliveson commented 5 months ago

/assign

VDliveson commented 5 months ago

@rimolive What is this annotation and how is it referenced here ?

annotations:
      testgrid-dashboards: googleoss-kubeflow-pipelines
      description: Postsubmit integration tests for kubeflow/pipeline.
      testgrid-alert-email: kubeflow-pipelines+test@google.com
      testgrid-num-failures-to-alert: "5"
DharmitD commented 5 months ago

@rimolive What is this annotation and how is it referenced here ?

annotations:
      testgrid-dashboards: googleoss-kubeflow-pipelines
      description: Postsubmit integration tests for kubeflow/pipeline.
      testgrid-alert-email: kubeflow-pipelines+test@google.com
      testgrid-num-failures-to-alert: "5"

@VDliveson the annotation provides metadata that is used by TestGrid, a dashboarding tool for displaying test results from CI systems like Prow.

DharmitD commented 5 months ago

@VDliveson could you provide an update on what the status is for this task? Let us know if you need any help, we'll be happy to collaborate and get this issue resolved.

VDliveson commented 5 months ago

@DharmitD As I don't have much experience with Github actions, I would like you to suggest any changes in my code and assist me with how to test this locally.

DharmitD commented 5 months ago

@VDliveson thank you for submitting a PR for this issue! I'm reviewing your PR, I would suggest adding this GH Action to your fork and triggering a run. That way you can test this action and ensure it executes all steps successfully and as expected.

I've reached out to you on the Kubeflow slack, you can ping me there if you have any additional questions.

github-actions[bot] commented 3 months ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

rimolive commented 3 months ago

/lifecycle frozen

sky0walker99 commented 2 weeks ago

Hi @VDliveson / team, I was wondering if there has been any recent progress on this issue or if anyone is currently working on it? I would be happy to assist or pick it up if it's not being actively worked on. Thanks!

VD-blinkit commented 2 weeks ago

Hi @sky0walker99 , The development has mostly been completed with the PR being raised for the same, but there has been no further reply on the team for the same

hbelmiro commented 1 week ago

/close as done.