Skip to content

feat(ci): add integration tests for AutoTLS in Beekeeper workflow #5350

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

Open
akrem-chabchoub wants to merge 19 commits into
base: master
Choose a base branch
from
Open

feat(ci): add integration tests for AutoTLS in Beekeeper workflow #5350

Changes from 7 commits
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
4420011
feat(ci): add integration tests for AutoTLS in Beekeeper workflow
akrem-chabchoub Feb 5, 2026
00c4ec7
feat(ci): enable P2P WebSocket support in Beekeeper workflow
akrem-chabchoub Feb 5, 2026
23279ad
fix(ci): update dependencies for ArgoCD trigger in Beekeeper workflow
akrem-chabchoub Feb 5, 2026
2b2b653
refactor(ci): add setup action for Beekeeper cluster deployment
akrem-chabchoub Feb 5, 2026
9ccf744
chore(ci): add checkout step to Beekeeper workflow for improved ref h…
akrem-chabchoub Feb 5, 2026
8ca3b1e
refactor(ci): remove setup action for Beekeeper cluster and inline st…
akrem-chabchoub Feb 5, 2026
282665c
refactor(ci): merge beekeeper and beekeeper-autotls
akrem-chabchoub Feb 9, 2026
31a1d0d
chore(ci): update Beekeeper workflow to use new branch and image tags
akrem-chabchoub Feb 16, 2026
a81b30f
chore(ci): update Beekeeper workflow to reference new autotls-check b…
akrem-chabchoub Feb 16, 2026
b25030a
chore(ci): update PEBBLE_IMAGE_TAG in Beekeeper workflow to v2.9.0 (t…
akrem-chabchoub Feb 16, 2026
3f59296
chore: update beekeeper branch
akrem-chabchoub Feb 17, 2026
1102828
chore: update beelocal and beekeeper branch
akrem-chabchoub Feb 17, 2026
00c7a11
chore: update beekeeper branch to fix/pebble-timout
akrem-chabchoub Feb 17, 2026
13c104e
fix: update pebble img tag to v2.9.0
akrem-chabchoub Feb 17, 2026
32c3a15
chore: update pebble and p2p-forge img tags
akrem-chabchoub Feb 17, 2026
19df6a0
chore: update beekeeper branch
akrem-chabchoub Feb 17, 2026
7fdabb3
Merge branch 'master' of https://github.com/ethersphere/bee into feat…
akrem-chabchoub Feb 20, 2026
d09d15c
chore: add pebble cert validity period
akrem-chabchoub Feb 20, 2026
a8da204
fix: simplify failure conditions in beekeeper workflow for artifact c…
akrem-chabchoub Feb 20, 2026
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
62 changes: 45 additions & 17 deletions .github/workflows/beekeeper.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,8 +13,8 @@ env:
RUN_TYPE: "PR RUN"
SETUP_CONTRACT_IMAGE: "ethersphere/bee-localchain"
SETUP_CONTRACT_IMAGE_TAG: "0.9.4"
BEELOCAL_BRANCH: "main"
BEEKEEPER_BRANCH: "master"
BEELOCAL_BRANCH: "akrem/p2p-wss-support-v2" # TODO: change to main (temporary testing branch)
BEEKEEPER_BRANCH: "feat/autotls-cluster" # TODO: change to master (temporary testing branch)
BEEKEEPER_METRICS_ENABLED: false
REACHABILITY_OVERRIDE_PUBLIC: true
BATCHFACTOR_OVERRIDE_PUBLIC: 2
Expand All @@ -25,6 +25,7 @@ env:
AWS_EC2_METADATA_DISABLED: true
AWS_ENDPOINT: fra1.digitaloceanspaces.com
VERTAG: ${GITHUB_RUN_ID}
P2P_WSS_ENABLE: true
jobs:
init:
name: Init
Expand All @@ -34,15 +35,9 @@ jobs:
steps:
- name: Checkout
uses: actions/checkout@v5
if: github.event.action != 'beekeeper'
with:
fetch-depth: 0
- name: Checkout
uses: actions/checkout@v5
if: github.event.action == 'beekeeper'
with:
fetch-depth: 0
ref: ${{ github.event.client_payload.ref }}
ref: ${{ github.event.action == 'beekeeper' && github.event.client_payload.ref || github.sha }}
- name: Setup Go
uses: actions/setup-go@v6
with:
Expand Down Expand Up @@ -99,6 +94,11 @@ jobs:
runs-on: ubuntu-latest
needs: [init]
steps:
- name: Checkout
uses: actions/checkout@v5
with:
fetch-depth: 0
ref: ${{ github.event.action == 'beekeeper' && github.event.client_payload.ref || github.sha }}
- name: Cache
uses: actions/cache@v4
with:
Expand All @@ -113,19 +113,17 @@ jobs:
run: |
chmod +x bee-1 beekeeper .github/bin/beekeeper_artifacts.sh
mv .beekeeper.yaml ~/.beekeeper.yaml
mkdir ~/.beekeeper && mv local.yaml ~/.beekeeper/local.yaml
mkdir -p ~/.beekeeper && mv local.yaml ~/.beekeeper/local.yaml
mv bee-1 bee
sudo mv beekeeper /usr/local/bin/beekeeper
- name: Prepare local cluster
run: |
timeout ${TIMEOUT} make beelocal OPTS='ci skip-vet' ACTION=prepare
run: timeout ${TIMEOUT} make beelocal OPTS='ci skip-vet' ACTION=prepare
- name: Set kube config
run: |
mkdir -p ~/.kube
cp /etc/rancher/k3s/k3s.yaml ~/.kube/config
- name: Set local cluster
run: |
timeout ${TIMEOUT} make deploylocal BEEKEEPER_CLUSTER=local-dns
run: timeout ${TIMEOUT} make deploylocal BEEKEEPER_CLUSTER=local-dns
- name: Test pingpong
id: pingpong
run: timeout ${TIMEOUT} bash -c 'until beekeeper check --cluster-name local-dns --checks ci-pingpong; do echo "waiting for pingpong..."; sleep .3; done'
Expand Down Expand Up @@ -182,8 +180,8 @@ jobs:
- name: Test feeds
id: feeds
run: timeout ${TIMEOUT} beekeeper check --cluster-name local-dns --checks=ci-feed
- name: Collect debug artifacts
if: failure()
- name: Collect debug artifacts (local-dns)
if: failure() && (steps.pingpong.outcome == 'failure' || steps.fullconnectivity.outcome == 'failure' || steps.settlements.outcome == 'failure' || steps.pss.outcome == 'failure' || steps.soc.outcome == 'failure' || steps.gsoc.outcome == 'failure' || steps.pushsync-chunks-1.outcome == 'failure' || steps.pushsync-chunks-2.outcome == 'failure' || steps.retrieval.outcome == 'failure' || steps.manifest.outcome == 'failure' || steps.manifest-v1.outcome == 'failure' || steps.postage-stamps.outcome == 'failure' || steps.stake.outcome == 'failure' || steps.withdraw.outcome == 'failure' || steps.redundancy.outcome == 'failure' || steps.act.outcome == 'failure' || steps.feeds.outcome == 'failure' || steps.feeds-v1.outcome == 'failure')
Copy link
Member

@gacevicljubisa gacevicljubisa Feb 20, 2026

Choose a reason for hiding this comment

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

this line are super long, can we just use if: failure()?

run: |
bash .github/bin/beekeeper_artifacts.sh local-dns
export FAILED='no-test'
Expand Down Expand Up @@ -211,10 +209,40 @@ jobs:
echo "sh <(curl -sSf https://lets.tunshell.com/init.sh) L $(echo $KEYS | jq -r .peer2_key) \${TUNSHELL_SECRET} eu.relay.tunshell.com"
curl -sSf https://lets.tunshell.com/init.sh | sh /dev/stdin T $(echo $KEYS | jq -r .peer1_key) ${{ secrets.TUNSHELL_SECRET }} eu.relay.tunshell.com
- uses: actions/upload-artifact@v4
if: failure()
if: failure() && (steps.pingpong.outcome == 'failure' || steps.fullconnectivity.outcome == 'failure' || steps.settlements.outcome == 'failure' || steps.pss.outcome == 'failure' || steps.soc.outcome == 'failure' || steps.gsoc.outcome == 'failure' || steps.pushsync-chunks-1.outcome == 'failure' || steps.pushsync-chunks-2.outcome == 'failure' || steps.retrieval.outcome == 'failure' || steps.manifest.outcome == 'failure' || steps.manifest-v1.outcome == 'failure' || steps.postage-stamps.outcome == 'failure' || steps.stake.outcome == 'failure' || steps.withdraw.outcome == 'failure' || steps.redundancy.outcome == 'failure' || steps.act.outcome == 'failure' || steps.feeds.outcome == 'failure' || steps.feeds-v1.outcome == 'failure')
with:
name: debug-dump
path: dump/
- name: Set local cluster (local-dns-autotls)
run: timeout ${TIMEOUT} make deploylocal BEEKEEPER_CLUSTER=local-dns-autotls
- name: Test pingpong (autotls)
id: pingpong-autotls
run: timeout ${TIMEOUT} bash -c 'until beekeeper check --cluster-name local-dns-autotls --checks ci-pingpong; do echo "waiting for pingpong..."; sleep .3; done'
- name: Test fullconnectivity (autotls)
id: fullconnectivity-autotls
run: timeout ${TIMEOUT} bash -c 'until beekeeper check --cluster-name local-dns-autotls --checks=ci-full-connectivity; do echo "waiting for full connectivity..."; sleep .3; done'
- name: Test retrieval (autotls)
id: retrieval-autotls
run: timeout ${TIMEOUT} beekeeper check --cluster-name local-dns-autotls --checks=ci-retrieval
- name: Test autotls
id: autotls
run: timeout ${TIMEOUT} beekeeper check --cluster-name local-dns-autotls --checks=ci-autotls
- name: Collect debug artifacts (autotls)
if: failure() && (steps.pingpong-autotls.outcome == 'failure' || steps.fullconnectivity-autotls.outcome == 'failure' || steps.retrieval-autotls.outcome == 'failure' || steps.autotls.outcome == 'failure')
run: |
bash .github/bin/beekeeper_artifacts.sh local-dns-autotls
export FAILED='no-test'
if ${{ steps.pingpong-autotls.outcome=='failure' }}; then FAILED=pingpong; fi
if ${{ steps.fullconnectivity-autotls.outcome=='failure' }}; then FAILED=fullconnectivity; fi
if ${{ steps.retrieval-autotls.outcome=='failure' }}; then FAILED=retrieval; fi
if ${{ steps.autotls.outcome=='failure' }}; then FAILED=autotls; fi
curl -sSf -X POST -H "Content-Type: application/json" -d "{\"text\": \"**${RUN_TYPE}** Beekeeper Autotls Error\nBranch: \`${{ github.head_ref }}\`\nUser: @${{ github.event.pull_request.user.login }}\nDebugging artifacts: [click](https://$BUCKET_NAME.$AWS_ENDPOINT/artifacts_$VERTAG.tar.gz)\nStep failed: \`${FAILED}\`\"}" https://beehive.ethswarm.org/hooks/${{ secrets.TUNSHELL_KEY }}
echo "Failed test: ${FAILED}"
- uses: actions/upload-artifact@v4
if: failure() && (steps.pingpong-autotls.outcome == 'failure' || steps.fullconnectivity-autotls.outcome == 'failure' || steps.retrieval-autotls.outcome == 'failure' || steps.autotls.outcome == 'failure')
Copy link
Member

@gacevicljubisa gacevicljubisa Feb 20, 2026

Choose a reason for hiding this comment

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

This as well

Copy link
Contributor Author

@akrem-chabchoub akrem-chabchoub Feb 20, 2026

Choose a reason for hiding this comment

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

Agreed, simplifying to if: failure() is much more readable.
One small tradeoff worth noting: if a local-dns test fails, failure() will also trigger the local-dns-autotls artifact collection and notification, even though autotls tests were just skipped (not failed).
So we could get a misleading alert when the actual issue was in local-dns.

with:
name: debug-dump-autotls
path: dump/
retag:
name: Retag and Trigger ArgoCD
env:
Expand Down
Loading