Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Fix failing CI tests in forked repo PRs #818

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.

Sign up for GitHub

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

Merged
merged 19 commits into from
Feb 6, 2025
Merged

Fix failing CI tests in forked repo PRs #818

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
9623ec1
Introduce a change in workflow file
kumaranvpl Feb 5, 2025
1b207a8
Update version
kumaranvpl Feb 5, 2025
4e35b81
Give access to secrets only if triggering actor in with write access
kumaranvpl Feb 5, 2025
195b147
Run on specific cases explicitly
kumaranvpl Feb 5, 2025
8b0c261
Run for all main branches
kumaranvpl Feb 5, 2025
bdda895
Remove branches restriction
kumaranvpl Feb 5, 2025
7099970
Add user check in core tests with llms
kumaranvpl Feb 5, 2025
f2782c3
Experiment using pull_request instead of pull_request_target
kumaranvpl Feb 5, 2025
c8afb8d
Check user permission in graph rag tests
kumaranvpl Feb 5, 2025
2f53f4c
Provide permission to pull_request_target
kumaranvpl Feb 5, 2025
fd26e1c
WIP: Try permissions with contents write
kumaranvpl Feb 5, 2025
d55754b
Use pull_request_target
kumaranvpl Feb 5, 2025
02ac2ba
Experiment disabling branch
kumaranvpl Feb 5, 2025
9b3fc96
Merge branch 'main' into check-change
kumaranvpl Feb 5, 2025
09bab7d
Merge pull request #1 from kumaranvpl/check-change
kumaranvpl Feb 5, 2025
db05da4
Experiment
kumaranvpl Feb 5, 2025
aa4d7b5
Revert experimental changes
kumaranvpl Feb 6, 2025
82a8f6e
Revert version back to 0.7.3
kumaranvpl Feb 6, 2025
f20ec21
Merge branch 'ag2ai:main' into main
kumaranvpl Feb 6, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
18 changes: 17 additions & 1 deletion .github/workflows/contrib-graph-rag-tests.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
name: ContribGraphRagTests

on:
pull_request:
pull_request_target:
branches: ["main"]
paths:
- "autogen/agentchat/contrib/graph_rag/**"
Expand All @@ -21,6 +21,7 @@ permissions:
# checks: read
# contents: read
# deployments: read

jobs:
GraphRagIntegrationTest-FalkorDB-Ubuntu:
runs-on: ubuntu-latest
Expand All @@ -34,6 +35,21 @@ jobs:
ports:
- 6379:6379
steps:
- name: Get User Permission
id: checkAccess
uses: actions-cool/check-user-permission@v2
with:
require: write
username: ${{ github.triggering_actor }}
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
- name: Check User Permission
if: steps.checkAccess.outputs.require-result == 'false'
run: |
echo "${{ github.triggering_actor }} does not have permissions on this repo."
echo "Current permission level is ${{ steps.checkAccess.outputs.user-permission }}"
echo "Job originally triggered by ${{ github.actor }}"
exit 1
- uses: actions/checkout@v4
- uses: astral-sh/setup-uv@v5
with:
Expand Down
48 changes: 47 additions & 1 deletion .github/workflows/contrib-llm-test.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
name: Contrib tests with LLMs

on:
pull_request:
pull_request_target:
branches: ["main"]
paths:
- "autogen/**"
Expand All @@ -13,6 +13,7 @@ on:
- "pyproject.toml"
permissions:
{}

jobs:
TeachableAgent:
strategy:
Expand All @@ -22,6 +23,21 @@ jobs:
runs-on: ${{ matrix.os }}
environment: openai1
steps:
- name: Get User Permission
id: checkAccess
uses: actions-cool/check-user-permission@v2
with:
require: write
username: ${{ github.triggering_actor }}
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
- name: Check User Permission
if: steps.checkAccess.outputs.require-result == 'false'
run: |
echo "${{ github.triggering_actor }} does not have permissions on this repo."
echo "Current permission level is ${{ steps.checkAccess.outputs.user-permission }}"
echo "Job originally triggered by ${{ github.actor }}"
exit 1
# checkout to pr branch
- name: Checkout
uses: actions/checkout@v4
Expand Down Expand Up @@ -63,6 +79,21 @@ jobs:
runs-on: ${{ matrix.os }}
environment: openai1
steps:
- name: Get User Permission
id: checkAccess
uses: actions-cool/check-user-permission@v2
with:
require: write
username: ${{ github.triggering_actor }}
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
- name: Check User Permission
if: steps.checkAccess.outputs.require-result == 'false'
run: |
echo "${{ github.triggering_actor }} does not have permissions on this repo."
echo "Current permission level is ${{ steps.checkAccess.outputs.user-permission }}"
echo "Job originally triggered by ${{ github.actor }}"
exit 1
# checkout to pr branch
- name: Checkout
uses: actions/checkout@v4
Expand Down Expand Up @@ -108,6 +139,21 @@ jobs:
runs-on: ${{ matrix.os }}
environment: openai1
steps:
- name: Get User Permission
id: checkAccess
uses: actions-cool/check-user-permission@v2
with:
require: write
username: ${{ github.triggering_actor }}
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
- name: Check User Permission
if: steps.checkAccess.outputs.require-result == 'false'
run: |
echo "${{ github.triggering_actor }} does not have permissions on this repo."
echo "Current permission level is ${{ steps.checkAccess.outputs.user-permission }}"
echo "Job originally triggered by ${{ github.actor }}"
exit 1
# checkout to pr branch
- name: Checkout
uses: actions/checkout@v4
Expand Down
17 changes: 16 additions & 1 deletion .github/workflows/core-llm-test.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
name: Core tests with LLMs

on:
pull_request:
pull_request_target:
branches: ["main"]
paths:
- "autogen/**"
Expand Down Expand Up @@ -33,6 +33,21 @@ jobs:
- 6379:6379
options: --entrypoint redis-server
steps:
- name: Get User Permission
id: checkAccess
uses: actions-cool/check-user-permission@v2
with:
require: write
username: ${{ github.triggering_actor }}
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
- name: Check User Permission
if: steps.checkAccess.outputs.require-result == 'false'
run: |
echo "${{ github.triggering_actor }} does not have permissions on this repo."
echo "Current permission level is ${{ steps.checkAccess.outputs.user-permission }}"
echo "Job originally triggered by ${{ github.actor }}"
exit 1
# checkout to pr branch
- name: Checkout
uses: actions/checkout@v4
Expand Down
Loading