This pull request includes several changes to the api, Makefile, and configuration files to improve testing, webhook setup, and default values. The most important changes include adding an E2E test workflow, updating default values and validation rules, and enhancing the webhook setup.
CI/CD Improvements:
.github/workflows/test.yml: Added an E2E test workflow that runs on pushes and pull requests to the main branch.
Makefile Enhancements:
Makefile: Added a step to copy CRD bases to the deployment charts directory in the manifests target.
Makefile: Updated the test target to include additional flags for verbose and trace output.
API and Webhook Updates:
api/v1beta1/automq_types.go: Changed the default value for the Enable field in MetricsSpec from prometheus to true.
This pull request includes several changes to the
api
,Makefile
, and configuration files to improve testing, webhook setup, and default values. The most important changes include adding an E2E test workflow, updating default values and validation rules, and enhancing the webhook setup.CI/CD Improvements:
.github/workflows/test.yml
: Added an E2E test workflow that runs on pushes and pull requests to themain
branch.Makefile Enhancements:
Makefile
: Added a step to copy CRD bases to the deployment charts directory in themanifests
target.Makefile
: Updated thetest
target to include additional flags for verbose and trace output.API and Webhook Updates:
api/v1beta1/automq_types.go
: Changed the default value for theEnable
field inMetricsSpec
fromprometheus
totrue
.api/v1beta1/automq_webhook.go
: Implemented defaulting logic for various fields in theAutoMQ
struct.api/v1beta1/webhook_suite_test.go
: Added tests for default values and included additional API schemes. [1] [2] [3] [4] [5] [6]Configuration Changes:
config/crd/bases/infra.cuisongliu.github.com_automqs.yaml
: Updated the default value for theenable
field in theMetrics
configuration totrue
.deploy/charts/automq-operator/crds/infra.cuisongliu.github.com_automqs.yaml
: Removed default values forname
fields and unnecessaryrequest
fields. [1] [2] [3] [4] [5] [6] [7]Codebase Simplification:
api/v1beta1/images.go
: Renamed frominternal/controller/images.go
and updated the package name tov1beta1
.gen/gen.go
: Updated references to usev1beta1.DefaultImageName
instead ofcontroller.DefaultImageName
. [1] [2]