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

Grow the Generic Embedded Config container to fit large config #3209

Merged
merged 1 commit into from
Jan 15, 2024

Conversation

scudette
Copy link
Contributor

With very large artifacts, the embedded config space reserved in the binary may be exceeded. In this case it should still be possible to use the Generic Collector container to hold large embedded artifact definitions.

This PR allows larger configs to be embedded in the GenericCollector container.

With very large artifacts, the embedded config space reserved in the
binary may be exceeded. In this case it should still be possible to
use the Generic Collector container to hold large embedded artifact
definitions.

This PR allows larger configs to be embedded in the GenericCollector
container.
@scudette scudette merged commit 9d69b4e into master Jan 15, 2024
3 checks passed
@scudette scudette deleted the generic branch January 15, 2024 05:20
scudette added a commit that referenced this pull request Feb 15, 2024
With very large artifacts, the embedded config space reserved in the
binary may be exceeded. In this case it should still be possible to use
the Generic Collector container to hold large embedded artifact
definitions.

This PR allows larger configs to be embedded in the GenericCollector
container.
scudette added a commit that referenced this pull request Feb 15, 2024
With very large artifacts, the embedded config space reserved in the
binary may be exceeded. In this case it should still be possible to use
the Generic Collector container to hold large embedded artifact
definitions.

This PR allows larger configs to be embedded in the GenericCollector
container.
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.

1 participant