Reconcile migration to v1 fluentd API and conform to latest code.#93
Reconcile migration to v1 fluentd API and conform to latest code.#93Cryptophobia wants to merge 1 commit intoGoogleCloudPlatform:masterfrom
Conversation
|
All (the pull request submitter and all commit authors) CLAs are signed, but one or more commits were authored or co-authored by someone other than the pull request submitter. We need to confirm that all authors are ok with their commits being contributed to this project. Please have them confirm that by leaving a comment that contains only Note to project maintainer: There may be cases where the author cannot leave a comment, or the comment is not properly detected as consent. In those cases, you can manually confirm consent of the commit author(s), and set the ℹ️ Googlers: Go here for more info. |
|
All test pass, please merge this @qingling128 ! |
|
Dear @qingling128 and @igorpeshansky , I am beginning to worry about you guys. Is everything okay at Google Cloud? Yours with PRs ready to merge, |
|
I heard about the Google Cloud networking outage yesterday. Is everything okay? If the Google Cloud is holding you hostage please blink twice and drop your gpg public keys from the honeypot. We can get you out with the eSWAT team. |
lvelvis
left a comment
There was a problem hiding this comment.
Please note that the current latest code does not support reloading. Therefore, when merging your code, please be mindful of this limitation and ensure that any necessary actions or configurations are taken into consideration to handle this situation appropriately.
|
Closes #77 |
- fixing missing code to make v1 API commit work with upstream changes - fixing functions for new parameter force_line_breaks - fixing rubocop errors and exanded lineLength to 100 chars - all tests pass on ruby 2.7.4 - this combines GoogleCloudPlatform#45 (resolves GoogleCloudPlatform#45) and reconciles it for the latest code changes in upstream Signed-off-by: Cryptophobia <aouzounov@gatech.edu>
|
@lvelvis @nshelke777 , can you please review and merge in? I have rebased and fixed all rake tests that were failing. |
fix: reconcile upstream with v1 API PR
force_line_breaksSigned-off-by: Cryptophobia aouzounov@gatech.edu