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

[resolution pt.3] bundles and deps variable source #97

Merged

Conversation

perdasilva
Copy link
Contributor

@perdasilva perdasilva commented Jan 24, 2023

This PR adds the Bundles and Dependencies variable source. It generate BundleVariables for all RequiredPackage variables found from executing the inputVariableSources. It does so by inspecting the BundleEntities included in each RequiredPackageVariable, resolving dependencies, and outputting BundleVariables for all BundleEntities and their resolved dependencies.

@perdasilva perdasilva force-pushed the resolution_pt3_bundles_and_deps branch from 740c9f8 to b6642a2 Compare January 24, 2023 12:17
@perdasilva perdasilva changed the title [resolution Pt3] bundles and deps variable source [resolution pt.3] - bundles and deps variable source Jan 24, 2023
@perdasilva perdasilva changed the title [resolution pt.3] - bundles and deps variable source [resolution pt.3] bundles and deps variable source Jan 24, 2023
@perdasilva perdasilva force-pushed the resolution_pt3_bundles_and_deps branch from b6642a2 to cf324af Compare January 25, 2023 11:04
Signed-off-by: perdasilva <perdasilva@redhat.com>
@perdasilva perdasilva force-pushed the resolution_pt3_bundles_and_deps branch from cf324af to 2eebbfa Compare January 25, 2023 11:06
Copy link
Member

@awgreene awgreene left a comment

Choose a reason for hiding this comment

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

Looks good!

@perdasilva perdasilva merged commit 1991c57 into operator-framework:main Jan 25, 2023
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.

None yet

2 participants