Skip to content

Try reverting some recent commits to see what CI does#1688

Closed
hoodmane wants to merge 1 commit intopyodide:mainfrom
hoodmane:test-ci
Closed

Try reverting some recent commits to see what CI does#1688
hoodmane wants to merge 1 commit intopyodide:mainfrom
hoodmane:test-ci

Conversation

@hoodmane
Copy link
Copy Markdown
Member

@hoodmane hoodmane commented Jul 3, 2021

This reverts commit 9ec832b.

Trying to diagnose recent timeout errors on CI.

@rth
Copy link
Copy Markdown
Member

rth commented Jul 3, 2021

I also restarted CI on 2ab427e before the last 3 commits, to see what happens. It was previously green.

@hoodmane
Copy link
Copy Markdown
Member Author

hoodmane commented Jul 3, 2021

Okay I think the failures of these

test_string_conversion[firefox]
test_bigint_conversions[firefox]
test_hyp_py2js2py[firefox]

on the rerun of 2ab427e demonstrates that my recent changes to string and bigint conversion aren't actually implicated in the timeouts...

@rth
Copy link
Copy Markdown
Member

rth commented Jul 4, 2021

Yes, as much as I like CircleCI, I think it might be time to evaluate an alternative CI provider to see if we also get such issues there. I'll try to setup Azure pipelines which have 10 parallel builds and unlimited CI minutes (though I'm not very fond of their UI).

This was referenced Jul 4, 2021
@rth
Copy link
Copy Markdown
Member

rth commented Jul 6, 2021

It might still be worth investigating Github Actions in #1690 in the future but for now CI appears to be green again.

@rth rth closed this Jul 6, 2021
@rth
Copy link
Copy Markdown
Member

rth commented Jul 6, 2021

Or at least I did see green CI in #1684 which included all of these commits.

@hoodmane hoodmane deleted the test-ci branch July 16, 2021 00:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants