-
Notifications
You must be signed in to change notification settings - Fork 6
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
Replace negz/provider-nop with crossplane/provider-nop #23
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Rahul Grover <[email protected]>
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.
All the tests pass successfully except TestConfiguration. That might require updating the configuration image being used for the test.
Could you elaborate on what is failing? Maybe paste the test failure in a comment.
prv := &pkgv1.Provider{ | ||
ObjectMeta: metav1.ObjectMeta{Name: internal.SuiteName}, | ||
Spec: pkgv1.ProviderSpec{ | ||
PackageSpec: pkgv1.PackageSpec{ | ||
Package: "negz/provider-nop:v0.1.0", | ||
Package: "crossplane/provider-nop:main", |
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.
Should we cut a release of provider-nop we can pin to? I can take care of doing that, since I believe it will require adding some secrets to the provider-nop GitHub repo.
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.
That will be great. Thanks!
In the which resulted in the following failure:
|
Signed-off-by: Rahul Grover [email protected]
Description of your changes
Since provider-nop is ready to use, it might be a good idea to use the same for conformance tests. I have replaced occurrences of
negz/provider-nop
withcrossplane/provider-nop
and added essential spec required for the same.I have:
make reviewable test
to ensure this PR is ready for review.How has this code been tested
I've run the test on a local kind cluster running Crossplane 1.2. All the tests pass successfully except
TestConfiguration
. That might require updating the configuration image being used for the test.