Skip to content

Commit 5b0df68

Browse files
Merge pull request #32 from EightfoldAI/feature/add-needs-local-processor-support
Add e2e_needs_sandbox labeler support for local processor sandbox
2 parents cad141f + 9f77fd6 commit 5b0df68

1 file changed

Lines changed: 16 additions & 17 deletions

File tree

entrypoint.sh

Lines changed: 16 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
# Varun Chopra <vchopra@eightfold.ai>
33
#
44
# This action runs every time a comment is added to a pull request.
5-
# Accepts the following commands: shipit, needs_ci, needs_sandbox
5+
# Accepts the following commands: shipit, needs_ci, needs_sandbox, e2e_needs_sandbox
66

77
set -e
88

@@ -198,14 +198,13 @@ if [[ $comment_body == "needs_ci${alt_python_version}:lite" ]]; then
198198
fi
199199
fi
200200

201-
# Add sandbox is needs_sandbox
202-
# Remove stop_sandbox if sandbox is requested again
201+
# Add sandbox label for needs_sandbox and e2e_needs_sandbox (e2e_ prefix enables local processor)
203202
already_needs_sandbox=false
204203
already_needs_alternate_version_sandbox=false
205204
alternate_python_version="3.13"
206205
current_python_version="3.13"
207206

208-
if [[ $comment_body =~ ^needs_sandbox(:${alternate_python_version})(:(eu|gov|ca|uae|wu|ap))?(:(dev|([0-9]+)(\.([0-9]+)?)?))?(:(([a-zA-Z0-9,]+)))?([ \t]*)?$ ]]; then
207+
if [[ $comment_body =~ ^(e2e_)?needs_sandbox(:${alternate_python_version})(:(eu|gov|ca|uae|wu|ap))?(:(dev|([0-9]+)(\.([0-9]+)?)?))?(:(([a-zA-Z0-9,]+)))?([ \t]*)?$ ]]; then
209208
for label in $labels; do
210209
case $label in
211210
sandbox:${alternate_python_version})
@@ -235,24 +234,24 @@ if [[ $comment_body =~ ^needs_sandbox(:${alternate_python_version})(:(eu|gov|ca|
235234
esac
236235
done
237236
if [[ "$already_needs_sandbox" == false && "$current_python_version" != "$alternate_python_version" ]]; then
238-
if [[ $comment_body =~ needs_sandbox:${alternate_python_version}:eu ]]; then
237+
if [[ $comment_body =~ (e2e_)?needs_sandbox:${alternate_python_version}:eu ]]; then
239238
add_label "sandbox:${alternate_python_version} :eu:"
240-
elif [[ $comment_body =~ needs_sandbox:${alternate_python_version}:ca ]]; then
239+
elif [[ $comment_body =~ (e2e_)?needs_sandbox:${alternate_python_version}:ca ]]; then
241240
add_label "sandbox:${alternate_python_version} :maple_leaf:"
242-
elif [[ $comment_body =~ needs_sandbox:${alternate_python_version}:gov ]]; then
241+
elif [[ $comment_body =~ (e2e_)?needs_sandbox:${alternate_python_version}:gov ]]; then
243242
add_label "sandbox:${alternate_python_version} :classical_building:"
244-
elif [[ $comment_body =~ needs_sandbox:${alternate_python_version}:uae ]]; then
243+
elif [[ $comment_body =~ (e2e_)?needs_sandbox:${alternate_python_version}:uae ]]; then
245244
add_label "sandbox:${alternate_python_version} :united_arab_emirates:"
246-
elif [[ $comment_body =~ needs_sandbox:${alternate_python_version}:wu ]]; then
245+
elif [[ $comment_body =~ (e2e_)?needs_sandbox:${alternate_python_version}:wu ]]; then
247246
add_label "sandbox:${alternate_python_version} :us:"
248-
elif [[ $comment_body =~ needs_sandbox:${alternate_python_version}:ap ]]; then
247+
elif [[ $comment_body =~ (e2e_)?needs_sandbox:${alternate_python_version}:ap ]]; then
249248
add_label "sandbox:${alternate_python_version} :australia:"
250249
else
251250
add_label "sandbox:${alternate_python_version}"
252251
fi
253252
fi
254253

255-
elif [[ $comment_body =~ ^needs_sandbox(:(eu|gov|ca|uae|wu|ap))?(:(dev|([0-9]+)(\.([0-9]+)?)?))?(:(([a-zA-Z0-9,]+)))?([ \t]*)?$ ]]; then
254+
elif [[ $comment_body =~ ^(e2e_)?needs_sandbox(:(eu|gov|ca|uae|wu|ap))?(:(dev|([0-9]+)(\.([0-9]+)?)?))?(:(([a-zA-Z0-9,]+)))?([ \t]*)?$ ]]; then
256255
for label in $labels; do
257256
case $label in
258257
sandbox)
@@ -282,17 +281,17 @@ elif [[ $comment_body =~ ^needs_sandbox(:(eu|gov|ca|uae|wu|ap))?(:(dev|([0-9]+)(
282281
esac
283282
done
284283
if [[ "$already_needs_sandbox" == false ]]; then
285-
if [[ $comment_body =~ needs_sandbox:eu ]]; then
284+
if [[ $comment_body =~ (e2e_)?needs_sandbox:eu ]]; then
286285
add_label "sandbox :eu:"
287-
elif [[ $comment_body =~ needs_sandbox:ca ]]; then
286+
elif [[ $comment_body =~ (e2e_)?needs_sandbox:ca ]]; then
288287
add_label "sandbox :maple_leaf:"
289-
elif [[ $comment_body =~ needs_sandbox:gov ]]; then
288+
elif [[ $comment_body =~ (e2e_)?needs_sandbox:gov ]]; then
290289
add_label "sandbox :classical_building:"
291-
elif [[ $comment_body =~ needs_sandbox:uae ]]; then
290+
elif [[ $comment_body =~ (e2e_)?needs_sandbox:uae ]]; then
292291
add_label "sandbox :united_arab_emirates:"
293-
elif [[ $comment_body =~ needs_sandbox:wu ]]; then
292+
elif [[ $comment_body =~ (e2e_)?needs_sandbox:wu ]]; then
294293
add_label "sandbox :us:"
295-
elif [[ $comment_body =~ needs_sandbox:ap ]]; then
294+
elif [[ $comment_body =~ (e2e_)?needs_sandbox:ap ]]; then
296295
add_label "sandbox :australia:"
297296
else
298297
add_label "sandbox"

0 commit comments

Comments
 (0)