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

tests(fix): conformance tests pass #275

Merged
merged 1 commit into from
May 20, 2024
Merged

tests(fix): conformance tests pass #275

merged 1 commit into from
May 20, 2024

Conversation

programmer04
Copy link
Member

@programmer04 programmer04 commented May 20, 2024

What this PR does / why we need it:

Followup for

This PR contains several fixes that make make tests.conformance pass and generate a report.

Please check on your machine too we don't run them in CI currently.

Fixes

Generated report

apiVersion: gateway.networking.k8s.io/v1alpha1
date: "2024-05-20T14:18:35+02:00"
gatewayAPIChannel: experimental
gatewayAPIVersion: v1.1.0
implementation:
  contact:
  - github.com/kong/gateway-operator/issues/new/choose
  organization: Kong
  project: gateway-operator
  url: github.com/kong/gateway-operator
  version: v1.2.2-39-g8f28499
kind: ConformanceReport
mode: default
profiles:
- core:
    result: partial
    skippedTests:
    - GatewayInvalidTLSConfiguration
    - GatewayModifyListeners
    - GatewayWithAttachedRoutes
    - HTTPRouteHTTPSListener
    - HTTPRouteHeaderMatching
    - HTTPRouteHostnameIntersection
    - HTTPRouteInvalidBackendRefUnknownKind
    - HTTPRouteListenerHostnameMatching
    - HTTPRouteObservedGenerationBump
    statistics:
      Failed: 0
      Passed: 24
      Skipped: 9
  name: GATEWAY-HTTP
  summary: Core tests partially succeeded with 9 test skips.

Which issue this PR fixes

Closes #264

@programmer04 programmer04 enabled auto-merge (squash) May 20, 2024 12:55
Copy link
Contributor

@czeslavo czeslavo left a comment

Choose a reason for hiding this comment

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

Nice 🎖️

@programmer04 programmer04 merged commit 4066cd5 into main May 20, 2024
14 checks passed
@programmer04 programmer04 deleted the conformance-runs branch May 20, 2024 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Make conformance test working
2 participants