Skip to content

gh action: fix the GNU testsuite job#2225

Merged
sylvestre merged 1 commit into
uutils:masterfrom
sylvestre:gnu
May 17, 2021
Merged

gh action: fix the GNU testsuite job#2225
sylvestre merged 1 commit into
uutils:masterfrom
sylvestre:gnu

Conversation

@sylvestre
Copy link
Copy Markdown
Contributor

No description provided.

@nicoonoclaste
Copy link
Copy Markdown
Contributor

@sylvestre That doesn't fix the fundamental flakiness problem with this testsuite. I will just “work” until GNU pushes something to their coreutils repo that breaks it.

@sylvestre
Copy link
Copy Markdown
Contributor Author

Sure but it doesn't have to be perfect. Many other testsuites have flaky tests

@nicoonoclaste
Copy link
Copy Markdown
Contributor

So... the answer is to teach people to ignore CI because it sometimes breaks? 😒

@sylvestre
Copy link
Copy Markdown
Contributor Author

No, that it isn't always green and it is expected with 27 testsuites on different OS+hardware

@nicoonoclaste
Copy link
Copy Markdown
Contributor

No, that it isn't always green and it is expected with 27 testsuites on different OS+hardware

Eh, no? The source of flakiness here is easily preventable, so I sent a PR with an alternative proposal.

@sylvestre
Copy link
Copy Markdown
Contributor Author

please provide patches to fix them ;)

@sylvestre
Copy link
Copy Markdown
Contributor Author

note that we are also experiencing some network issues on some worker from time to time
or install issues

@sylvestre sylvestre merged commit dc02c36 into uutils:master May 17, 2021
@sylvestre sylvestre deleted the gnu branch May 18, 2021 17:01
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