Dev e2e ci cd#425
Open
vipbhardwaj wants to merge 5 commits into
Open
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Adds a Jenkins CI/CD job to run the tests/dev_e2e suite against a locally provisioned backend (CBS in Docker + SGW built from a specified ref), and updates local environment tooling to support starting/stopping a local CBL-C test server for that workflow.
Changes:
- Add a new
jenkins/pipelines/dev_e2e/sgw/pipeline (Jenkinsfile + test + teardown scripts) to build SGW from a ref and rundev_e2eagainst local CBS. - Update
environment/local/start_local.pyto always build the local CBL-C test server from source and add a--stopmode for teardown. - Remove the tracked placeholder
servers/c/lib/README.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| servers/c/lib/README | Removes the last tracked file under servers/c/lib (directory placeholder). |
| jenkins/pipelines/dev_e2e/sgw/test.sh | New script to start CBS container, build/start SGW, start local CBL-C server, and run dev_e2e. |
| jenkins/pipelines/dev_e2e/sgw/teardown.sh | New teardown script to collect artifacts and stop SGW/CBL-C/CBS resources. |
| jenkins/pipelines/dev_e2e/sgw/Jenkinsfile | New Jenkins pipeline wiring parameters, timeouts, teardown, and artifact archiving. |
| environment/local/start_local.py | Adds --stop and changes local CBL-C test server setup to always build from source. |
Comments suppressed due to low confidence (1)
servers/c/lib/README:1
- Removing this file causes
servers/c/lib/to disappear from a clean checkout, but several C testserver build scripts assume${LIB_DIR}exists (e.g.servers/c/scripts/download_cbl.shdoescp -r ... "${LIB_DIR}/libcblite"without creating${LIB_DIR}). This can break C test server builds/downloads. Keep a tracked placeholder (README/.gitkeep) or update the scripts tomkdir -p "${LIB_DIR}"before copying.
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 7 out of 7 changed files in this pull request and generated 3 comments.
Comments suppressed due to low confidence (1)
servers/c/lib/README:1
- Removing the last tracked file under
servers/c/lib/will cause Git to drop the empty directory, but the C test server build/download scripts expect${SCRIPT_DIR}/../libto exist (e.g.,servers/c/scripts/download_cbl.shcopies into${LIB_DIR}/libcblitewithout creating${LIB_DIR}). This makes fresh checkouts fail when building C.
Keep a tracked placeholder (e.g. restore this README / add a .gitkeep) or update the scripts to mkdir -p "$LIB_DIR" before copying.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.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.
CBG-5439