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

hotfix(privatek8s) correct outbound LB SNAT port #581

Merged
merged 1 commit into from
Jan 17, 2024
Merged

Conversation

dduportal
Copy link
Contributor

@dduportal dduportal commented Jan 17, 2024

Fixup of #579

Twin of #580

The documentation at https://learn.microsoft.com/en-us/azure/load-balancer/load-balancer-outbound-connections#outboundrules states

Calculate ports per instance as follows:
Number of frontend IPs * 64K / Number of backend instances

Today, it means: 64000/44 = ~1454

(edit)

Fixes the following error from the apply of #579

Kubernetes Cluster Name: "privatek8s-emerging-ram"): managedclusters.ManagedClustersClient#CreateOrUpdate: Failure sending request: StatusCode=0 -- Original Error: Code="InvalidLoadBalancerProfileAllocatedOutboundPorts" Message="Load balancer profile allocated ports 1600 is not in an allowable range given the number of nodes and IPs provisioned. Total node count 50 requires 80000 ports but only 64000 ports are available given 1 outbound public IPs. Refer to https://aka.ms/aks/slb-ports for more details." Target="networkProfile.loadBalancerProfile.allocatedOutboundPorts"

@dduportal dduportal marked this pull request as ready for review January 17, 2024 14:09
@dduportal dduportal requested a review from a team January 17, 2024 14:09
Copy link
Collaborator

@smerle33 smerle33 left a comment

Choose a reason for hiding this comment

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

🚀

@dduportal dduportal marked this pull request as draft January 17, 2024 14:26
@dduportal dduportal marked this pull request as ready for review January 17, 2024 14:28
@dduportal dduportal merged commit af40e4c into main Jan 17, 2024
2 checks passed
@dduportal dduportal deleted the dduportal-patch-2 branch January 17, 2024 14:28
dduportal pushed a commit that referenced this pull request May 22, 2024
<Actions>
<action
id="296d75eab55b9d23bd1e94dc34cea43b964c29945c12fefcb674e3c068a0a767">
        <h3>Bump Terraform `random` provider version</h3>
<details
id="1d9343c012f5434ac9fe8a98135bae3667b399259be16d9b14302ea3bd424a24">
            <summary>Update Terraform lock file</summary>
<p>changes detected:&#xA;&#x9;&#34;hashicorp/random&#34; updated from
&#34;3.6.1&#34; to &#34;3.6.2&#34; in file
&#34;.terraform.lock.hcl&#34;</p>
            <details>
                <summary>3.6.2</summary>
<pre>Changelog retrieved
from:&#xA;&#x9;https://github.com/hashicorp/terraform-provider-random/releases/tag/v3.6.2&#xA;NOTES:&#xA;&#xA;*
resource/random_pet: Results have been updated to the latest upstream
petname data
([#581](https://github.com/hashicorp/terraform-provider-random/issues/581))&#xA;&#xA;&#xA;</pre>
            </details>
        </details>
<a
href="https://infra.ci.jenkins.io/job/updatecli/job/azure/job/main/193/">Jenkins
pipeline link</a>
    </action>
</Actions>

---

<table>
  <tr>
    <td width="77">
<img src="https://www.updatecli.io/images/updatecli.png" alt="Updatecli
logo" width="50" height="50">
    </td>
    <td>
      <p>
Created automatically by <a
href="https://www.updatecli.io/">Updatecli</a>
      </p>
      <details><summary>Options:</summary>
        <br />
<p>Most of Updatecli configuration is done via <a
href="https://www.updatecli.io/docs/prologue/quick-start/">its
manifest(s)</a>.</p>
        <ul>
<li>If you close this pull request, Updatecli will automatically reopen
it, the next time it runs.</li>
<li>If you close this pull request and delete the base branch, Updatecli
will automatically recreate it, erasing all previous commits made.</li>
        </ul>
        <p>
Feel free to report any issues at <a
href="https://github.com/updatecli/updatecli/issues">github.com/updatecli/updatecli</a>.<br
/>
If you find this tool useful, do not hesitate to star <a
href="https://github.com/updatecli/updatecli/stargazers">our GitHub
repository</a> as a sign of appreciation, and/or to tell us directly on
our <a
href="https://matrix.to/#/#Updatecli_community:gitter.im">chat</a>!
        </p>
      </details>
    </td>
  </tr>
</table>

Co-authored-by: Jenkins Infra Bot (updatecli) <60776566+jenkins-infra-bot@users.noreply.github.com>
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