-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
[Filebeat][CometD] Add CometD input for Salesforce connector #30089
Conversation
…om/kush-elastic/beats into salesforce_cometd_input
This pull request is now in conflicts. Could you fix it? 🙏
|
This pull request does not have a backport label. Could you fix it @yug-elastic? 🙏
NOTE: |
I'm not sure where the following changelog entry should be added for this PR. Kindly point me to the right direction. Thanks!
|
/test |
Here, the lint is getting failed because of the following reason: CC: @mtojek |
@yug-crest Please take a look at the consoleText output for the build. Search for "go.mod" and you will see the following dump: [2022-01-30T02:43:19.999Z] >> fmt - go-licenser: Checking for missing headers
[2022-01-30T02:43:20.260Z] make[1]: Leaving directory '/var/lib/jenkins/workspace/Beats_beats_PR-30089/src/github.com/elastic/beats'
[2022-01-30T02:43:20.260Z] make[1]: Entering directory '/var/lib/jenkins/workspace/Beats_beats_PR-30089/src/github.com/elastic/beats'
[2022-01-30T02:43:21.210Z] make[1]: Leaving directory '/var/lib/jenkins/workspace/Beats_beats_PR-30089/src/github.com/elastic/beats'
[2022-01-30T02:43:21.210Z] make[1]: Entering directory '/var/lib/jenkins/workspace/Beats_beats_PR-30089/src/github.com/elastic/beats'
[2022-01-30T02:43:23.759Z] diff --git a/NOTICE.txt b/NOTICE.txt
[2022-01-30T02:43:23.759Z] index 75163cd38c..d811d05814 100644
[2022-01-30T02:43:23.759Z] --- a/NOTICE.txt
[2022-01-30T02:43:23.759Z] +++ b/NOTICE.txt
[2022-01-30T02:43:23.759Z] @@ -12408,6 +12408,37 @@ freely, subject to the following restrictions:
[2022-01-30T02:43:23.759Z] distribution.
[2022-01-30T02:43:23.759Z]
[2022-01-30T02:43:23.759Z]
[2022-01-30T02:43:23.759Z] +--------------------------------------------------------------------------------
[2022-01-30T02:43:23.759Z] +Dependency : github.com/kush-elastic/bayeux
[2022-01-30T02:43:23.759Z] +Version: v1.0.1
[2022-01-30T02:43:23.759Z] +Licence type (autodetected): MIT
[2022-01-30T02:43:23.759Z] +--------------------------------------------------------------------------------
[2022-01-30T02:43:23.759Z] +
[2022-01-30T02:43:23.759Z] +Contents of probable licence file $GOMODCACHE/github.com/kush-elastic/[email protected]/LICENSE:
[2022-01-30T02:43:23.759Z] +
[2022-01-30T02:43:23.759Z] +MIT License
[2022-01-30T02:43:23.759Z] +
[2022-01-30T02:43:23.759Z] +Copyright (c) 2017 Zander Hill
[2022-01-30T02:43:23.759Z] +
[2022-01-30T02:43:23.759Z] +Permission is hereby granted, free of charge, to any person obtaining a copy
[2022-01-30T02:43:23.759Z] +of this software and associated documentation files (the "Software"), to deal
[2022-01-30T02:43:23.759Z] +in the Software without restriction, including without limitation the rights
[2022-01-30T02:43:23.759Z] +to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
[2022-01-30T02:43:23.759Z] +copies of the Software, and to permit persons to whom the Software is
[2022-01-30T02:43:23.759Z] +furnished to do so, subject to the following conditions:
[2022-01-30T02:43:23.759Z] +
[2022-01-30T02:43:23.759Z] +The above copyright notice and this permission notice shall be included in all
[2022-01-30T02:43:23.759Z] +copies or substantial portions of the Software.
[2022-01-30T02:43:23.759Z] +
[2022-01-30T02:43:23.759Z] +THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
[2022-01-30T02:43:23.759Z] +IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
[2022-01-30T02:43:23.759Z] +FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
[2022-01-30T02:43:23.759Z] +AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
[2022-01-30T02:43:23.759Z] +LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
[2022-01-30T02:43:23.759Z] +OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
[2022-01-30T02:43:23.759Z] +SOFTWARE.
[2022-01-30T02:43:23.759Z] +
[2022-01-30T02:43:23.759Z] +
[2022-01-30T02:43:23.759Z] --------------------------------------------------------------------------------
[2022-01-30T02:43:23.759Z] Dependency : github.com/lib/pq
[2022-01-30T02:43:23.759Z] Version: v1.10.3
[2022-01-30T02:43:23.759Z] diff --git a/go.mod b/go.mod
[2022-01-30T02:43:23.759Z] index 7fbb47fabf..d388153171 100644
[2022-01-30T02:43:23.759Z] --- a/go.mod
[2022-01-30T02:43:23.759Z] +++ b/go.mod
[2022-01-30T02:43:23.759Z] @@ -193,6 +193,8 @@ require (
[2022-01-30T02:43:23.759Z] kernel.org/pub/linux/libs/security/libcap/cap v1.2.57
[2022-01-30T02:43:23.759Z] )
[2022-01-30T02:43:23.759Z]
[2022-01-30T02:43:23.759Z] +require github.com/kush-elastic/bayeux v1.0.1
[2022-01-30T02:43:23.759Z] +
[2022-01-30T02:43:23.759Z] require (
[2022-01-30T02:43:23.759Z] cloud.google.com/go v0.97.0 // indirect
[2022-01-30T02:43:23.759Z] code.cloudfoundry.org/gofileutils v0.0.0-20170111115228-4d0c80011a0f // indirect
[2022-01-30T02:43:23.759Z] @@ -250,7 +252,6 @@ require (
[2022-01-30T02:43:23.759Z] github.com/json-iterator/go v1.1.12 // indirect
[2022-01-30T02:43:23.759Z] github.com/karrick/godirwalk v1.15.6 // indirect
[2022-01-30T02:43:23.759Z] github.com/klauspost/compress v1.13.6 // indirect
[2022-01-30T02:43:23.759Z] - github.com/kush-elastic/bayeux v1.0.1 // indirect
[2022-01-30T02:43:23.759Z] github.com/markbates/pkger v0.17.0 // indirect
[2022-01-30T02:43:23.759Z] github.com/mattn/go-isatty v0.0.12 // indirect
[2022-01-30T02:43:23.759Z] github.com/mattn/go-runewidth v0.0.9 // indirect
[2022-01-30T02:43:24.705Z] NOTICE.txt: needs update
[2022-01-30T02:43:24.705Z] go.mod: needs update
[2022-01-30T02:43:24.705Z] Makefile:123: recipe for target 'check-no-changes' failed
[2022-01-30T02:43:24.705Z] make[1]: *** [check-no-changes] Error 1
[2022-01-30T02:43:24.705Z] make[1]: Leaving directory '/var/lib/jenkins/workspace/Beats_beats_PR-30089/src/github.com/elastic/beats'
[2022-01-30T02:43:24.705Z] Makefile:108: recipe for target 'check-default' failed
[2022-01-30T02:43:24.705Z] make: *** [check-default] Error 2 You can try to achieve it with |
This pull request is now in conflicts. Could you fix it? 🙏
|
/test |
/test |
This pull request is now in conflicts. Could you fix it? 🙏
|
/test |
5 similar comments
/test |
/test |
/test |
/test |
/test |
This pull request is now in conflicts. Could you fix it? 🙏
|
/test |
1 similar comment
/test |
This is a known flaky test: elastic/e2e-testing#2420. The full list is here: https://github.com/elastic/e2e-testing/issues The E2E tests in general are not as reliable as we would like them to be. If you only have E2E test failures we should be able to merge this. |
Thanks for passing links, @cmacknz! There are only have E2E test failures, all the other checks are green, but we don't have the authority to merge it. Can you please help us ship it? |
I'll let the current build complete and then merge assuming we only see unrelated E2E test failures. |
@cmacknz Sure, thank you! |
@cmacknz FYI: The current build is complete except the E2E tests which seem stuck. Could you please help us merge the PR? |
…#30089) * Add authentication mechanism for cometd input for Salesforce connector * Add data collection mechanism for Salesforce cometd input * Add asciidoc for cometd input Co-authored-by: Kush Rana <[email protected]> Co-authored-by: yug-crest <[email protected]> Co-authored-by: Craig MacKenzie <[email protected]> Co-authored-by: Kush Rana <[email protected]>
The tests for this are flaky:
From the first build after merging into main: https://beats-ci.elastic.co/blue/organizations/jenkins/Beats%2Fbeats%2Fmain/detail/main/402/pipeline/ |
…lastic#30089)" This reverts commit 7ad5703.
I'm just going to revert the merge to main while a fix for the test timeout is above is found, rather than leave main in an unstable state. You will need to open a new pull request bringing back the change and applying the needed fix. You can undo my revert by reverting it. |
* Add authentication mechanism for cometd input for Salesforce connector * Add data collection mechanism for Salesforce cometd input * Add asciidoc for cometd input Co-authored-by: Kush Rana <[email protected]> Co-authored-by: yug-crest <[email protected]> Co-authored-by: Craig MacKenzie <[email protected]> Co-authored-by: Kush Rana <[email protected]>
What does this PR do?
Real-time logs from the Salesforce Streaming API can be collected using the cometD input. For example, live Login & Logout related logs.
Right now with this PR, when the cometd input is enabled, you can start seeing the authentication log messages that are retrieved after successful authentication using the credentials provided as a part of the configuration. You will also be able to see the real-time data being collected from the Salesforce Steaming API.
For configuration:
Why is it important?
Checklist
CHANGELOG.next.asciidoc
orCHANGELOG-developer.next.asciidoc
.