-
Notifications
You must be signed in to change notification settings - Fork 25
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鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add HorizonSource
#416
Add HorizonSource
#416
Conversation
Codecov Report
@@ Coverage Diff @@
## main #416 +/- ##
===========================================
- Coverage 81.62% 66.23% -15.39%
===========================================
Files 27 38 +11
Lines 1382 2070 +688
===========================================
+ Hits 1128 1371 +243
- Misses 226 655 +429
- Partials 28 44 +16
Continue to review full report at Codecov.
|
ec76de1
to
fc81a77
Compare
samples/horizon/README.md
Outdated
|
||
### Deploy the Source | ||
|
||
Finally, deploy the `HorizonSource`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This trailing spaces and I think is good to go
samples/horizon/README.md
Outdated
|
||
### Configure the Source | ||
|
||
Modify the `horizon-source.yml` according to your environment. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
.
Closes: vmware-tanzu#392 Signed-off-by: Michael Gasch <mgasch@vmware.com> Signed-off-by: Michael Gasch <15986659+embano1@users.noreply.github.com>
A new
HorizonSource
is available reusing most of the code running in production from the VMware Event Broker Appliance Horizon eventprocessor
.Fixes #392
Proposed Changes
HorizonSource
<product>-source-<webhook|adapter|controller>
, e.g.horizon-source-controller
(also see Break up VSphereSource webhook into controller and webhook component聽#407)Pre-review Checklist
Notes:
adapter
for aHorizonSource
Release Note