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

[Backport 2.x] add IT flow agent with search index tool #2460

Merged
merged 3 commits into from
Jun 11, 2024

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 99e75aa from #2448

Signed-off-by: Jing Zhang <[email protected]>
(cherry picked from commit 99e75aa)
Zhangxunmt
Zhangxunmt previously approved these changes May 17, 2024
@dhrubo-os
Copy link
Collaborator

WAIT, why it's failing the tests? Please check.

Copy link

codecov bot commented May 17, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.08%. Comparing base (d37f636) to head (d37f636).
Report is 13 commits behind head on 2.x.

Current head d37f636 differs from pull request most recent head 1f39ba9

Please upload reports for the commit 1f39ba9 to get more accurate results.

Additional details and impacted files
@@            Coverage Diff            @@
##                2.x    #2460   +/-   ##
=========================================
  Coverage     81.08%   81.08%           
  Complexity     5989     5989           
=========================================
  Files           565      565           
  Lines         24821    24821           
  Branches       2619     2619           
=========================================
  Hits          20127    20127           
  Misses         3587     3587           
  Partials       1107     1107           
Flag Coverage Δ
ml-commons 81.08% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@yuye-aws
Copy link
Member

WAIT, why it's failing the tests? Please check.

From the error log, it seems to be an import issue due to http:

import org.apache.hc.core5.http.ParseException;

@yuye-aws
Copy link
Member

@jngz-es Can you double confirm?

@ylwu-amzn
Copy link
Collaborator

WAIT, why it's failing the tests? Please check.

From the error log, it seems to be an import issue due to http:

import org.apache.hc.core5.http.ParseException;

The class org.apache.hc.core5.http.ParseException doesn't exist in 2.x branch.

* fix import error

Signed-off-by: yuye-aws <[email protected]>

* spotless apply

Signed-off-by: yuye-aws <[email protected]>

---------

Signed-off-by: yuye-aws <[email protected]>
@ylwu-amzn ylwu-amzn temporarily deployed to ml-commons-cicd-env June 11, 2024 09:44 — with GitHub Actions Inactive
@ylwu-amzn ylwu-amzn temporarily deployed to ml-commons-cicd-env June 11, 2024 09:44 — with GitHub Actions Inactive
@ylwu-amzn ylwu-amzn temporarily deployed to ml-commons-cicd-env June 11, 2024 09:44 — with GitHub Actions Inactive
@ylwu-amzn ylwu-amzn had a problem deploying to ml-commons-cicd-env June 11, 2024 09:44 — with GitHub Actions Failure
@ylwu-amzn ylwu-amzn temporarily deployed to ml-commons-cicd-env June 11, 2024 16:40 — with GitHub Actions Inactive
@ylwu-amzn ylwu-amzn temporarily deployed to ml-commons-cicd-env June 11, 2024 16:40 — with GitHub Actions Inactive
@ylwu-amzn ylwu-amzn temporarily deployed to ml-commons-cicd-env June 11, 2024 16:40 — with GitHub Actions Inactive
@ylwu-amzn ylwu-amzn had a problem deploying to ml-commons-cicd-env June 11, 2024 16:40 — with GitHub Actions Failure
@ylwu-amzn ylwu-amzn temporarily deployed to ml-commons-cicd-env June 11, 2024 17:21 — with GitHub Actions Inactive
@ylwu-amzn ylwu-amzn temporarily deployed to ml-commons-cicd-env June 11, 2024 17:21 — with GitHub Actions Inactive
@ylwu-amzn ylwu-amzn temporarily deployed to ml-commons-cicd-env June 11, 2024 17:21 — with GitHub Actions Inactive
@ylwu-amzn ylwu-amzn temporarily deployed to ml-commons-cicd-env June 11, 2024 17:21 — with GitHub Actions Inactive
@ylwu-amzn ylwu-amzn temporarily deployed to ml-commons-cicd-env June 11, 2024 17:21 — with GitHub Actions Inactive
@ylwu-amzn ylwu-amzn temporarily deployed to ml-commons-cicd-env June 11, 2024 17:21 — with GitHub Actions Inactive
@ylwu-amzn ylwu-amzn temporarily deployed to ml-commons-cicd-env June 11, 2024 18:13 — with GitHub Actions Inactive
@ylwu-amzn ylwu-amzn temporarily deployed to ml-commons-cicd-env June 11, 2024 18:13 — with GitHub Actions Inactive
@ylwu-amzn ylwu-amzn temporarily deployed to ml-commons-cicd-env June 11, 2024 18:13 — with GitHub Actions Inactive
@jngz-es jngz-es merged commit d6ec823 into 2.x Jun 11, 2024
15 checks passed
opensearch-trigger-bot bot added a commit that referenced this pull request Oct 1, 2024
* add IT flow agent with search index tool (#2448)

Signed-off-by: Jing Zhang <[email protected]>
(cherry picked from commit 99e75aa)

* fix import error (#2527)

Signed-off-by: yuye-aws <[email protected]>

* Fix:run spotless apply for branch Backport/backport 2448 to 2.x (#2529)

* fix import error

Signed-off-by: yuye-aws <[email protected]>

* spotless apply

Signed-off-by: yuye-aws <[email protected]>

---------

Signed-off-by: yuye-aws <[email protected]>

---------

Signed-off-by: yuye-aws <[email protected]>
Co-authored-by: Jing Zhang <[email protected]>
Co-authored-by: yuye-aws <[email protected]>
(cherry picked from commit d6ec823)
dhrubo-os pushed a commit that referenced this pull request Oct 1, 2024
* add IT flow agent with search index tool (#2448)

Signed-off-by: Jing Zhang <[email protected]>
(cherry picked from commit 99e75aa)

* fix import error (#2527)

Signed-off-by: yuye-aws <[email protected]>

* Fix:run spotless apply for branch Backport/backport 2448 to 2.x (#2529)

* fix import error

Signed-off-by: yuye-aws <[email protected]>

* spotless apply

Signed-off-by: yuye-aws <[email protected]>

---------

Signed-off-by: yuye-aws <[email protected]>

---------

Signed-off-by: yuye-aws <[email protected]>
Co-authored-by: Jing Zhang <[email protected]>
Co-authored-by: yuye-aws <[email protected]>
(cherry picked from commit d6ec823)

Co-authored-by: opensearch-trigger-bot[bot] <98922864+opensearch-trigger-bot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants