Skip to content

feat: Add PAC github test#707

Open
srivickynesh wants to merge 1 commit intoopenshift-pipelines:masterfrom
srivickynesh:pac-github
Open

feat: Add PAC github test#707
srivickynesh wants to merge 1 commit intoopenshift-pipelines:masterfrom
srivickynesh:pac-github

Conversation

@srivickynesh
Copy link
Collaborator

No description provided.

@srivickynesh srivickynesh changed the title feat: Add github test feat: Add PAC github test Jan 5, 2026
@srivickynesh srivickynesh self-assigned this Feb 18, 2026
This scenario tests configuring PAC with push and pull_request events in GitHub.

Steps:
// * Validate PAC Info Install
Copy link
Collaborator

Choose a reason for hiding this comment

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

Can you remove the comment for this step

ctx := context.Background()
org := os.Getenv("GITHUB_ORG")
smeeURL := store.GetScenarioData("SMEE_URL")
token := os.Getenv("GITHUB_TOKEN")
Copy link
Collaborator

Choose a reason for hiding this comment

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

The current GITHUB_TOKEN has the creds of openshift-pipelines.
If we use the same, repos might get created under openshift-pipelines .
So it would be better if we can use a new org token as disccussed.
Also need to rename the token GITHUB_TOKEN which you are using to some other name as the current one is used for configuration of few tests.
Also need to include the new env variable in CI secrets .

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants