Alioss Integration Tests Migration to Github Worfklow #9
Conversation
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
- scripts to run, teardown and setup are added.
cf-foundation-community-automation
bot
moved this to Inbox
in Foundational Infrastructure Working Group
beyhan
requested review from
a team,
evanfarrar and
gururajsh
and removed request for
a team
beyhan
moved this from Inbox
to Pending Review | Discussion
in Foundational Infrastructure Working Group
There was a problem hiding this comment.
Pull Request Overview
This PR migrates the Alioss integration tests from a Concourse pipeline to a GitHub Workflow, consolidating CI/CD operations within the storage-cli repository. The implementation introduces helper scripts that dynamically provision and clean up Alibaba OSS buckets during test execution.
Key Changes:
- Added GitHub Workflow (
.github/workflows/alioss-integration.yml) that orchestrates integration tests with dynamic bucket provisioning - Created helper scripts (
setup.sh,run-int.sh,teardown.sh) to manage test environment lifecycle - Updated documentation in
alioss/README.mdto reflect new testing procedures
Reviewed Changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 11 comments.
Show a summary per file
| File | Description |
|---|---|
.github/workflows/alioss-integration.yml |
New GitHub Actions workflow for running Alioss integration tests on pull requests |
.github/scripts/alioss/utils.sh |
Utility functions for bucket management, Aliyun CLI configuration, and file-based bucket name tracking |
.github/scripts/alioss/setup.sh |
Script to initialize test environment by creating dynamically named OSS buckets |
.github/scripts/alioss/run-int.sh |
Script to execute integration tests with proper environment variable configuration |
.github/scripts/alioss/teardown.sh |
Cleanup script to delete test buckets after test execution |
alioss/README.md |
Updated documentation with separate unit and integration test instructions, including manual setup guide |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
fmoehler
previously approved these changes
Nov 21, 2025
github-project-automation
bot
moved this from Pending Review | Discussion
to Pending Merge | Prioritized
in Foundational Infrastructure Working Group
stephanme
requested changes
Nov 21, 2025
github-project-automation
bot
moved this from Pending Merge | Prioritized
to Waiting for Changes | Open for Contribution
in Foundational Infrastructure Working Group
stephanme
approved these changes
Nov 21, 2025
github-project-automation
bot
moved this from Waiting for Changes | Open for Contribution
to Pending Merge | Prioritized
in Foundational Infrastructure Working Group
github-project-automation
bot
moved this from Pending Merge | Prioritized
to Done
in Foundational Infrastructure Working Group
35 tasks
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
Context
The integration tests previously ran in a Concourse pipeline. As part of consolidating all clis into storage-cli repository, the Concourse pipelines needed to be fully migrated to GitHub Workflows.
Solution
A new GitHub Workflow has been introduced. The workflow includes one job, alioss-general-integration-tests. Helper scripts like: setup, run, teardown created. With that each job can create the required Ali oss bucket dynamically during execution and cleans them up afterward.
All secrets needed for provisioning these resources are stored under the secrets in GitHub.