Skip to content

TST: Move 32-bit and parallel job to Actions - #11043

Merged
saimn merged 3 commits into
astropy:masterfrom
pllim:actions-32bit
Nov 25, 2020
Merged

saimn merged 3 commits into
astropy:masterfrom
pllim:actions-32bit

Conversation

@pllim

@pllim pllim commented Nov 14, 2020 •

Copy link
Copy Markdown
Member

Description

This pull request is to move "32-bit and parallel" job from CircleCI to GitHub Actions. I purposely let the full CI run to make sure I didn't mess up CircleCI config. (CircleCI seems to work as expected in the first commit, so now I will skip it.)

There were talks about we should stop using Docker image or something but I am not sure what to do with that part.

This is a follow up of #10388 and address part of #11038 .

@pllim pllim added this to the v4.3 milestone Nov 14, 2020
@pllim pllim mentioned this pull request Nov 14, 2020
18 tasks done
Comment thread .github/workflows/ci_workflows.yml Outdated
image: quay.io/pypa/manylinux1_i686
steps:
- name: Checkout code
uses: actions/checkout@v2

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Looks like this doesn't work with container?

/usr/bin/docker exec  .... sh -c "cat /etc/*release | grep ^ID"
OCI runtime exec failed: exec failed: container_linux.go:370: starting container process caused: no such file or directory: unknown

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Looks like a known and unresolved issue at actions/checkout#334

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Subscribed!

@pllim
pllim requested review from Cadair and astrofrog November 14, 2020 15:39
@pllim

pllim commented Nov 24, 2020

Copy link
Copy Markdown
Member Author

Is there anything that is blocking this?

Comment thread .github/workflows/ci_workflows.yml Resolved

@astrojuanlu astrojuanlu left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

The Actions spec looks like a good translation of the Circle CI one. I went over the Actions logs and didn't spot anything suspicious.

@saimn saimn left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM.

@saimn

saimn commented Nov 25, 2020

Copy link
Copy Markdown
Contributor

I don't see a reason to delay this, so let's merge! Thanks @pllim.

@saimn
saimn merged commit 883a7c4 into astropy:master Nov 25, 2020
@pllim
pllim deleted the actions-32bit branch November 25, 2020 15:41
@pllim

pllim commented Nov 25, 2020

Copy link
Copy Markdown
Member Author

Thanks for the reviews, y'all!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants