Skip to content

CI: fix docker-cpi deploy#509

Merged
aramprice merged 3 commits into
developfrom
fix-docker-job
May 22, 2026
Merged

CI: fix docker-cpi deploy#509
aramprice merged 3 commits into
developfrom
fix-docker-job

Conversation

@aramprice
Copy link
Copy Markdown
Member

No description provided.

Copilot AI review requested due to automatic review settings May 22, 2026 17:51
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 22, 2026

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 68c76165-e347-4bc4-b308-1a21d5c500c6

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • ✅ Review completed - (🔄 Check again to review again)
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix-docker-job

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the Docker CPI CI task script to be more robust and to deploy using paths that are stable relative to the Concourse task working directory.

Changes:

  • Switch to bash with set -euo pipefail, plus optional DEBUG tracing.
  • Update the director environment sourcing path to /tmp/local-bosh/director/bosh-env.
  • Use ${PWD}/bosh-deployment-anchored paths for the zookeeper deploy manifest and improve log timestamp substitution.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread ci/tasks/test-main-docker-cpi.sh Outdated
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 5 out of 5 changed files in this pull request and generated no new comments.

@aramprice aramprice merged commit fb91d0c into develop May 22, 2026
3 checks passed
@aramprice aramprice deleted the fix-docker-job branch May 22, 2026 18:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants