-
Bug
-
Resolution: Done
-
Major
-
AMQ 7.6.0.GA
-
None
-
False
-
False
-
In prior releases, if you defined an acceptor to use the STOMP protocol, but did not specify a port for the acceptor to use, the Operator failed to create a Service and Route for the acceptor. This issue is now resolved.
-
Documented as Resolved Issue
-
Lifted from conversation:
ok well, i added a stomp acceptor, and it partially worked
it just didn't actually create the service
spec:
acceptors:
- expose: true
name: stomp
protocols: stomp
console:
expose: true
deploymentPlan:
image: 'registry.redhat.io/amq7/amq-broker:7.5'
messageMigration: true
size: 1
upgrades:
enabled: false
minor: false
looks like the operator failed to create the service
{"level":"error","ts":1603911962.7169914,"logger":"controller_v2alpha2activemqartemis","msg":"Failed to create resource","ActiveMQArtemis Name":"srt-broker","kind":"srt-broker-stomp-0-svc","requested":{"apiVersion":"v1","kind":"Service","namespace":"srt","name":"srt-broker-stomp-0-svc"},"error":"Service \"srt-broker-stomp-0-svc\" is invalid: [spec.ports[0].port: Invalid value: 0: must be between 1 and 65535, inclusive, spec.ports[0].targetPort: Invalid value: 0: must be between 1 and 65535, inclusive]","stacktrace":"github.com/rh-messaging/activemq-artemis-operator/vendor/github.com/go-logr/zapr.(*zapLogger).Error\n\t/tmp/go/src/github.com/rh-messaging/activemq-artemis-operator/vendor/github.com/go-logr/zapr/zapr.go:128\ngithub.com/rh-messaging/activemq-artemis-operator/pkg/controller/broker/v2alpha2/activemqartemis.(*ActiveMQArtemisReconciler).ProcessUpgrade\n\t/tmp/go/src/github.com/rh-messaging/activemq-artemis-operator/pkg/controller/broker/v2alpha2/activemqartemis/activemqartemis_reconciler.go:1011\ngithub.com/rh-messaging/activemq-artemis-operator/pkg/controller/broker/v2alpha2/activemqartemis.(*ActiveMQArtemisReconciler).Process\n\t/tmp/go/src/github.com/rh-messaging/activemq-artemis-operator/pkg/controller/broker/v2alpha2/activemqartemis/activemqartemis_reconciler.go:116\ngithub.com/rh-messaging/activemq-artemis-operator/pkg/controller/broker/v2alpha2/activemqartemis.(*ContainerRunningState).Update\n\t/tmp/go/src/github.com/rh-messaging/activemq-artemis-operator/pkg/controller/broker/v2alpha2/activemqartemis/containerRunning_state.go:93\ngithub.com/rh-messaging/activemq-artemis-operator/pkg/utils/fsm.(*Machine).Update\n\t/tmp/go/src/github.com/rh-messaging/activemq-artemis-operator/pkg/utils/fsm/fsm.go:91\ngithub.com/rh-messaging/activemq-artemis-operator/pkg/controller/broker/v2alpha2/activemqartemis.(*ActiveMQArtemisFSM).Update\n\t/tmp/go/src/github.com/rh-messaging/activemq-artemis-operator/pkg/controller/broker/v2alpha2/activemqartemis/activemqartemis_fsm.go:133\ngithub.com/rh-messaging/activemq-artemis-operator/pkg/controller/broker/v2alpha2/activemqartemis.(*ReconcileActiveMQArtemis).Reconcile\n\t/tmp/go/src/github.com/rh-messaging/activemq-artemis-operator/pkg/controller/broker/v2alpha2/activemqartemis/activemqartemis_controller.go:158\ngithub.com/rh-messaging/activemq-artemis-operator/vendor/sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).processNextWorkItem\n\t/tmp/go/src/github.com/rh-messaging/activemq-artemis-operator/vendor/sigs.k8s.io/controller-runtime/pkg/internal/controller/controller.go:215\ngithub.com/rh-messaging/activemq-artemis-operator/vendor/sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func1\n\t/tmp/go/src/github.com/rh-messaging/activemq-artemis-operator/vendor/sigs.k8s.io/controller-runtime/pkg/internal/controller/controller.go:158\ngithub.com/rh-messaging/activemq-artemis-operator/vendor/k8s.io/apimachinery/pkg/util/wait.JitterUntil.func1\n\t/tmp/go/src/github.com/rh-messaging/activemq-artemis-operator/vendor/k8s.io/apimachinery/pkg/util/wait/wait.go:133\ngithub.com/rh-messaging/activemq-artemis-operator/vendor/k8s.io/apimachinery/pkg/util/wait.JitterUntil\n\t/tmp/go/src/github.com/rh-messaging/activemq-artemis-operator/vendor/k8s.io/apimachinery/pkg/util/wait/wait.go:134\ngithub.com/rh-messaging/activemq-artemis-operator/vendor/k8s.io/apimachinery/pkg/util/wait.Until\n\t/tmp/go/src/github.com/rh-messaging/activemq-artemis-operator/vendor/k8s.io/apimachinery/pkg/util/wait/wait.go:88"}
i guess I NEEDED to specify a port
yeah if you don't specify a port it breaks. sounds like the crd needs to make that required or something
