Skip to content

feat: Trigger enterprise testing on commit#2298

Merged
andrew-sha merged 13 commits intomainfrom
enterprise_testing_webhook
Mar 22, 2024
Merged

feat: Trigger enterprise testing on commit#2298
andrew-sha merged 13 commits intomainfrom
enterprise_testing_webhook

Conversation

@andrew-sha
Copy link
Contributor

@andrew-sha andrew-sha commented Mar 18, 2024

This adds a job in the main_pr CircleCI workflow which emits a webhook to enterprise's pipeline endpoint to trigger testing (PR #195 in sqlmesh-enterprise repository)

@andrew-sha andrew-sha changed the title enterprise testing webhook feat: Trigger enterprise testing on commit Mar 19, 2024
@andrew-sha andrew-sha marked this pull request as ready for review March 19, 2024 16:21
--url https://circleci.com/api/v2/project/gh/TobikoData/sqlmesh-enterprise/pipeline \
--header "Circle-Token: $ENTERPRISE_CIRCLE_API_TOKEN" \
--header "content-type: application/json" \
--data '{"branch":"trigger_workflow_on_post","parameters":{"run_main_pr":false, "run_core_commit":true, "sqlmesh_version":"'"$CIRCLE_BRANCH"'"}}'
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Once the complementary changes in enterprise are merged this changes to "branch":"main"--I have it set to the feature branch for testing

@andrew-sha andrew-sha requested a review from a team March 19, 2024 16:27
@andrew-sha andrew-sha marked this pull request as draft March 20, 2024 17:30
workflows:
main_pr:
jobs:
- trigger_private_tests
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we move this to the end to prevent unnecessary triggers? If the PR's tests fail, no need to trigger.

@CLAassistant
Copy link

CLAassistant commented Mar 21, 2024

CLA assistant check
All committers have signed the CLA.

@andrew-sha andrew-sha force-pushed the enterprise_testing_webhook branch from 0725960 to b60cc96 Compare March 21, 2024 18:38
@andrew-sha andrew-sha marked this pull request as ready for review March 22, 2024 15:51
@andrew-sha andrew-sha merged commit 326263c into main Mar 22, 2024
@andrew-sha andrew-sha deleted the enterprise_testing_webhook branch March 22, 2024 15:51
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.

4 participants