Skip to content
This repository was archived by the owner on Nov 24, 2025. It is now read-only.

Dev CiaB: Use Almquist shell for TP/TPv2 since the base image is Alpine#7188

Merged
ocket8888 merged 1 commit intoapache:masterfrom
zrhoffman:tp-sh
Nov 14, 2022
Merged

Dev CiaB: Use Almquist shell for TP/TPv2 since the base image is Alpine#7188
ocket8888 merged 1 commit intoapache:masterfrom
zrhoffman:tp-sh

Conversation

@zrhoffman
Copy link
Copy Markdown
Member

This PR fixes an issue introduced by merging #7142, which changes the run.sh shell for the trafficportal and tpv2 Dev CDN-in-a-Box services to bash because dash (sh on Debian) does not have [[ ]] expressions, after #7169, which changes the Linux distro used for the trafficportal and tpv2 services from Debian to Alpine.


Which Traffic Control components are affected by this PR?

  • CDN in a Box for Developers

What is the best way to verify this PR?

Verify that Traffic Portal and Traffic Portal v2 start in CDN in a Box for Developers, rather than them failing with

/bin/sh: /go/src/github.com/apache/trafficcontrol/dev/tpv2/run.sh: not found

(which it says if /bin/bash is not found)

PR submission checklist

@zrhoffman zrhoffman added regression bug a bug in existing functionality introduced by a new version dev-ciab Related to CDN in a Box for Developers labels Nov 14, 2022
@ocket8888 ocket8888 self-assigned this Nov 14, 2022
@ocket8888 ocket8888 merged commit 501d86e into apache:master Nov 14, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

dev-ciab Related to CDN in a Box for Developers regression bug a bug in existing functionality introduced by a new version

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants