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

New pipelines for testing application-connector components #5563

Merged
merged 51 commits into from
Jul 14, 2022

Conversation

koala7659
Copy link
Contributor

@koala7659 koala7659 commented Jun 10, 2022

Adding to kyma-integration pipelines pre and post submit jobs:

  • main-kyma-integration-k3d-app-gateway for testing (Central) Application Gateway

  • main-kyma-integration-k3d-app-conn-validator for testing (Central) Application Connectivity Validator

  • main-kyma-integration-k3d-runtime-agent for testing Compass Runtime Agent

  • All executed tests are in tests/components/application-connector

  • Testing platform: k3d installed in GCP VM

  • Minimal Kyma installation for both OS and SKR setup

  • The run if changed rule fires pipelines after files are changed at locations:

    • resources/application-connector fires pre-main-kyma-integration-k3d-app-gateway and pre-main-kyma-integration-k3d-app-conn-validator
    • resources/compass-runtime-agent fires pre-main-kyma-integration-k3d-runtime-agent

@koala7659 koala7659 requested a review from a team as a code owner June 10, 2022 09:21
@koala7659 koala7659 requested review from halamix2 and Ressetkk June 10, 2022 09:21
@kyma-bot kyma-bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Jun 10, 2022
@kyma-bot kyma-bot added size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Jun 13, 2022
@kyma-bot kyma-bot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. labels Jun 14, 2022
@kyma-bot kyma-bot added the lgtm Looks good to me! label Jul 12, 2022
@franpog859
Copy link
Contributor

/test pre-main-test-infra-pjtester

franpog859
franpog859 previously approved these changes Jul 13, 2022
@koala7659 koala7659 dismissed stale reviews from franpog859 and majakurcius via c851f9d July 13, 2022 08:52
@kyma-bot kyma-bot removed the lgtm Looks good to me! label Jul 13, 2022
franpog859
franpog859 previously approved these changes Jul 13, 2022
@kyma-bot kyma-bot added the lgtm Looks good to me! label Jul 13, 2022
Copy link
Contributor Author

@koala7659 koala7659 left a comment

Choose a reason for hiding this comment

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

LGTM

Ressetkk
Ressetkk previously approved these changes Jul 14, 2022
klaudiagrz
klaudiagrz previously approved these changes Jul 14, 2022
@koala7659 koala7659 dismissed stale reviews from klaudiagrz, Ressetkk, and franpog859 via 5c1c161 July 14, 2022 10:06
@kyma-bot kyma-bot removed the lgtm Looks good to me! label Jul 14, 2022
@kyma-bot
Copy link
Contributor

kyma-bot commented Jul 14, 2022

@contributor: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
pre-main-test-infra-lint 241c588 link false /test pre-main-test-infra-lint
pre-main-test-infra-pjtester-v2 241c588 link true /test pre-main-test-infra-pjtester-v2
pre-main-test-infra-vpathguard 6d8288d link true /test pre-main-test-infra-vpathguard

Full PR test history. Your PR dashboard.

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.

@kyma-bot kyma-bot added the lgtm Looks good to me! label Jul 14, 2022
@kyma-bot kyma-bot merged commit 5277558 into kyma-project:main Jul 14, 2022
@kyma-bot
Copy link
Contributor

@koala7659: Updated the following 2 configmaps:

  • config configmap in namespace default at cluster default using the following files:
    • key config.yaml using file prow/config.yaml
  • job-config configmap in namespace default at cluster default using the following files:
    • key kyma-integration.yaml using file prow/jobs/kyma/kyma-integration.yaml

In response to this:

Adding to kyma-integration pipelines pre and post submit jobs:

  • main-kyma-integration-k3d-app-gateway for testing (Central) Application Gateway

  • main-kyma-integration-k3d-app-conn-validator for testing (Central) Application Connectivity Validator

  • main-kyma-integration-k3d-runtime-agent for testing Compass Runtime Agent

  • All executed tests are in tests/components/application-connector

  • Testing platform: k3d installed in GCP VM

  • Minimal Kyma installation for both OS and SKR setup

  • The run if changed rule fires pipelines after files are changed at locations:

    • resources/application-connector fires pre-main-kyma-integration-k3d-app-gateway and pre-main-kyma-integration-k3d-app-conn-validator
    • resources/compass-runtime-agent fires pre-main-kyma-integration-k3d-runtime-agent

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm Looks good to me! size/XL Denotes a PR that changes 500-999 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants