-
Notifications
You must be signed in to change notification settings - Fork 53
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.5] olm variable source #99
[resolution pt.5] olm variable source #99
Conversation
ee1c20d
to
f228f35
Compare
f228f35
to
86dcc2b
Compare
86dcc2b
to
21f0e5a
Compare
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.
Looks great!
It("should return an errors when they occur", func() { | ||
olmVariableSource := olm.NewOLMVariableSource("prometheus", "packageA") | ||
_, err := olmVariableSource.GetVariables(context.Background(), FailEntitySource{}) | ||
Expect(err).To(HaveOccurred()) | ||
}) |
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.
Does this sufficiently handle the case where an OLMVariableSource is provided that can't be fulfilled by the entity source?
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.
I suppose deppy would just return an error if resolution failed so we're probably good.
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.
Yeah, it should basically throw it's hand up in the air XD
Signed-off-by: perdasilva <perdasilva@redhat.com>
21f0e5a
to
44f0bd3
Compare
Signed-off-by: perdasilva <perdasilva@redhat.com>
44f0bd3
to
958d435
Compare
This PR adds the
OLMVariableSource
which orchestrates variable creation by using the other variables sources (e.g.RequiredPackage
,BundlesAndDeps
, andGlobalConstraints
)