Skip to content

chore: update az aks create invocation in azure setup docs #1091

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

Merged
merged 2 commits into from
Aug 29, 2022
Merged

Conversation

johnstcn
Copy link
Member

  • Cannot set --network-policy azure with --network-plugin kubenet:
(NetworkPolicyAzureNotSupportedForKubenet) NetworkPolicy azure is not supported for network plugin kubenet
Code: NetworkPolicyAzureNotSupportedForKubenet
Message: NetworkPolicy azure is not supported for network plugin kubenet
Target: networkProfile.networkPolicy
  • Must set the --node-count parameter equal to --min-count as it appears to default to 0 if unspecified.

@johnstcn johnstcn self-assigned this Aug 19, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Aug 19, 2022

✨ Coder.com for PR #1091 deployed! It will be updated on every commit.

@johnstcn johnstcn requested a review from ericpaulsen August 19, 2022 10:08
Copy link
Member

@bpmct bpmct left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me, matches the Limitations & considerations for kubenet
doc

* Cannot set `--network-policy azure` with `--network-plugin kubenet`:
```
(NetworkPolicyAzureNotSupportedForKubenet) NetworkPolicy azure is not supported for network plugin kubenet
Code: NetworkPolicyAzureNotSupportedForKubenet
Message: NetworkPolicy azure is not supported for network plugin kubenet
Target: networkProfile.networkPolicy
```

* Must set the `--node-count` parameter equal to `--min-count` as it appears to default to 0 if unspecified.
Comment on lines +80 to +82
Coder's trial license does not work in an air-gapped environment. If your
organization is interested in evaluating Coder air-gapped, please contact
[sales@coder.com](mailto:sales@coder.com) to discuss license requirements.
Copy link
Member

@bpmct bpmct Aug 29, 2022

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

0cffb50 introduced a linter error, which was causing tests to fail

@bpmct
Copy link
Member

bpmct commented Aug 29, 2022

Rebased and merging

@bpmct bpmct merged commit 98e08af into main Aug 29, 2022
@bpmct bpmct deleted the cj/azure-setup branch August 29, 2022 10:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants