crystal-lang / install-crystal

GitHub Action: Install Crystal programming language
https://crystal-lang.github.io/install-crystal/configurator.html
MIT License
67 stars 8 forks source link

Nightly Installs Failing #11

Closed grepsedawk closed 1 year ago

grepsedawk commented 2 years ago

Right now, when you try to install a nightly, it results in a circleCI 401 (unauthenticated) error: https://github.com/public-law/devise.cr/runs/5568601799?check_suite_focus=true#step:3:12

beta-ziliani commented 2 years ago

Working on it. Thanks!

straight-shoota commented 2 years ago

For reference: https://github.com/crystal-lang/crystal/pull/11902

grepsedawk commented 2 years ago

Cool, I thought I saw reference to somebody working on it, just wanted to make sure something was documented for others' having similar issues

For reference, the changes required to fix the common ci.yml look like this: https://github.com/grepsedawk/sharded.cr/commit/77ec4ca2df67cede29bb13fc43e1eb8c9a1b294d

straight-shoota commented 2 years ago

Removing nightly builds shouldn't be necessary, really. I'm confident we'll have this fixed and running quickly. Having a nightly build fail once shouldn't be an issue (they're unstable nightlies after all, so failure could happen for all kinds of reasons).

straight-shoota commented 2 years ago

The next nightlies should be available again (https://github.com/crystal-lang/crystal/pull/11902). We're just missing an update for the action.

oprypin commented 2 years ago

Ah, in the meantime the old approach to fetching the nightlies started working again on its own 😅

straight-shoota commented 2 years ago

🤦‍♂️

Maybe too many people complained at circle ci and they rolled back?

I think we should still continue with this change, though. We'd just depend on cicleci changing their mind again.

beta-ziliani commented 2 years ago

More importantly, it's The Right Thing to Do™

Blacksmoke16 commented 2 years ago

Failed again for me, but with a different error, could just be another blip :shrug:.

Installing package dependencies
Looking for latest Crystal build
sudo -n apt-get install -qy --no-install-recommends --no-upgrade -- libevent-dev libgmp-dev libpcre3-dev libssl-dev libxml2-dev libyaml-dev
Found Crystal build https://circleci.com/gh/crystal-lang/crystal/6[8](https://github.com/Blacksmoke16/oq/runs/5698495489?check_suite_focus=true#step:3:8)260

Error: TypeError: Cannot read property 'url' of undefined

https://github.com/Blacksmoke16/oq/runs/5698495489?check_suite_focus=true

Blacksmoke16 commented 1 year ago

This seems to have come back up, although with a new error:

2023-03-17T02:17:04.4892859Z Run crystal-lang/install-crystal@v1
2023-03-17T02:17:04.4893171Z with:
2023-03-17T02:17:04.4893381Z   crystal: nightly
2023-03-17T02:17:04.4893777Z   token: ***
2023-03-17T02:17:04.8896321Z Installing package dependencies
2023-03-17T02:17:04.8914142Z Looking for latest Crystal build
2023-03-17T02:17:04.9214428Z [command]sudo -n apt-get install -qy --no-install-recommends --no-upgrade -- libevent-dev libgmp-dev libpcre3-dev libssl-dev libxml2-dev libyaml-dev
2023-03-17T02:17:06.0214546Z Found Crystal build https://circleci.com/gh/crystal-lang/crystal/75069
2023-03-17T02:17:06.8993191Z ##[error]TypeError: artifacts.find is not a function

https://github.com/athena-framework/athena/actions/runs/4443502701/jobs/7800816554

EDIT: Seems to be intermittent. Probably something to do with the request to get the artifacts failing and returning an error object versus array.

Blacksmoke16 commented 1 year ago

https://discuss.circleci.com/t/issues-fetching-artifacts-vi-ui-and-api/47496/5

Thanks for reporting this. We have identified the root cause and will be rolling it back within the next hour.

That has since been completed and re-ran some of the jobs that failed due to this and they passed. So should be good to go.

straight-shoota commented 3 months ago

I'm seeing this error again: artifacts.find is not a function 😞

https://github.com/crystal-lang/test-ecosystem/actions/runs/9549827678/job/26320931726

Blacksmoke16 commented 3 months ago

Doesn't seem to be exact issue as last time. I'm kinda at a loss on this one. Can see this failure: https://github.com/Blacksmoke16/oq/actions/runs/9554914440/job/26338348064

Found Crystal build https://circleci.com/gh/crystal-lang/crystal/84001

But if I follow along with the code https://github.com/crystal-lang/install-crystal/blob/d2446e1c80c6741d8639115c1166578b43c62265/index.js#L405-L411

It works fine, in Chrome console at least:

image

:thinking:

Blacksmoke16 commented 3 months ago

Ah, I figured it out. When I reproduced the steps I was logged into CircleCI. However, hitting up the artifacts endpoint for that build in incognito results in:

{"message":"You must log in first"}

I found a thread and bumped it as it seems there was a change in behavior. TBD if that is intended or not.

oprypin commented 3 months ago

OK we have a problem indeed. It also happens on test runs of the action from today https://github.com/crystal-lang/install-crystal/actions/runs/9559784277

Blacksmoke16 commented 3 months ago

Seems to be working again as of now. Tho as per the other responses to that thread, TBD if it's actually resolved, or will break again at some point in the near future...