Skip to content

Conversation

@fabiobrz
Copy link
Collaborator

@fabiobrz fabiobrz commented Sep 23, 2024

Short description of what this resolves:

Changes aim at resuming Arquillian Cube tests for Kubernetes integration.

Changes proposed in this pull request:

  • Fixing Kubernetes integration implementation
  • Fixing test resources and tests
  • Removing the CircleCI script which does nothing relevant to actual integration tests, and improving the GitHub actions script n order to run a matrix on the docker and kubernetes targets.

Fixes #1313

…t scope to functional test modules, as required after the split of such artifacts in the Fabri8 Kubernetes client project
…t Accept header for the HTTP call that verifies the version
… to use the correct ones, i.e. apps/replicasets
… .metadata.annotations in a Service resource and avoid NPE, when creatiing an URL for the service itself
@fabiobrz fabiobrz force-pushed the issues-1313.resume-k8s-it branch 2 times, most recently from ec9699d to 064fafe Compare September 23, 2024 11:18
…fix imports plus expected body contentsin HelloWorldIT
@fabiobrz fabiobrz force-pushed the issues-1313.resume-k8s-it branch 8 times, most recently from 6578487 to e2b0ccb Compare September 23, 2024 14:27
@fabiobrz fabiobrz marked this pull request as draft September 23, 2024 14:40
@fabiobrz fabiobrz marked this pull request as ready for review September 23, 2024 15:22
@fabiobrz fabiobrz force-pushed the issues-1313.resume-k8s-it branch from e2b0ccb to 05bb397 Compare September 23, 2024 15:22
@fabiobrz fabiobrz closed this Sep 23, 2024
@fabiobrz fabiobrz reopened this Sep 23, 2024
@fabiobrz fabiobrz closed this Sep 23, 2024
@fabiobrz fabiobrz reopened this Sep 23, 2024
@fabiobrz fabiobrz force-pushed the issues-1313.resume-k8s-it branch from 05bb397 to e0e477c Compare September 24, 2024 13:05
@fabiobrz fabiobrz requested a review from jimma September 24, 2024 15:30
@fabiobrz
Copy link
Collaborator Author

I am merging this, based on the approval and related discussion with @jimma about the pending comment, which has been agreed upon to be marked as resolved.

@fabiobrz fabiobrz merged commit ef368b4 into arquillian:main Sep 27, 2024
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.

Make K8s integration tests green again, and resume the related CI execution

2 participants