CloudEvent ID is too long - 72 chars

[talebzeghmi]

Describe the bug

argo-events/eventsources/eventing.go

Line 550 in dc0cb64

event.SetID(fmt.Sprintf("%x", uuid.New()))

The intention of the code is to convert the UUID to a hex string without the dashes, but the eventID is of form and length 72 633930656462302d376131632d346236372d393236642d353930393166663764626431 because it is converting the string representation of the UUID including the dashes.

The impact of this bug is that it makes it impossible for an EventSensor to add the ID as a workflow label because k8s limits the value to length of 63, see link

The fix would be to use [:]

uuidNew = uuid.New()
event.SetID(fmt.Sprintf("%x", uuidNew[:]))

To Reproduce
Steps to reproduce the behavior:

1. Have the EventSource consume and event to forward to the EventBus
2. Look at the log and the CloudEvent ID is of length 72

eventSourceType":"sqs","eventID":"63653966636463352d636139632d346331352d626464642d356136666139343333306664"}

Expected behavior
The expected format of cd2a34891fe847ab871f937679ebf950 which is of length 32.

Environment (please complete the following information):

• Argo Events: v1.9.1

---

Message from the maintainers:

If you wish to see this enhancement implemented please add a 👍 reaction to this issue! We often sort issues this way to know what to prioritize.