Skip to content

Commit

Permalink
[CI] Update required checks to match pulsar-ci.yaml, remove workaround
Browse files Browse the repository at this point in the history
- Python client is required for Functions and Schema tests and
  tests were needed to move to a different category

- Sql integration tests ("CI - System - Sql") are disabled until
  #14951 has been resolved

- remove workaround from pulsar-ci.yaml which was needed to make
  the required checks pass
  • Loading branch information
lhotari committed Mar 30, 2022
1 parent 8221770 commit 4fb17ac
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 19 deletions.
7 changes: 4 additions & 3 deletions .asf.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -64,19 +64,20 @@ github:
- Build Pulsar java-test-image docker image
- CI - Integration - Backwards Compatibility
- CI - Integration - Cli
- CI - Integration - Function
- CI - Integration - Messaging
- CI - Integration - Schema
- CI - Integration - Shade
- CI - Integration - Standalone
- CI - Integration - Transaction
- Build Pulsar docker image
- CI - System - Function
- CI - System - Pulsar Connectors - Process
- CI - System - Pulsar Connectors - Thread
- CI - System - Pulsar IO
- CI - System - Sql
- CI - System - Schema
- CI - System - Tiered FileSystem
- CI - System - Tiered JCloud
# Sql integration tests are disabled until https://github.com/apache/pulsar/issues/14951 has been resolved
# - CI - System - Sql

required_pull_request_reviews:
dismiss_stale_reviews: false
Expand Down
16 changes: 0 additions & 16 deletions .github/workflows/pulsar-ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -765,19 +765,3 @@ jobs:
- name: Delete maven repository binaries from GitHub Actions Artifacts
run: |
gh-actions-artifact-client.js delete pulsar-maven-repository-binaries.tar.zst
# temporary workaround to make the required checks defined in .asf.yaml to pass
# these will be removed in later PRs once .asf.yaml has been edited
required-checks-workaround:
name: ${{ matrix.name }}
runs-on: ubuntu-20.04
strategy:
matrix:
name:
- CI - Integration - Function
- CI - Integration - Schema
- CI - System - Sql
steps:
- name: No-op step
run: |
echo "This is just to make the required checks to pass and will be removed in later PR to change .asf.yaml"

0 comments on commit 4fb17ac

Please sign in to comment.