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

Use default dnsPolicy for AKS #3701

Merged
merged 1 commit into from
Apr 29, 2022
Merged

Use default dnsPolicy for AKS #3701

merged 1 commit into from
Apr 29, 2022

Conversation

tnqn
Copy link
Member

@tnqn tnqn commented Apr 28, 2022

AKS overrides KUBERNETES_SERVICE_HOST with a FQDN value. Using
ClusterFirstWithHostNet would lead to antrea-agent start failure.

Signed-off-by: Quan Tian qtian@vmware.com

For #3699

@tnqn tnqn requested a review from antoninbas April 28, 2022 16:45
antoninbas
antoninbas previously approved these changes Apr 28, 2022
@antoninbas
Copy link
Contributor

@tnqn not merging this yet, because I imagine you want to run the AKS Jenkins job for this PR first?

@antoninbas
Copy link
Contributor

/skip-all

@tnqn
Copy link
Member Author

tnqn commented Apr 29, 2022

@tnqn not merging this yet, because I imagine you want to run the AKS Jenkins job for this PR first?

@antoninbas yes, I have ran AKS test once manually with combination of #3700, #3701, and #3702 successfully. The test requires all of them to pass but they are independent and some of them will need backport so I don't just use one PR. Could you please help review #3700? It seems CI testbeds are affected by it because it updated the sonobuoy version but other PRs were using an old sonobuoy image, which caused test timeout.

AKS overrides KUBERNETES_SERVICE_HOST with a FQDN value. Using
ClusterFirstWithHostNet would lead to antrea-agent start failure.

Signed-off-by: Quan Tian <qtian@vmware.com>
@tnqn
Copy link
Member Author

tnqn commented Apr 29, 2022

/skip-all

@tnqn tnqn merged commit 206d06f into antrea-io:main Apr 29, 2022
@tnqn tnqn deleted the fix-dns-for-aks branch April 29, 2022 11:59
@tnqn tnqn added action/backport Indicates a PR that requires backports. action/release-note Indicates a PR that should be included in release notes. labels Apr 29, 2022
tnqn added a commit that referenced this pull request Apr 29, 2022
* Determine conformance container image version at runtime

Since Sonobuoy 0.20, Sonobuoy releases are no longer attached to
Kubernetes releases and can determine the images at runtime according to
the Kubernetes server's version. This patch removes pinned K8s test
image versions to fix compatability issues caused by version skew.

Signed-off-by: Quan Tian <qtian@vmware.com>

* Use default dnsPolicy for AKS

AKS overrides KUBERNETES_SERVICE_HOST with a FQDN value. Using
ClusterFirstWithHostNet would lead to antrea-agent start failure.

Signed-off-by: Quan Tian <qtian@vmware.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
action/backport Indicates a PR that requires backports. action/release-note Indicates a PR that should be included in release notes.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants