Uploaded image for project: 'AMQ Broker'
  1. AMQ Broker
  2. ENTMQBR-4159

If you don't specify a port it breaks when using stomp protocol

    XMLWordPrintable

Details

    • Bug
    • Resolution: Done
    • Major
    • AMQ 7.8.0.CR2
    • AMQ 7.6.0.GA
    • operator
    • 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

    Description

      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

      Attachments

        Activity

          People

            rhn-support-rkieley Roderick Kieley
            rhn-support-rkieley Roderick Kieley
            Mikhail Krutov Mikhail Krutov
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved: