open-telemetry/opentelemetry-collector

[CI] Changes to workflows are not tested on PRs

Open

#9,676 opened on Mar 4, 2024

View on GitHub
 (14 comments) (0 reactions) (1 assignee)Go (2,146 forks)auto 404
bugci-cdgood first issuepriority:p2release:retro

Repository metrics

Stars
 (7,237 stars)
PR merge metrics
 (PR metrics pending)

Description

During the v0.96.0 release cycle I had to file #9675 to fix the release pipeline. This is because #9409 accidentally broke the workflow. Mistakes happen, and it's important that we test workflows so that we catch these issues during the PR process.

Github seems to have something builtin, so probably we can reuse an action of theirs for this.

Contributor guide

Research direction
Investigate GitHub’s built in workflow testing actions (e.g., `github/action validator`) or simulate workflow changes via `act` locally. Set up a PR check that validates workflow file syntax or runs a dry run.
Tech stack
gogithub actions
Domain
ci cddevops
Issue type
DevOps
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
GitGitHub Actions
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
65