Skip to content

Commit fd4d626

Browse files
authored
fix(ci): unblock CI — approved paths-filter pin + happybase test flake (#405)
1 parent 263a936 commit fd4d626

2 files changed

Lines changed: 7 additions & 2 deletions

File tree

.github/workflows/CI.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -88,7 +88,7 @@ jobs:
8888
with:
8989
persist-credentials: false
9090
- name: Check for file changes
91-
uses: dorny/paths-filter@de90cc6fb38fc0963ad72b210f1f284cd68cea36 # v3.0.2
91+
uses: dorny/paths-filter@fbd0ab8f3e69293af611ebaee6363fc25e6d187d # v4.0.1
9292
id: filter
9393
with:
9494
# The following filters indicate a category along with

tests/plugin/data/sw_happybase/test_happybase.py

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,12 @@
2727
@pytest.fixture
2828
def prepare():
2929
# type: () -> Callable
30-
return lambda *_: requests.get('http://0.0.0.0:9090/users', timeout=5)
30+
# HBase cold-start `create_table` over Thrift can exceed a few seconds. A short
31+
# client timeout makes the readiness loop (conftest) retry while the server has
32+
# already created the table, so the retried request hits AlreadyExists -> 500 and
33+
# an extra error segment leaks into the collector (breaking segmentSize: 1).
34+
# Use a generous timeout so the first reachable request completes cleanly.
35+
return lambda *_: requests.get('http://0.0.0.0:9090/users', timeout=60)
3136

3237

3338
class TestPlugin(TestPluginBase):

0 commit comments

Comments
 (0)