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.

Sign up for GitHub

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

Jump to bottom

[R][CI] Installing quarto is failing #43635

Closed
jonkeane opened this issue Aug 10, 2024 · 2 comments
Closed

[R][CI] Installing quarto is failing #43635

jonkeane opened this issue Aug 10, 2024 · 2 comments
Assignees
@jonkeane
Labels
Component: Continuous Integration Component: R Type: bug
Milestone
18.0.0

Comments

@jonkeane
Copy link
Member

Describe the bug, including details regarding any error messages, version, and platform.

We've started getting this on at least the minimal libarrow check (possibly alsewhere):

Download action repository 'actions/checkout@3df4ab11eba7bda6032a0b82a6bb43b11571feac' (SHA:3df4ab11eba7bda6032a0b82a6bb43b11571feac)
Download action repository 'r-lib/actions@v2' (SHA:e6be4b3706e0f39bc7a4cf4496a5f2c4cb840040)
Getting action download info
Error: Bad request - quarto-dev/quarto-actions/setup@577558713f28cb3a0e7a735f80dce913d89b77d2 is not allowed to be used in apache/arrow. Actions in this workflow must be: within a repository owned by apache, created by GitHub, verified in the GitHub Marketplace, or matching the following: */*@[a-f0-9][a-f0-9][a-f0-9][a-f0-9][a-f0-9][a-f0-9][a-f0-9]+, AdoptOpenJDK/install-jdk@*, JamesIves/github-pages-deploy-action@5dc1d5a192aeb5ab5b7d5a77b7d36aea4a7f5c92, TobKed/label-when-approved-action@*, actions-cool/issues-helper@*, actions-rs/*, al-cheb/configure-pagefile-action@*, amannn/action-semantic-pull-request@*, apache/*, burrunan/gradle-cache-action@*, bytedeco/javacpp-presets/.github/actions/*, chromaui/action@*, codecov/codecov-action@*, conda-incubator/setup-miniconda@*, container-tools/kind-action@*, container-tools/microshift-action@*, dawidd6/action-download-artifact@*, delaguardo/setup-graalvm@*, docker://jekyll/jekyll:*, docker://pandoc/core:2.9, eps1lon/actions-label-merge-conflict@*, gaurav-nels...

We should probably ask INFRA to add quarto-dev to the approved list, but we might also be able to force this to not install (I'm not sure it's needed, and I'm a little surprised it's suddenly started being installed(?) or if there was a change elsewhere? We are using v2 of the r-lib actions, so that shouldn't have changed!

Component(s)

Continuous Integration, R
@github-actions github-actions bot mentioned this issue Aug 10, 2024
Merged
@jonkeane
Copy link
Member Author

That PR didn't work because it looks like the check is before we load. I've submitted the infra ticket: https://issues.apache.org/jira/browse/INFRA-26031

jonkeane added a commit that referenced this issue Aug 11, 2024
@jonkeane
GH-43635: [R][CI] Don't install Quarto (#43636)
ee86db5 
Pin to an old version of r-lib/actions that does not reference quarto-dev/quarto-actions. Though we still want to ask INFRA to add quarto-dev to the approved list.

* GitHub Issue: #43635

Authored-by: Jonathan Keane <[email protected]>
Signed-off-by: Jonathan Keane <[email protected]>
@jonkeane jonkeane added this to the 18.0.0 milestone Aug 11, 2024
@jonkeane
Copy link
Member Author

Issue resolved by pull request 43636
#43636

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jonkeane jonkeane

Labels
Component: Continuous Integration Component: R Type: bug
Projects
None yet
Milestone
18.0.0
Development

No branches or pull requests
1 participant
@jonkeane