Skip to content

Use reusable Splunk agreements workflow#47

Closed
jcsco wants to merge 1 commit into
v2from
agent/use-reusable-agreements
Closed

Use reusable Splunk agreements workflow#47
jcsco wants to merge 1 commit into
v2from
agent/use-reusable-agreements

Conversation

@jcsco

@jcsco jcsco commented Jul 8, 2026

Copy link
Copy Markdown
Collaborator

Related to #46


Before the change?

  • The repository defined its CLA and Code of Conduct jobs locally with contributor-assistant/github-action@v2.3.0.
  • Both jobs failed while reading splunk/cla-agreement with Could not retrieve repository contents. Status: 401.
  • The workflow granted write access to repository contents and actions.

After the change?

  • Delegate both checks to Splunk's shared reusable-agreements.yaml@v1.7.2 workflow, following the pattern used by splunk-connect-for-snmp.
  • Map the centrally managed PAT_CLATOOL secret to the reusable workflow's PERSONAL_ACCESS_TOKEN input.
  • Reduce caller permissions to the least-privilege set required by the shared workflow.
  • Pick up contributor-assistant/github-action@v2.6.1 through the pinned reusable workflow.

Pull request checklist

  • Workflow syntax validated with actionlint v1.7.7
  • Docs reviewed; no documentation changes are needed

Validation Output

actionlint .github/workflows/cla.yaml

# no findings

The agreement jobs on this PR will still use the existing base-branch workflow because pull_request_target loads workflow definitions from the base branch. The reusable workflow will take effect after this change is merged.

Does this introduce a breaking change?

  • Yes
  • No

@jcsco jcsco marked this pull request as ready for review July 8, 2026 16:54
@jcsco jcsco closed this Jul 9, 2026
@jcsco jcsco deleted the agent/use-reusable-agreements branch July 9, 2026 00:29
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 9, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants