Skip to content

chore: initialize fullsend per-repo installation#993

Merged
TomasTomecek merged 1 commit into
packit:mainfrom
TomasTomecek:fullsend/scaffold-install
Jul 22, 2026
Merged

chore: initialize fullsend per-repo installation#993
TomasTomecek merged 1 commit into
packit:mainfrom
TomasTomecek:fullsend/scaffold-install

Conversation

@TomasTomecek

@TomasTomecek TomasTomecek commented Jul 22, 2026

Copy link
Copy Markdown
Member

This PR adds the fullsend scaffold files for per-repo installation.

Merge this PR to activate fullsend workflows.

Edit: this PR was created with fullsend's CLI, hence the content, title and description

@TomasTomecek
TomasTomecek requested a review from a team as a code owner July 22, 2026 14:55
@TomasTomecek
TomasTomecek requested review from lbarcziova and removed request for a team July 22, 2026 14:55
@packit-as-a-service-stg

This comment was marked as outdated.

@packit-as-a-service

This comment was marked as outdated.

@packit-as-a-service-stg

This comment was marked as outdated.

@centosinfra-prod-github-app

This comment was marked as outdated.

@packit-as-a-service-stg

This comment was marked as outdated.

@TomasTomecek
TomasTomecek force-pushed the fullsend/scaffold-install branch from 4ecf9c1 to f27ffa3 Compare July 22, 2026 14:58
@centosinfra-prod-github-app

Copy link
Copy Markdown
Contributor

@lbarcziova lbarcziova left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

few comments, but I'm fine with testing this out

Comment thread .github/workflows/fullsend.yaml Outdated
secrets:
FULLSEND_GCP_WIF_PROVIDER: ${{ secrets.FULLSEND_GCP_WIF_PROVIDER }}
FULLSEND_GCP_PROJECT_ID: ${{ secrets.FULLSEND_GCP_PROJECT_ID }}
OTEL_EXPORTER_OTLP_TRACES_HEADERS: ${{ secrets.OTEL_EXPORTER_OTLP_TRACES_HEADERS }}

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I don't see this set in repo, is it mandatory?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

yes, that's a good point, I'm gonna drop it; it was a default configuration

Comment on lines +22 to +28
permissions:
actions: write
id-token: write
contents: write
issues: write
packages: read
pull-requests: write

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

these look quite broad 👀

Comment thread .github/workflows/fullsend.yaml Outdated
Comment on lines +57 to +89
stop-fix:
if: >-
github.event_name == 'issue_comment'
&& github.event.issue.pull_request
&& github.event.comment.user.type != 'Bot'
&& github.event.comment.body == '/fs-fix-stop'
&& (
github.event.comment.author_association == 'OWNER'
|| github.event.comment.author_association == 'MEMBER'
|| github.event.comment.author_association == 'COLLABORATOR'
|| github.event.comment.author_association == 'CONTRIBUTOR'
|| github.event.comment.user.login == github.event.issue.user.login
)
runs-on: ubuntu-24.04
permissions:
contents: read
issues: write
pull-requests: write
steps:
- name: Add fullsend-no-fix label and notify
env:
GH_TOKEN: ${{ github.token }}
PR_NUMBER: ${{ github.event.issue.number }}
REPO: ${{ github.repository }}
run: |
set -euo pipefail
gh label create "fullsend-no-fix" --repo "$REPO" \
--description "Skip bot-triggered fix agent runs" --color "FBCA04" \
--force 2>/dev/null || true
gh pr edit "$PR_NUMBER" --repo "$REPO" \
--add-label "fullsend-no-fix"
gh pr comment "$PR_NUMBER" --repo "$REPO" \
--body "Fix agent disabled for this PR. Remove the \`fullsend-no-fix\` label or use \`/fs-fix\` to re-engage."

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

does this essentially just label the issue? (in which case, do we really need it?)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

it is actually weird that only fix-stop workflow is here, and not the actual review, triage, code workflows

@TomasTomecek
TomasTomecek force-pushed the fullsend/scaffold-install branch from f27ffa3 to 4b119bc Compare July 22, 2026 15:34
@centosinfra-prod-github-app

Copy link
Copy Markdown
Contributor

@TomasTomecek
TomasTomecek merged commit 8a97872 into packit:main Jul 22, 2026
32 of 34 checks passed
@github-project-automation github-project-automation Bot moved this from New to Done in Packit pull requests Jul 22, 2026
@TomasTomecek
TomasTomecek deleted the fullsend/scaffold-install branch July 22, 2026 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

3 participants