Skip to content

Fix auto-update-community-members workflow#7743

Merged
svrnm merged 1 commit intoopen-telemetry:mainfrom
embrace-io:aridemarco/auto-update-community-members-fix
Sep 11, 2025
Merged

Fix auto-update-community-members workflow#7743
svrnm merged 1 commit intoopen-telemetry:mainfrom
embrace-io:aridemarco/auto-update-community-members-fix

Conversation

@ArielDemarco
Copy link
Member

Overview

This PR attempts to fix the auto-update-community-members workflow.

Details

The auto-update-community-members job has been failing for the past 3 weeks (first failure). The last successful run was https://github.com/open-telemetry/opentelemetry.io/actions/runs/17059027027.

The issue stems from the introduction of the fix:refcache step, which internally executes npm run _hugo. This command was failing because the required dependencies weren't installed in the current working directory. While the workflow does install dependencies earlier, it does so in the subdirectory of the script to generate the community data (./scripts/generate-community-data), but the fix:refcache command runs from the root directory where node_modules aren't available.

The fix is simple: add npm install --omit=optional before running npm run fix:refcache to ensure all necessary dependencies are available when the script executes.

@ArielDemarco ArielDemarco requested a review from a team as a code owner September 10, 2025 14:07
@ArielDemarco ArielDemarco changed the title Installing dependencies so fix:refcache can work properly Fix auto-update-community-members workflow Sep 10, 2025
Copy link
Member

@vitorvasc vitorvasc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for noticing it and also thanks for bringing this fix. Appreciate it!

@vitorvasc vitorvasc added the ready-to-be-merged This PR is ready to be merged by a maintainer label Sep 10, 2025
@svrnm svrnm added this pull request to the merge queue Sep 11, 2025
Merged via the queue into open-telemetry:main with commit 8e00950 Sep 11, 2025
19 checks passed
asapple pushed a commit to asapple/opentelemetry.io that referenced this pull request Sep 27, 2025
jaehanbyun pushed a commit to jaehanbyun/opentelemetry.io that referenced this pull request Dec 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-be-merged This PR is ready to be merged by a maintainer

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants