Skip to content

docs(ci-cd): use enterprise install script for cloud replay#881

Merged
pathakharshit merged 1 commit into
mainfrom
docs/add-enterprise-install-script
Jun 26, 2026
Merged

docs(ci-cd): use enterprise install script for cloud replay#881
pathakharshit merged 1 commit into
mainfrom
docs/add-enterprise-install-script

Conversation

@pathakharshit

Copy link
Copy Markdown
Contributor

Replace the manual binary download (curl+chmod+mv) with the enterprise install script across all three CI/CD docs (github.md, gitlab.md, jenkins.md). Also updates the callout note in each file to reference the install script URL.

Replace the manual binary download (curl+chmod+mv) with the
enterprise install script across all three CI/CD docs.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Signed-off-by: Harshit Pathak <harshit07pathak@gmail.com>

@Aditya-eddy Aditya-eddy left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@pathakharshit pathakharshit merged commit 11febac into main Jun 26, 2026
7 checks passed
@pathakharshit pathakharshit deleted the docs/add-enterprise-install-script branch June 26, 2026 12:18
@amaan-bhati

Copy link
Copy Markdown
Member

Hey @pathakharshit there is one thing that i wanted to point out but the pr got merged before i could do so, hence can you kindly resolve this in a different pr:

The install command works fine in GitHub Actions and GitLab CI since both default to bash, but the Jenkins example will break.

sh '''...''' in a Declarative Pipeline runs /bin/sh, which is dash on Ubuntu - and dash does not have source. Running this as-is on Jenkins would give you source: not found.

Fix: replace source with . (the POSIX equivalent), which works in both sh and bash:

curl --silent -O -L https://keploy.io/ent/install.sh && . install.sh

This is the only thing to fix - the GitHub Actions and GitLab CI snippets are fine.

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.

3 participants