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

v2.0.1 changes causing failed step in GitHub Actions #608

Closed
gpaulfleetwood opened this issue Apr 24, 2024 · 2 comments · Fixed by #610
Closed

v2.0.1 changes causing failed step in GitHub Actions #608

gpaulfleetwood opened this issue Apr 24, 2024 · 2 comments · Fixed by #610
Assignees
Labels
bug Something isn't working

Comments

@gpaulfleetwood
Copy link

gpaulfleetwood commented Apr 24, 2024

Describe the bug

Recent changes are causing an error in GitHub Actions. See output from workflow below:

Run ts-graphviz/setup-graphviz@v2
  with:
    ubuntu-skip-apt-update: false
    macos-skip-brew-update: false
  env:
    pythonLocation: /opt/hostedtoolcache/Python/3.11.9/x64
    PKG_CONFIG_PATH: /opt/hostedtoolcache/Python/3.11.9/x64/lib/pkgconfig
    Python_ROOT_DIR: /opt/hostedtoolcache/Python/3.11.9/x64
    Python2_ROOT_DIR: /opt/hostedtoolcache/Python/3.11.9/x64
    Python3_ROOT_DIR: /opt/hostedtoolcache/Python/3.11.9/x64
    LD_LIBRARY_PATH: /opt/hostedtoolcache/Python/3.11.9/x64/lib
/usr/bin/sudo apt-get clean
/usr/bin/sudo sudo rm /etc/apt/sources.list.d/microsoft-prod.list
root is not in the sudoers file.  This incident will be reported.
Error: The process '/usr/bin/sudo' failed with exit code 1

Workflow fails at this step.

To Reproduce

Steps to reproduce the behavior:
Use in a GitHub Action workflow like below:

      - name: 'Step 2 - Install Graphviz'
        uses: ts-graphviz/setup-graphviz@v2

Expected behavior

Graphviz should install. I have verified this works if changing the version to v2.0.0 instead of v2.

Additional context

I have also verified that the updated version v2.0.1 works on GitHub hosted runners. However, we are using self-hosted runners. Our solution may be pinning the action to v2.0.0. Not sure if self-hosted runners are used to test changes for this action, but consider that.

@kamiazya
Copy link
Member

@all-contributors please add @gpaulfleetwood for bug.

Copy link
Contributor

@kamiazya

I've put up a pull request to add @gpaulfleetwood! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants