-
Notifications
You must be signed in to change notification settings - Fork 137
Support building integration packages with required input dependencies #3459
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
Draft
teresaromero
wants to merge
28
commits into
elastic:main
Choose a base branch
from
teresaromero:feat/3277-composable-input-bundling
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Changes from all commits
Commits
Show all changes
28 commits
Select commit
Hold shift + click to select a range
b5c957a
feat: extend package manifest for required input dependencies
teresaromero d9f3026
feat: add registry client package download with TLS
teresaromero aa4c8f9
feat: optional PGP verification for downloaded package zips
teresaromero 84efe17
feat: bundle required input packages at build time
teresaromero 9d71a69
feat: wire required input resolver into build and EPR-aware commands
teresaromero 18a010c
test: add manual package fixtures for composable required inputs
teresaromero 1f81dbe
docs: composable integrations, dependencies, and local registry
teresaromero 0dd8203
fix(requiredinputs): satisfy golangci-lint
teresaromero f788b32
test(archetype): remove unnecessary resolver mock
teresaromero f3745fe
refactor(requiredinputs): extract processDataStreamManifest sub-function
teresaromero ce1685e
refactor(requiredinputs): extract shared collectAndCopyPolicyTemplate…
teresaromero 06e7072
Merge branch 'main' of github.com:elastic/elastic-package into feat/3…
teresaromero 575a756
Merge branch 'main' of github.com:elastic/elastic-package into feat/3…
teresaromero 4a6e784
Add composable CI test packages and two-phase zip build
teresaromero 34f2f17
Fix composable CI package manifests for spec and Fleet
teresaromero a36572e
Skip composable integration in test-build-zip
teresaromero dc05353
fix(registry): propagate HTTP client errors and harden package download
teresaromero b6ff9cb
build: resolve EPR base URL from profile like other commands
teresaromero 90d857b
refactor(requiredinputs): split merge/resolve helpers and extend tests
teresaromero 206316b
ci: install yq for build-install-zip-file integration targets
teresaromero e7b6f0a
scripts: isolate stack env to USE_SHELLINIT branch
teresaromero d88455e
fix(tests): update error handling for package verification
teresaromero f05fcbc
feat(ci): add composable integration tests and update build scripts
teresaromero 16dc21d
refactor(tests): simplify cleanup logic in package download tests
teresaromero 56443be
revert(tests): remove composable integration targets from build scripts
teresaromero bdcc821
refactor(requiredinputs): migrate from gopkg.in/yaml.v3 to goccy/go-yaml
teresaromero f6af43c
chore(requiredinputs): remove unnecessary blank line in streamdefs.go
teresaromero 4a7129e
refactor(verification): remove package signature verification logic
teresaromero File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
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
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
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
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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jsoriano with the use of profile at the build command, we are coupling the stack with build. the command itself does not have the flag to select profile.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need it to support the local package registry, right?
What it is true is that it should remain working even if there is no stack started, this should be optional.