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

[Epic] Have all manifests in example YAML be able to go through Uptest #22

Open
muvaf opened this issue Aug 15, 2022 · 0 comments
Open
Labels
enhancement New feature or request

Comments

@muvaf
Copy link
Contributor

muvaf commented Aug 15, 2022

What problem are you facing?

We have examples manifests in each provider but not all of them can work in Uptest.

How could Official Providers help solve your problem?

There are roughly the following classes of resources:

  • Examples of inherited resources from Jet providers.
    • These were assumed to be working since we had merged them with manual testing but that's not the case for all of them.
  • Examples we merged after a report of manual testing.
    • We manually tested and reported the results in the PR body before Uptest was ready to be used for all scenarios.
  • Examples that require manual intervention, like file upload, but not marked as such.

Ideally, we should make all manifests be able to go through Uptest without errors so that we can run the pipeline whenever we need to, like TF provide upgrade, and not get false negatives.

In order to reach that, we should do a pass over them to fix bugs and mark the ones that require manual intervention with uptest.upbound.io/manual-intervention annotation to tell users what they need to do and to instruct Uptest to skip that specific resource during testing. Later, we can improve Uptest to take those actions and remove the annotations over time. But first, we need to get to a place where all examples can be run with Uptest without error.

This is an epic issue, related current issues are:

More issues will be needed for the rest of the resources until /test-examples="provider-{aws,gcp,azure}/examples" works.

@muvaf muvaf added the enhancement New feature or request label Aug 15, 2022
@luebken luebken changed the title Have all manifests in example YAML be able to go through Uptest [Epic] Have all manifests in example YAML be able to go through Uptest Aug 15, 2022
@jeanduplessis jeanduplessis transferred this issue from another repository Oct 11, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant