Skip to content
This repository has been archived by the owner on Jan 15, 2024. It is now read-only.

[v0.x] Backport CI from Master to v0.x #1531

Merged
merged 14 commits into from
Mar 5, 2021
Merged

Conversation

barry-jin
Copy link
Contributor

Description

fix issue #1529

Checklist

Essentials

  • PR's title starts with a category (e.g. [BUGFIX], [MODEL], [TUTORIAL], [FEATURE], [DOC], etc)
  • Changes are complete (i.e. I finished coding on this PR)
  • All changes have test coverage
  • Code is well-documented

Changes

  • Feature1, tests, (and when applicable, API doc)
  • Feature2, tests, (and when applicable, API doc)

Comments

  • If this change is a backward incompatible change, why must this change be made.
  • Interesting edge cases to note here

cc @dmlc/gluon-nlp-team

@barry-jin barry-jin requested a review from a team as a code owner February 24, 2021 18:18
@szha
Copy link
Member

szha commented Mar 2, 2021

we probably need to move the test tasks to our own workers.

@barry-jin barry-jin changed the title [WIP][v0.x] Backport CI from Master to v0.x [v0.x] Backport CI from Master to v0.x Mar 4, 2021
@szha
Copy link
Member

szha commented Mar 5, 2021

submit-job.py: error: unrecognized arguments: --conda-env docker/py3

@barry-jin
Copy link
Contributor Author

barry-jin commented Mar 5, 2021

submit-job.py: error: unrecognized arguments: --conda-env docker/py3

For v0.x CI, should we replace current jenkins CI with github actions or keep some of the jenkins jobs and move the others to github actions. In issue #1529, the jenkins CI will pass randomly, so I try to replace current jenkins CI with github actions by backporting the cpu, gpu unittests and website build from master branch, including the submit-job.py script.

@szha
Copy link
Member

szha commented Mar 5, 2021

OK. I removed the required status for the old jenkins checks.

@leezu leezu merged commit 8b79a33 into dmlc:v0.x Mar 5, 2021
szha pushed a commit that referenced this pull request Mar 21, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants