Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: source is valid and fully-qualified url #524

Merged
merged 1 commit into from
Jun 22, 2023

Conversation

embano1
Copy link
Contributor

@embano1 embano1 commented Jun 22, 2023

The source property of the adapter was set using the URL host field which is incomplete and confusing for users migrating from the VEBA project to Sources. This fix changes the source semantic to be a valid and fully-qualifed URL of the configured vCenter event source.

Fixes #523

Proposed Changes

  • 🐛 Make Source valid and fully-qualified URL

Pre-review Checklist

  • At least 80% unit test coverage
  • E2E tests for any new behavior
  • Docs for any user-facing impact

Release Note

The CloudEvent `Source` attribute now is a valid fully-qualified URL of the configured vCenter event source instead of just `host:port`. This enables smooth transition from [VEBA](http://vmweventbroker.io) users as well as aligns closer with the CloudEvents specification of this attribute. Most users won't be impacted by this change, but it is a breaking change nonetheless since the field property semantics change. Users which have defined specific filters relying on the old behavior may need to change filter/parsing logic for this field.

The source property of the adapter was set using the URL host field which is incomplete and confusing for users migrating from the VEBA project to Sources. This fix changes the source semantic to be a valid and fully-qualifed URL of the configured vCenter event source.

Closes: vmware-tanzu#523
Signed-off-by: Michael Gasch <[email protected]>
@embano1 embano1 requested a review from a team as a code owner June 22, 2023 06:56
@gabo1208 gabo1208 merged commit 537aee9 into vmware-tanzu:main Jun 22, 2023
@gabo1208
Copy link
Contributor

/cherry-pick release-1.10

gabo1208 pushed a commit that referenced this pull request Jun 26, 2023
The source property of the adapter was set using the URL host field which is incomplete and confusing for users migrating from the VEBA project to Sources. This fix changes the source semantic to be a valid and fully-qualifed URL of the configured vCenter event source.

Closes: #523

Signed-off-by: Michael Gasch <[email protected]>
gabo1208 pushed a commit that referenced this pull request Jun 26, 2023
The source property of the adapter was set using the URL host field which is incomplete and confusing for users migrating from the VEBA project to Sources. This fix changes the source semantic to be a valid and fully-qualifed URL of the configured vCenter event source.

Closes: #523

Signed-off-by: Michael Gasch <[email protected]>
gabo1208 added a commit that referenced this pull request Jun 26, 2023
The source property of the adapter was set using the URL host field which is incomplete and confusing for users migrating from the VEBA project to Sources. This fix changes the source semantic to be a valid and fully-qualifed URL of the configured vCenter event source.

Closes: #523

Signed-off-by: Michael Gasch <[email protected]>
Co-authored-by: Michael Gasch <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Event Source should use different method call
2 participants