Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Distribution packaging continued #135

Merged
merged 12 commits into from
Jul 31, 2024
Merged

Distribution packaging continued #135

merged 12 commits into from
Jul 31, 2024

Conversation

Mpdreamz
Copy link
Member

@Mpdreamz Mpdreamz commented Jul 31, 2024

Continues #133

This now puts all required distributions under .artifacts/elastic-distribution.

This includes elastic-*.zip versions of the auto instrumentation zips.
These zips include
* our plugin dll (twice for -windows.zip).
* _instrument.sh a copy of the original instrument.sh
* instrument.sh which sets our plugin var and calls _instrument.sh

This also includes elastic versions of the installation bash script and the powershell module to instrument and install/update on windows.

These files should now upload as part of our release and uploaded as artifacts on each commit in main.

Locally to build the distributables call:

./build.sh redistribute

To validate the distribution there is now a new dockerfile that validates our install and instrumentation scripts:

docker build -t distribution.autoinstrumentation:latest -f examples/Example.AutoInstrumentation/distribution.Dockerfile  . && \
         docker run -it --rm -p 5000:8080 --name distri --platform linux/arm64 distribution.autoinstrumentation:latest

The latter will also form the basis for more integration tests which i'll follow up with

@Mpdreamz Mpdreamz merged commit 28ec302 into main Jul 31, 2024
5 checks passed
@Mpdreamz Mpdreamz deleted the feature/packaging-continued branch July 31, 2024 18:37
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