openshift-online / maestro

Maestro Service Repo
Apache License 2.0
9 stars 17 forks source link

add e2e testing. #121

Closed morvencao closed 3 months ago

morvencao commented 3 months ago
  1. Add spec resync testing for agent restart
  2. Add status resync testing for server restart
  3. Add gRPC API testing
  4. Enable MTLS auth for MQTT broker.
  5. Add resync test after reconnecting for Maestro agent.
morvencao commented 3 months ago

/assign @qiujian16 @clyang82 @skeeey

morvencao commented 3 months ago

@clyang82 Another look?

clyang82 commented 3 months ago

/ok-to-test

clyang82 commented 3 months ago
Status resync Resource resync resource status after maestro server reconnects ensure the resource status is resynced [e2e-tests-status-resync]
/home/runner/work/maestro/maestro/test/e2e/pkg/status_resync_test.go:336
• [117.135 seconds]

Resources Resource CreateOnly UpdateStrategy Tests patch the nginx resource [e2e-tests-resources]
/home/runner/work/maestro/maestro/test/e2e/pkg/resources_test.go:184
• [30.017 seconds]

Spec resync Resource resync resource spec after maestro agent restarts ensure the nginx-1 resource is not updated [e2e-tests-spec-resync]
/home/runner/work/maestro/maestro/test/e2e/pkg/spec_resync_test.go:108
• [30.001 seconds]

Spec resync Resource resync resource spec after maestro agent restarts ensure the nginx-2 resource is not deleted [e2e-tests-spec-resync]
/home/runner/work/maestro/maestro/test/e2e/pkg/spec_resync_test.go:130
• [30.001 seconds]

Spec resync Resource resync resource spec after maestro agent restarts ensure the nginx-3 resource is not created [e2e-tests-spec-resync]
/home/runner/work/maestro/maestro/test/e2e/pkg/spec_resync_test.go:155
• [30.001 seconds]

Spec resync Resource resync resource spec after maestro agent reconnects ensure the nginx-1 resource is not updated [e2e-tests-spec-resync]
/home/runner/work/maestro/maestro/test/e2e/pkg/spec_resync_test.go:402
• [30.001 seconds]
------------------------------
Spec resync Resource resync resource spec after maestro agent reconnects ensure the nginx-2 resource is not deleted [e2e-tests-spec-resync]
/home/runner/work/maestro/maestro/test/e2e/pkg/spec_resync_test.go:424
• [30.000 seconds]
------------------------------
Spec resync Resource resync resource spec after maestro agent reconnects ensure the nginx-3 resource is not created [e2e-tests-spec-resync]
/home/runner/work/maestro/maestro/test/e2e/pkg/spec_resync_test.go:449
• [30.001 seconds]

I listed some cases that take a longer time to complete. Is this desired? Can the time be reduced?