Skip to content

Allow windows hosts file for netty dns resolving#73541

Closed
rjernst wants to merge 5 commits intoelastic:masterfrom
rjernst:azure_upgrade6
Closed

Allow windows hosts file for netty dns resolving#73541
rjernst wants to merge 5 commits intoelastic:masterfrom
rjernst:azure_upgrade6

Conversation

@rjernst
Copy link
Copy Markdown
Member

@rjernst rjernst commented May 28, 2021

The update of the Azure SDK to 12.11.1 started using the netty dns
resolver. This reads /etc/hosts on nix machines. But we also need to
account for Windows machines, which have a similar hosts file.

closes #73539

The update of the Azure SDK to 12.11.1 started using the netty dns
resolver. This reads /etc/hosts on nix machines. But we also need to
account for Windows machines, which have a similar hosts file.

closes elastic#73539
@rjernst rjernst added >bug :Distributed/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs v8.0.0 v7.14.0 labels May 28, 2021
@elasticmachine elasticmachine added the Team:Distributed Meta label for distributed team. label May 28, 2021
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-distributed (Team:Distributed)

@mark-vieira mark-vieira added the test-windows Trigger CI checks on Windows label May 28, 2021
@mark-vieira
Copy link
Copy Markdown
Contributor

@elasticmachine run elasticsearch-ci/part-1-windows

@rjernst
Copy link
Copy Markdown
Member Author

rjernst commented May 28, 2021

@elasticmachine run elasticsearch-ci/part-2-windows

1 similar comment
@rjernst
Copy link
Copy Markdown
Member Author

rjernst commented May 28, 2021

@elasticmachine run elasticsearch-ci/part-2-windows

@fcofdez
Copy link
Copy Markdown
Contributor

fcofdez commented Jun 1, 2021

It seems like the SecurityManager is still unhappy about the read permissions 🤔 @rjernst

Caused by: java.security.AccessControlException: access denied ("java.io.FilePermission" "C:\Windows\system32\drivers\etc\hosts" "read")

@rjernst
Copy link
Copy Markdown
Member Author

rjernst commented Jun 3, 2021

@elasticmachine run elasticsearch-ci/part-2-windows

@rjernst
Copy link
Copy Markdown
Member Author

rjernst commented Jun 7, 2021

The Azure upgrade that necessitated this resolving is being reverted (#73837). Closing this PR as it is no longer necessary until Azure is upgraded again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

>bug :Distributed/Snapshot/Restore Anything directly related to the `_snapshot/*` APIs Team:Distributed Meta label for distributed team. test-windows Trigger CI checks on Windows v7.14.0 v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CI] Windows azure repository tests fail with AccessControlException in AzureRepositoryPlugin

5 participants