Skip to content
This repository has been archived by the owner on Oct 17, 2021. It is now read-only.

Error pushing bottle with this action #9

Closed
renehernandez opened this issue Dec 1, 2020 · 14 comments · Fixed by #10 or #11
Closed

Error pushing bottle with this action #9

renehernandez opened this issue Dec 1, 2020 · 14 comments · Fixed by #10 or #11

Comments

@renehernandez
Copy link

I am following the example you provided in the docs to bottle a formula and update the downstream tap, but it fails with the following error:

Error: Container action is only supported on Linux

Github Action run with the failure: https://github.com/renehernandez/appfile/runs/1480833469?check_suite_focus=true

@MaxDesiatov
Copy link

You need to specify runs-on: ubuntu-latest to use this formula. Container actions can't be run on macOS GitHub Actions agents.

@renehernandez
Copy link
Author

In the Readme example at updating-a-formula-with-a-bottle the job specifies runs-on: macos-10.15

@renehernandez
Copy link
Author

You need to specify runs-on: ubuntu-latest to use this formula. Container actions can't be run on macOS GitHub Actions agents.

Does it means, that I will need to cache the generated bottle file and run the action to update the tap on another job (running on ubuntu?

@MaxDesiatov
Copy link

MaxDesiatov commented Dec 1, 2020

In my understanding yes, but maybe @mattt can clarify it.

@renehernandez
Copy link
Author

If that is the case, then the example in Readme should be updated to reflect the correct usage. Once I get it working I'll submit a PR

@mattt
Copy link
Member

mattt commented Dec 1, 2020

@renehernandez Thanks for reporting this. Unfortunately, testing GitHub Actions is proving to be difficult, and I haven't had much bandwidth to invest in a reasonable solution.

I just opened #10 which should (🤞) work. If you have some time, would you be able to re-run with this to confirm that it resolves your issue?

@renehernandez
Copy link
Author

Testing it now

@renehernandez
Copy link
Author

The change you suggested fixes the issue of running the job 👍 !!

Though I ran into another issue it seems, the formula file is not being update with the bottle information.

Github Action run: https://github.com/renehernandez/appfile/actions/runs/396906682
Formula in tap repo: https://github.com/renehernandez/homebrew-taps/blob/main/Formula/appfile.rb

@mattt mattt closed this as completed in #10 Dec 3, 2020
@renehernandez
Copy link
Author

@mattt Should I open a new issue about my last comment?

@mattt
Copy link
Member

mattt commented Dec 3, 2020

@renehernandez Thanks so much for confirming the fix!

I have a theory about what caused your last run to fail, which #11 may address (namely, a disparity in the v prefix for tag name and semantic version). Whenever it's convenient, could you re-run against NSHipster/update-homebrew-formula-action@fix-bottle-regex?

If that doesn't fix it, we can open up a new issue thread to track this.

@renehernandez
Copy link
Author

Whenever it's convenient, could you re-run against NSHipster/update-homebrew-formula-action@fix-bottle-regex?

I'll take a look and get back to you later today

@renehernandez
Copy link
Author

@mattt Should we continue on a new Issue?

@mattt mattt closed this as completed in #11 Dec 21, 2020
@mattt mattt reopened this Dec 21, 2020
@mattt
Copy link
Member

mattt commented Dec 21, 2020

@renehernandez I just successfully* ran it with the latest HEAD on main NSHipster/homebrew-formulae@c780bc0. If you're still having problems, please open a new issue. Thanks again for your help tracking this down.

* The workflow is failing on workflows/upload-release-asset with "Error: request to [...] failed, reason: socket hang up" (actions/upload-release-asset#69 🤷‍♂️.

@mattt mattt closed this as completed Dec 21, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
3 participants