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.

Sign up for GitHub

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

Jump to bottom

[community-4.9] WINC-607: Add support for platform=none in e2e test suite (Cont.) #938

Merged
merged 3 commits into from
Mar 1, 2022
Merged

[community-4.9] WINC-607: Add support for platform=none in e2e test suite (Cont.) #938

merged 3 commits into from
Mar 1, 2022

Commits on Feb 25, 2022

  1. Skip loadbalancer tests for platform=none 

    - Disable the application ingress load balancer test for
platform=none
- Use t.Skipf() to ignore the test with better logging
- Move the check to the beginning of the test to avoid the creation
of the WindowsWebServer deployment

Follow-up to 91ecc9d
    @jrvaldes
    jrvaldes authored and openshift-cherrypick-robot committed Feb 25, 2022
    Configuration menu
    Copy the full SHA
    24ad56a View commit details
    Browse the repository at this point in the history

  2. Skip MachineSet update in deletion test 

    This commit skips the MachineSet update in the Windows node deletion
test if no MachineSet is found with the Windows label. This is a
corner case for platform-agnostic infrastructure and e2e tests
executions where no instances are configured via MachineController, for
example: `hack/run-ci-e2e-test.sh -m 0`.
    @jrvaldes
    jrvaldes authored and openshift-cherrypick-robot committed Feb 25, 2022
    Configuration menu
    Copy the full SHA
    e05500a View commit details
    Browse the repository at this point in the history

  3. Install jq in Dockerfile.ci 

    `jq` is needed in e2e test script to count the number of data items
in the windows-instances ConfigMap.
    @jrvaldes
    jrvaldes authored and openshift-cherrypick-robot committed Feb 25, 2022
    Configuration menu
    Copy the full SHA
    124c7e1 View commit details
    Browse the repository at this point in the history