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

Add e2e test for different runtimes and allow them to be parallely runned #9223

Merged
merged 4 commits into from Aug 27, 2020

Conversation

dbadura
Copy link
Contributor

@dbadura dbadura commented Aug 11, 2020

Description

Changes proposed in this pull request:

  • add test for python37
  • add abilit to run test for python and nodejs in the same time.
    Related issue(s)

@dbadura dbadura force-pushed the add-more-runtimes-e2e-tests branch 2 times, most recently from b35461b to 4603019 Compare August 11, 2020 18:18
@netlify
Copy link

netlify bot commented Aug 11, 2020

🥰 Documentation preview ready! 🥰

Built with commit 66bcbff

https://deploy-preview-9223--kyma-project-docs-preview.netlify.app

@dbadura dbadura changed the title Add more runtimes e2e tests Add e2e test for different runtimes and allow them to be parallely runned Aug 12, 2020
@dbadura
Copy link
Contributor Author

dbadura commented Aug 12, 2020

/retest

@dbadura dbadura marked this pull request as ready for review August 13, 2020 11:03
@dbadura dbadura requested a review from a user August 13, 2020 11:03
@dbadura dbadura requested a review from sayanh as a code owner August 13, 2020 11:03
@dbadura dbadura force-pushed the add-more-runtimes-e2e-tests branch 2 times, most recently from df5baaf to ffad48e Compare August 13, 2020 11:22
@kyma-project kyma-project deleted a comment from kyma-bot Aug 13, 2020
@dbadura
Copy link
Contributor Author

dbadura commented Aug 13, 2020

/retest

@dbadura dbadura force-pushed the add-more-runtimes-e2e-tests branch 2 times, most recently from 2dcee1e to 3af9e5a Compare August 13, 2020 13:54
@dbadura dbadura force-pushed the add-more-runtimes-e2e-tests branch 2 times, most recently from 0e52cb8 to a45d478 Compare August 13, 2020 14:39
@aerfio aerfio self-assigned this Aug 18, 2020
@aerfio aerfio force-pushed the add-more-runtimes-e2e-tests branch 3 times, most recently from f0a6a14 to 42efd5c Compare August 25, 2020 10:03
@aerfio aerfio force-pushed the add-more-runtimes-e2e-tests branch 2 times, most recently from 66bcbff to f68f987 Compare August 25, 2020 10:10
@aerfio
Copy link
Contributor

aerfio commented Aug 25, 2020

/retest

@aerfio aerfio force-pushed the add-more-runtimes-e2e-tests branch from bab0428 to 23580a7 Compare August 26, 2020 09:28
@aerfio
Copy link
Contributor

aerfio commented Aug 26, 2020

/retest

1 similar comment
@aerfio
Copy link
Contributor

aerfio commented Aug 26, 2020

/retest

aerfio
aerfio previously approved these changes Aug 26, 2020
Copy link
Contributor

@aerfio aerfio left a comment

Choose a reason for hiding this comment

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

LGTM, most of the work was done by @dbadura

@kyma-bot
Copy link
Contributor

kyma-bot commented Aug 26, 2020

@dbadura: The following tests failed, say /retest to rerun all failed tests:

Test name Commit Details Rerun command
pre-master-kyma-gke-upgrade-for-debug e5676a8 link /test pre-master-kyma-gke-upgrade-for-debug
kyma-governance 66bcbff link /test kyma-governance

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@aerfio
Copy link
Contributor

aerfio commented Aug 27, 2020

/retest

@tgorgol tgorgol merged commit 95e3dfe into kyma-project:master Aug 27, 2020
@dbadura dbadura deleted the add-more-runtimes-e2e-tests branch October 7, 2020 08:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/serverless Issues or PRs related to serverless
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants