bugfix: Adds cluster role for otel daemonset deployment mode

[arunvthangaraj]

Issue #, if available: 990

Description of changes: This PR adds the required cluster role for the daemonset template.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

[elamaran11]

LGTM.

[elamaran11]

@shapirov103 Please approve and Merge. E2E is not required.

[shapirov103]

LGTM

[shapirov103]

/do-e2e-tests

[aws-ia-ci]

end to end tests failed. A maintainer can provide more details.

[shapirov103]

/do-e2e-tests

[aws-ia-ci]

end to end tests failed. A maintainer can provide more details.

[shapirov103]

@shapirov103 Please approve and Merge. E2E is not required.

@elamaran11 aren't we deploying this addon in e2e, so if there is an error in the modified kubernetes manifest we should be able to catch it. Ideally, functional verification is also needed, e.g. similar to what we do in Conformitron. That will improve quality assurance and reduce issues from customers.

I will look into the e2e failure.

[elamaran11]

@shapirov103 Please approve and Merge. E2E is not required.

@elamaran11 aren't we deploying this addon in e2e, so if there is an error in the modified kubernetes manifest we should be able to catch it. Ideally, functional verification is also needed, e.g. similar to what we do in Conformitron. That will improve quality assurance and reduce issues from customers.

I will look into the e2e failure.

@shapirov103 We do have E2E for this addon already but the addon supports 4 different types of modes - deployment daemonset statefulset and sidecar we cant accomodate all of these in one e2e deployment. So our e2e deploys only deployment mode. We can alter it but we dont use daemonset approach in most situations as its an anti-pattern. Arun is working on a blog with daemonset so this was broken and this change is required. But e2e failure is not related to this change. So once e2e issues are fixed, we can merge this.

[elamaran11]

@shapirov103 Can this be merged?

Failure unrelated to the modified code (sporadic network connection issue)

[shapirov103]

Yes, the error on e2e was unrelated.