Azure-Samples / MqttApplicationSamples

Samples implementing common PubSub patterns for Edge and Cloud Brokers
MIT License
22 stars 24 forks source link

Setup: Configuring EventGrid namespace properties fails #59

Open sseiber opened 10 months ago

sseiber commented 10 months ago

This issue is for a: (mark with an x)

- [ ] bug report -> please search issues before submitting
- [ ] feature request
- [X] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

The Setup section asks the user to create/configure an EventGrid namespace. An Azure CLI command is given to set properties:

az resource create --id $res_id --is-full-object --properties '{
  "properties": {
    "isZoneRedundant": true,
    "topicsConfiguration": {
      "inputSchema": "CloudEventSchemaV1_0"
    },
    "topicSpacesConfiguration": {
      "state": "Enabled"
    }
  },
  "location": "westus2"
}'

However, in the current version of EventGrid there is an MQTT option, which when enabled, will set these properties to the desired value (see pic): image

Without enabling the MQTT option these properties are read-only and the command will fail when trying to set them.

Expected/desired behavior

The documentation should be updated to instruct the user to enable the "MQTT" feature.


Thanks! We'll be in touch soon.

rido-min commented 10 months ago

I've deployed a new EG instance with the provided script, and the EG resource was created with the correct configuration and no issues.

Did you create the resource from the portal and tried to configure with the script?

rido-min commented 10 months ago

talked to @sseiber offline

He was confused with the section title, as he thought the instance should be created in the portal and then configured.

An easy fix will be to rename the configure-event-grid-namespace section in Setup.md to create-and-configure-event-grid-namespace