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 a prow_config.yaml file to configure our prow jobs. #388

Merged
merged 21 commits into from
Feb 15, 2018

Conversation

jlewi
Copy link
Contributor

@jlewi jlewi commented Feb 12, 2018

  • See checkout.sh should support checking out extra repositories. testing#12

  • Our prow jobs will trigger a script that will process prow_config.yaml
    and figure out which workflows to run.

  • This will allow us to easily run multiple workflows as part of our E2E tests.

  • Fix one source of flakiness in TFJob test

    • If we get the status right after creating the TFJob the status might not be set yet
    • This was causing wait_for_tfjob to exit with KeyError exception which caused the test
      to fail with a very cryptic error message
    • We fix this by explicitly allowing for status not being set yet.

This change is Reviewable

* See kubeflow/testing#12
* Our prow jobs will trigger a script that will process prow_config.yaml
  and figure out which workflows to run.
* This will allow us to easily run multiple workflows as part of our E2E tests.
@jlewi
Copy link
Contributor Author

jlewi commented Feb 12, 2018

/test all

5 similar comments
@jlewi
Copy link
Contributor Author

jlewi commented Feb 12, 2018

/test all

@jlewi
Copy link
Contributor Author

jlewi commented Feb 12, 2018

/test all

@jlewi
Copy link
Contributor Author

jlewi commented Feb 13, 2018

/test all

@jlewi
Copy link
Contributor Author

jlewi commented Feb 13, 2018

/test all

@jlewi
Copy link
Contributor Author

jlewi commented Feb 13, 2018

/test all

@jlewi
Copy link
Contributor Author

jlewi commented Feb 14, 2018

The test failures are due to kubeflow/testing#30

@jlewi jlewi changed the title [wip] Add a prow_config.yaml file to configure our prow jobs. Add a prow_config.yaml file to configure our prow jobs. Feb 14, 2018
@jlewi
Copy link
Contributor Author

jlewi commented Feb 14, 2018

@gaocegege @lluunn This is ready for review. The tests are passing; the failure is due to kubeflow/testing#30.

@jlewi
Copy link
Contributor Author

jlewi commented Feb 14, 2018

/test all

1 similar comment
@jlewi
Copy link
Contributor Author

jlewi commented Feb 14, 2018

/test all

@@ -8,7 +8,7 @@
// Each object below should correspond to a component in the components/ directory
workflows: {
bucket: "mlkube-testing_temp",
name: "jlewi-tf-k8s-presubmit-test-374-6e32",
name: "some-very-very-very-very-very-long-name-jlewi-tf-k8s-presubmit-test-374-6e32",
Copy link
Member

Choose a reason for hiding this comment

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

Why do we need a very very very long name 🤔

Copy link
Member

Choose a reason for hiding this comment

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

OK, I saw the code below.

Copy link
Member

@gaocegege gaocegege left a comment

Choose a reason for hiding this comment

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

LGTM 🎉

@jlewi
Copy link
Contributor Author

jlewi commented Feb 15, 2018

/test al

@jlewi
Copy link
Contributor Author

jlewi commented Feb 15, 2018

/test all

@jlewi
Copy link
Contributor Author

jlewi commented Feb 15, 2018

Woo Hoo
presubmit is passing again.

@jlewi jlewi merged commit 37f5533 into kubeflow:master Feb 15, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants