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

Fixes and Tweaks with Uptest related targets #143

Merged
merged 3 commits into from
Nov 14, 2022

Conversation

turkenh
Copy link
Contributor

@turkenh turkenh commented Nov 14, 2022

Description of your changes

This PR fixes couple of issues with local dev flow with uptest:

  • Configures Test as default condition
  • Splits local deployment as a separate target so that make uptest could be called when provider is running externally with make run (i.e. previous workflow with make uptest-local)
  • updates uptest version to latest to get a fix with data source parsing

I have:

  • Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

UPTEST_DATASOURCE_PATH=.work/uptest-datasource.yaml UPTEST_EXAMPLE_LIST=examples/codecommit/approvalruletemplateassociation.yaml make e2e

Also

# on terminal A
make run

# on terminal B
# get secret and provider config configured beforehand
UPTEST_EXAMPLE_LIST=examples/iam/user.yaml make uptest

Signed-off-by: Hasan Turken <turkenh@gmail.com>
@turkenh turkenh changed the title Fixes with uptest local dev Fixes and Tweaks with Uptest local dev Nov 14, 2022
@turkenh turkenh changed the title Fixes and Tweaks with Uptest local dev Fixes and Tweaks with Uptest related targets Nov 14, 2022
@pull-request-size pull-request-size bot added size/M and removed size/S labels Nov 14, 2022
@turkenh turkenh marked this pull request as ready for review November 14, 2022 09:20
Signed-off-by: Hasan Turken <turkenh@gmail.com>
@turkenh
Copy link
Contributor Author

turkenh commented Nov 14, 2022

/test-examples="examples/codecommit/approvalruletemplateassociation.yaml"

@turkenh turkenh removed the request for review from ulucinar November 14, 2022 09:50
@turkenh
Copy link
Contributor Author

turkenh commented Nov 14, 2022

/test-examples="examples/codecommit/approvalruletemplateassociation.yaml"

Signed-off-by: Hasan Turken <turkenh@gmail.com>
@turkenh
Copy link
Contributor Author

turkenh commented Nov 14, 2022

/test-examples="examples/codecommit/approvalruletemplateassociation.yaml"

Copy link
Collaborator

@ulucinar ulucinar left a comment

Choose a reason for hiding this comment

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

Thank you @turkenh.

@turkenh turkenh merged commit e283844 into crossplane-contrib:main Nov 14, 2022
@turkenh turkenh deleted the fix-local-dev branch November 14, 2022 12:20
turkenh added a commit to turkenh/upbound-provider-gcp that referenced this pull request Nov 21, 2022
Signed-off-by: Hasan Turken <turkenh@gmail.com>
turkenh added a commit to turkenh/upbound-provider-azure that referenced this pull request Nov 21, 2022
Signed-off-by: Hasan Turken <turkenh@gmail.com>
turkenh added a commit to turkenh/upbound-provider-gcp that referenced this pull request Nov 21, 2022
Signed-off-by: Hasan Turken <turkenh@gmail.com>
turkenh added a commit to turkenh/upbound-provider-azure that referenced this pull request Nov 21, 2022
Signed-off-by: Hasan Turken <turkenh@gmail.com>
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.

2 participants