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

Hack around CI problems by sudo chown and disabling cache for Windows validate #8739

Merged
merged 12 commits into from
Feb 8, 2023

Conversation

Mikolaj
Copy link
Member

@Mikolaj Mikolaj commented Feb 7, 2023

The chown problem is this one: actions/runner-images#7061

The Windows validate problems are the standard kind

PackageTests\PreProcess\Hsc2HsOptionsCC\setup.test.hs:4:1: error:
Error:     Ambiguous module name ‘System.Directory’:
      it was found in multiple packages:
      directory-1.3.7.1 directory-1.3.7.1
  |
4 | import System.Directory (findExecutable)

see #8356. They resurfaced again now that github is updating images and the store/ caches are being somehow mangled/reverted to old versions/merged/AI-trained/whatever. Note, however, that previous tests show that invalidating caches does not fix (at least some of) these errors, which is why such a desperate step. We should recover the caching once someone implements #8742.

@Mikolaj Mikolaj force-pushed the try-chmod-to-fix-CI branch 2 times, most recently from 8f1185b to dde95c7 Compare February 8, 2023 08:09
@Mikolaj Mikolaj reopened this Feb 8, 2023
@Mikolaj Mikolaj mentioned this pull request Feb 8, 2023
10 tasks
@Mikolaj Mikolaj changed the title Try to fix CI via sudo chmod in ghcup cache Hack around GHA problems by sudo chown and disabling cache for Windows validate Feb 8, 2023
@Mikolaj Mikolaj changed the title Hack around GHA problems by sudo chown and disabling cache for Windows validate Hack around CI problems by sudo chown and disabling cache for Windows validate Feb 8, 2023
@Mikolaj Mikolaj added the squash+merge me Tell Mergify Bot to squash-merge label Feb 8, 2023
@Mikolaj Mikolaj added the merge delay passed Applied (usually by Mergify) when PR approved and received no updates for 2 days label Feb 8, 2023
@Mikolaj
Copy link
Member Author

Mikolaj commented Feb 8, 2023

@mergify rebase

@mergify
Copy link
Contributor

mergify bot commented Feb 8, 2023

rebase

✅ Nothing to do for rebase action

@Mikolaj
Copy link
Member Author

Mikolaj commented Feb 13, 2023

@mergify backport

@mergify
Copy link
Contributor

mergify bot commented Feb 13, 2023

backport

❌ No backport have been created

No destination branches found

@Mikolaj
Copy link
Member Author

Mikolaj commented Feb 13, 2023

@mergify backport 3.10

@mergify
Copy link
Contributor

mergify bot commented Feb 13, 2023

backport 3.10

✅ Backports have been created

mergify bot pushed a commit that referenced this pull request Feb 13, 2023
… validate (#8739)

* Try to fix ghcup cache in CI via sudo chmod

* Revert "Try to fix ghcup cache in CI via sudo chmod"

This reverts commit 0bb71d1.

* Try not caching store in validate jobs to avoid double package bugs

* Revert "Try not caching store in validate jobs to avoid double package bugs"

This reverts commit b3f3782.

* Try not to cache store on Windows in validate jobs

to avoid double directory package bugs.

* Revert "Revert "Try to fix ghcup cache in CI via sudo chmod""

This reverts commit b3a843a.

* Try sudo only on (new) Linux, where it should be available

* Also change the user

* Document the particular hack snippets in validate.yml

* Add an extra diagnostic ls for ghcup dirs

* Also hack the other affected CI scripts

* Also print the version of ghcup used for debugging

(cherry picked from commit 22dcb3c)
mergify bot added a commit that referenced this pull request Feb 13, 2023
Hack around CI problems by sudo chown and disabling cache for Windows validate (backport #8739)
@ulysses4ever ulysses4ever mentioned this pull request Mar 17, 2023
15 tasks
ulysses4ever added a commit that referenced this pull request Mar 23, 2023
because of #8739 on all platforms!
ulysses4ever added a commit that referenced this pull request Mar 23, 2023
because of #8739 on all platforms!
ulysses4ever added a commit that referenced this pull request Mar 23, 2023
because of #8739 on all platforms!
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
attention: needs-review continuous-integration merge delay passed Applied (usually by Mergify) when PR approved and received no updates for 2 days squash+merge me Tell Mergify Bot to squash-merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant