Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

[BUG]Clusters deployed using IPV6 do not support the DiscoverNodes method. #458

Closed
Zhang-jie-jun opened this issue Jan 18, 2024 · 1 comment
Closed

[BUG]Clusters deployed using IPV6 do not support the DiscoverNodes method. #458

Zhang-jie-jun opened this issue Jan 18, 2024 · 1 comment
Labels
bug Something isn't working

Comments

@Zhang-jie-jun
Copy link
Contributor

What is the bug?

When using the opensearch-go library to access an opensearch cluster deployed with IPV6, an access error will occur when calling other APIs after calling the DiscoverNodes method.

How can one reproduce the bug?

  • Construct a client using NewClient
  • Call client.DiscoverNodes method
  • Calling another API
    For example: opensearchapi.CatMasterRequest{}.Do(context.Background(), client)

What is the expected behavior?

Able to use the opensearch-go library to normally access the OpenSearch cluster deployed with IPV6

What is your host/environment?

go1.17.1 Windows/amd64

Do you have any screenshots?

image

Do you have any additional context?

  • Failure
    image

+Success
image
@Zhang-jie-jun Zhang-jie-jun added bug Something isn't working untriaged labels Jan 18, 2024
@dblock
Copy link
Member

dblock commented Jan 18, 2024

Looks legit. Want to try to add a test and implement a fix for this?

@dblock dblock removed the untriaged label Jan 18, 2024
Zhang-jie-jun added a commit to Zhang-jie-jun/opensearch-go that referenced this issue Jan 19, 2024
@Zhang-jie-jun
fix:When using the opensearch-go library to access an opensearch clus…
0903c1d 
…ter deployed with IPV6, an access error will occur when calling other APIs after calling the DiscoverNodes method.

opensearch-project#458

Signed-off-by: 张洁俊 <82801500+Zhang-jie-jun@users.noreply.github.com>
@Zhang-jie-jun Zhang-jie-jun mentioned this issue Jan 19, 2024
Closed
Zhang-jie-jun pushed a commit to Zhang-jie-jun/opensearch-go that referenced this issue Jan 19, 2024
@zhang-jiejun
fix:opensearch-project#458
8212ccd
Zhang-jie-jun pushed a commit to Zhang-jie-jun/opensearch-go that referenced this issue Jan 19, 2024
@zhang-jiejun
fix:opensearch-project#458
c83d128
@Zhang-jie-jun Zhang-jie-jun mentioned this issue Jan 19, 2024
Merged
Zhang-jie-jun pushed a commit to Zhang-jie-jun/opensearch-go that referenced this issue Jan 19, 2024
fix:opensearch-project#458
b06ce74
Zhang-jie-jun pushed a commit to Zhang-jie-jun/opensearch-go that referenced this issue Jan 19, 2024
fix:opensearch-project#458
e909617
Zhang-jie-jun pushed a commit to Zhang-jie-jun/opensearch-go that referenced this issue Jan 19, 2024
fix:opensearch-project#458
97feadc
Zhang-jie-jun pushed a commit to Zhang-jie-jun/opensearch-go that referenced this issue Jan 19, 2024
fix:opensearch-project#458
c4c9a6b
Zhang-jie-jun pushed a commit to Zhang-jie-jun/opensearch-go that referenced this issue Jan 19, 2024
fix:opensearch-project#458
5e81a5f
Zhang-jie-jun added a commit to Zhang-jie-jun/opensearch-go that referenced this issue Jan 24, 2024
@Zhang-jie-jun
fix:When using the opensearch-go library to access an opensearch clus…
3d1442a 
…ter deployed with IPV6, an access error will occur when calling other APIs after calling the DiscoverNodes method.

opensearch-project#458

Signed-off-by: 张洁俊 <82801500+Zhang-jie-jun@users.noreply.github.com>
Signed-off-by: 张洁俊（Jack） <zhang.jiejun@outlook.com>
Zhang-jie-jun pushed a commit to Zhang-jie-jun/opensearch-go that referenced this issue Jan 24, 2024
@zhang-jiejun
fix:opensearch-project#458
33dfdf8 
Signed-off-by: 张洁俊（Jack） <zhang.jiejun@outlook.com>
Zhang-jie-jun pushed a commit to Zhang-jie-jun/opensearch-go that referenced this issue Jan 24, 2024
@zhang-jiejun
fix:opensearch-project#458
e9cf0d4 
Signed-off-by: 张洁俊（Jack） <zhang.jiejun@outlook.com>
Zhang-jie-jun pushed a commit to Zhang-jie-jun/opensearch-go that referenced this issue Jan 24, 2024
fix:opensearch-project#458
9f81e4f 
Signed-off-by: 张洁俊（Jack） <zhang.jiejun@outlook.com>
Zhang-jie-jun pushed a commit to Zhang-jie-jun/opensearch-go that referenced this issue Jan 24, 2024
fix:opensearch-project#458
6c17293 
Signed-off-by: 张洁俊（Jack） <zhang.jiejun@outlook.com>
Zhang-jie-jun pushed a commit to Zhang-jie-jun/opensearch-go that referenced this issue Jan 24, 2024
fix:opensearch-project#458
06d3c1c 
Signed-off-by: 张洁俊（Jack） <zhang.jiejun@outlook.com>
Zhang-jie-jun pushed a commit to Zhang-jie-jun/opensearch-go that referenced this issue Jan 24, 2024
fix:opensearch-project#458
3d68c23 
Signed-off-by: 张洁俊（Jack） <zhang.jiejun@outlook.com>
Zhang-jie-jun pushed a commit to Zhang-jie-jun/opensearch-go that referenced this issue Jan 24, 2024
fix:opensearch-project#458
b61a874 
Signed-off-by: 张洁俊（Jack） <zhang.jiejun@outlook.com>
Jakob3xD pushed a commit that referenced this issue Jan 24, 2024
@Zhang-jie-jun @zhang-jiejun
fix:Clusters deployed using IPV6 do not support the DiscoverNodes met…
47d276c 
…hod (#461)

* fix:When using the opensearch-go library to access an opensearch cluster deployed with IPV6, an access error will occur when calling other APIs after calling the DiscoverNodes method.

#458

Signed-off-by: 张洁俊 <82801500+Zhang-jie-jun@users.noreply.github.com>
Signed-off-by: 张洁俊（Jack） <zhang.jiejun@outlook.com>

* fix:#458

Signed-off-by: 张洁俊（Jack） <zhang.jiejun@outlook.com>

---------

Signed-off-by: 张洁俊 <82801500+Zhang-jie-jun@users.noreply.github.com>
Signed-off-by: 张洁俊（Jack） <zhang.jiejun@outlook.com>
Co-authored-by: 张洁俊（Jack） <jack.zhang@eisoo.com>
Co-authored-by: 张洁俊（Jack） <zhang.jiejun@outlook.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@dblock @Jakob3xD @Zhang-jie-jun