Repository navigation
feat: migrate FlagGems actions to FlagOps for org-wide sharing - #15
Merged
Merged
Conversation
Migrates 5 composite actions from flagos-ai/FlagGems/.github/actions/ into flagos-ai/FlagOps/actions/ for organization-level sharing and generalization. Actions added: - actions/checkout-retry: wraps actions/checkout with up to 3 retries and 30s sleep between attempts; direct port from FlagGems - actions/checkout-pr: installs gh CLI and checks out a PR branch by number; adds gh-version and gh-mirror-url parameters so the download source can be overridden from the hardcoded internal CDN - actions/robust-checkout: SSH→HTTPS rotation with hard-kill probes and throughput-based transfer judging; direct port from FlagGems - actions/setup-project: generalized from setup-flaggems; setup-script, vllm-version, vllm-backend-prefix, vllm-safe-deps-script, layer1-packages-pattern, and gpu-check-script are all parameterized - actions/run-ondemand-test: generalized from ondemand-test; FlagGems- specific op/backend/runner/existing inputs replaced by a single test-command string; artifact-name, artifact-path, and comment-message-id are fully parameterized README updated with: - full action catalog organized by category (checkout, setup, test, report, notify) - decision guide for choosing the right checkout action - containerized-runner usage note (must use fully-qualified refs) Co-Authored-By: Claude Code <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Migrates 5 composite actions from
flagos-ai/FlagGems/.github/actions/toflagos-ai/FlagOps/actions/for organization-level sharing and generalization. Adds READMEs for each action and updates the main README.Actions added
checkout-retrycheckout-prgh-version+gh-mirror-urlparametersrobust-checkoutsetup-projectsetup-flaggemssetup-script,vllm-version,vllm-backend-prefix,layer1-packages-pattern,gpu-check-scriptparameterizedrun-ondemand-testondemand-testtest-command+artifact-name+comment-message-idAnalysis & optimization notes
Issues found in FlagGems source
checkout-retrypinsactions/checkout@de0fac2e...# v6.0.2— there is no publicactions/checkoutv6; this SHA should be verified or updated to a known stable release. Carried over as-is pending verification.setup-flaggemshardcodesVLLM_VERSION=0.21.0— moved to thevllm-versioninput so callers can upgrade without touching the shared action.checkout-prhardcodes the internal CDN URL — now agh-mirror-urlparameter with the CDN as default; consumers outside the internal network can override it.ondemand-testembeds FlagGems tool paths (tools/run_tests.py,tools/psum_text) — moved into the caller'stest-command, making the action project-agnostic.setup-flaggemsuses snake_case inputgpu_check_script— renamed togpu-check-script(kebab-case) to match the rest of the org's convention.Org-level sharing assessment
setup.sh <backend>convention. The vllm layer-integrity logic is opt-in.Next steps
A follow-up PR to
flagos-ai/FlagGemswill updatecommand.yamlandbackend-test.yamlto reference these shared actions instead of the local copies.