Skip to content

Commit 4039509

Browse files
committed
collect metrics on existing workflow logic.
1 parent 37c46c2 commit 4039509

File tree

5 files changed

+759
-495
lines changed

5 files changed

+759
-495
lines changed

.github/workflows/functional-test-cloud.yaml

Lines changed: 42 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -299,15 +299,40 @@ jobs:
299299
number: ${{ env.PR_NUMBER }}
300300
append: true
301301
message: |
302-
:hourglass: Building Radius and pushing container images for functional tests...
302+
:hourglass: Building Radius images for functional tests...
303303
304-
- name: Build and Push container images
304+
- name: Build and package images (no push)
305305
run: |
306-
make build && make docker-build && make docker-push
306+
make artifacts-no-docker
307+
make docker-build DOCKER_CACHE_GHA=1
308+
make docker-save-images
307309
env:
308310
DOCKER_REGISTRY: ${{ env.CONTAINER_REGISTRY }}
309311
DOCKER_TAG_VERSION: ${{ env.REL_VERSION }}
310312

313+
- name: Collect build metrics
314+
run: |
315+
make build-metrics METRICS_COLLECT_ONLY=1
316+
echo '## Build metrics (cloud)' >> $GITHUB_STEP_SUMMARY
317+
cat dist/metrics/metrics.txt >> $GITHUB_STEP_SUMMARY
318+
319+
- name: Upload build metrics (cloud)
320+
if: always()
321+
uses: actions/upload-artifact@v4
322+
with:
323+
name: build-metrics-cloud-${{ github.run_id }}-${{ github.run_attempt }}
324+
path: dist/metrics/*
325+
retention-days: 14
326+
if-no-files-found: warn
327+
328+
- name: Upload built images
329+
uses: actions/upload-artifact@v4
330+
with:
331+
name: radius_images
332+
path: |
333+
./dist/images/*.tar
334+
if-no-files-found: error
335+
311336
- name: Upload CLI binary
312337
uses: actions/upload-artifact@v4
313338
with:
@@ -601,12 +626,19 @@ jobs:
601626
helm repo add azure-workload-identity https://azure.github.io/azure-workload-identity/charts
602627
helm install workload-identity-webhook azure-workload-identity/workload-identity-webhook --namespace radius-default --create-namespace --version ${{ env.AZURE_WORKLOAD_IDENTITY_WEBHOOK_VER }} --set azureTenantID=${{ secrets.AZURE_SP_TESTS_TENANTID }}
603628
604-
- name: Login to GitHub Container Registry
605-
uses: docker/login-action@v3
629+
- name: Download built images
630+
uses: actions/download-artifact@v5
606631
with:
607-
registry: ghcr.io
608-
username: ${{ github.actor }}
609-
password: ${{ secrets.GITHUB_TOKEN }}
632+
name: radius_images
633+
path: dist/images
634+
635+
- name: Load images into KinD nodes
636+
run: |
637+
for f in dist/images/*.tar; do
638+
[ -e "$f" ] || continue
639+
echo "Loading $f to kind..."
640+
./kind load image-archive "$f" --name radius
641+
done
610642
611643
- name: Download Bicep
612644
run: |
@@ -638,12 +670,9 @@ jobs:
638670
echo "*** Installing Radius to Kubernetes ***"
639671
rad install kubernetes \
640672
--chart ${{ env.RADIUS_CHART_LOCATION }} \
641-
--set rp.image=${{ env.CONTAINER_REGISTRY }}/applications-rp,rp.tag=${{ env.REL_VERSION }} \
642-
--set dynamicrp.image=${{ env.CONTAINER_REGISTRY }}/dynamic-rp,dynamicrp.tag=${{ env.REL_VERSION }} \
643-
--set controller.image=${{ env.CONTAINER_REGISTRY }}/controller,controller.tag=${{ env.REL_VERSION }} \
644-
--set ucp.image=${{ env.CONTAINER_REGISTRY }}/ucpd,ucp.tag=${{ env.REL_VERSION }} \
673+
--set global.imageRegistry=${{ env.CONTAINER_REGISTRY }} \
674+
--set global.imageTag=${{ env.REL_VERSION }} \
645675
--set de.image=${{ env.DE_IMAGE }},de.tag=${{ env.DE_TAG }} \
646-
--set bicep.image=${{ env.CONTAINER_REGISTRY }}/bicep,bicep.tag=${{ env.REL_VERSION }} \
647676
--set global.azureWorkloadIdentity.enabled=true \
648677
--set global.aws.irsa.enabled=true
649678

0 commit comments

Comments
 (0)