Closed cisaacstern closed 1 year ago
The test Dataflow jobs deployed from this PR were correctly assigned max_num_workers=50
(based on the label here).
but I now realize that this label will not be discoverable (with the current implementation from #37) on push
events... 🤔
Ok! The rewritten workflow logic successfully deployed the test jobs to dataflow with max_num_workers=50
. I believe this logic should now work for push
events as well, so going to merge this and hope we get the desired result (production runs of both recipes with only 50 workers each).
See discussion in #36.