Skip to content

Commit

Permalink
deploy chart on worker updates
Browse files Browse the repository at this point in the history
  • Loading branch information
waveywaves committed Mar 29, 2024
1 parent 794b161 commit 3335d6c
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 5 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/build-operator.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ jobs:
image:
repository: registry.uffizzi.com/${{ env.UUID_OPERATOR }}
tag: 48h
concurrent: 5
concurrent: 1
EOF
cat helm-values.yaml # For debugging, to check the contents of the file.
- name: Upload Helm Values as Artifact
Expand Down
9 changes: 6 additions & 3 deletions .github/workflows/e2e-perf.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -106,15 +106,18 @@ jobs:
- name: Time taken to create UffizziClusters with different numbers of workers
run: |
n_simultaneous_clusters=3
for n_workers in $(seq 1 5 31); do
# update concurrent workers
yq -i '.concurrent = "$n_workers"' helm-values.yaml
n_simultaneous_clusters=3
# upgrade helm chart
helm upgrade --install --wait pr-${{ github.event.pull_request.number }} \
./chart -f helm-values.yaml
# time taken to create n clusters simultaneously
time=$(bash hack/e2e/perf/01-multicluster.sh $n_simultaneous_clusters)
bash $E2E_UTILS update_json_with_workers_and_time $n_workers $time $PERF_DATA_FILE
done
cat $PERF_DATA_FILE
Expand Down
2 changes: 1 addition & 1 deletion hack/e2e/perf/viz/generate_graph.py
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@

# Add title, labels, grid, and legend
plt.title('Performance Comparison: PR vs Main Branch')
plt.xlabel('Number of Workers')
plt.xlabel('Number of Workers (n_simultaneous_clusters=3')
plt.ylabel('Time Taken (seconds)')
plt.grid(True)
plt.legend()
Expand Down

0 comments on commit 3335d6c

Please sign in to comment.