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

Ability to filter Ipv4 per ENIs #144

Open
ghost opened this issue Jul 15, 2022 · 2 comments
Open

Ability to filter Ipv4 per ENIs #144

ghost opened this issue Jul 15, 2022 · 2 comments
Labels
enhancement New feature or request good first issue Good for newcomers

Comments

@ghost
Copy link

ghost commented Jul 15, 2022

When we want to calculate the max pods in the EKS case of an instance type we need to know both the number of ENIs and the number of Ip per ENIs.

Actually we need to do the check manually in order not to have problems in the EKS clusters.

@bwagner5 bwagner5 added the enhancement New feature or request label Jul 15, 2022
@bwagner5
Copy link
Contributor

Good feature request! We can definitely get this into ec2-instance-selector:

For reference: (.NetworkInfo.Ipv4AddressesPerInterface and .NetworkInfo.Ipv6AddressesPerInterface)

    {
        "AutoRecoverySupported": true,
        "BareMetal": false,
        "BurstablePerformanceSupported": false,
        "CurrentGeneration": true,
        "DedicatedHostsSupported": true,
        "EbsInfo": {
            "EbsOptimizedInfo": {
                "BaselineBandwidthInMbps": 650,
                "BaselineIops": 3600,
                "BaselineThroughputInMBps": 81.25,
                "MaximumBandwidthInMbps": 4750,
                "MaximumIops": 18750,
                "MaximumThroughputInMBps": 593.75
            },
            "EbsOptimizedSupport": "default",
            "EncryptionSupport": "supported",
            "NvmeSupport": "required"
        },
        "FpgaInfo": null,
        "FreeTierEligible": false,
        "GpuInfo": null,
        "HibernationSupported": true,
        "Hypervisor": "nitro",
        "InferenceAcceleratorInfo": null,
        "InstanceStorageInfo": null,
        "InstanceStorageSupported": false,
        "InstanceType": "m5.large",
        "MemoryInfo": {
            "SizeInMiB": 8192
        },
        "NetworkInfo": {
            "DefaultNetworkCardIndex": 0,
            "EfaInfo": null,
            "EfaSupported": false,
            "EnaSupport": "required",
            "EncryptionInTransitSupported": false,
            "Ipv4AddressesPerInterface": 10,
            "Ipv6AddressesPerInterface": 10,
            "Ipv6Supported": true,
            "MaximumNetworkCards": 1,
            "MaximumNetworkInterfaces": 3,
            "NetworkCards": [
                {
                    "MaximumNetworkInterfaces": 3,
                    "NetworkCardIndex": 0,
                    "NetworkPerformance": "Up to 10 Gigabit"
                }
            ],
            "NetworkPerformance": "Up to 10 Gigabit"
        },
        "PlacementGroupInfo": {
            "SupportedStrategies": [
                "cluster",
                "partition",
                "spread"
            ]
        },
        "ProcessorInfo": {
            "SupportedArchitectures": [
                "x86_64"
            ],
            "SustainedClockSpeedInGhz": 3.1
        },
        "SupportedBootModes": [
            "legacy-bios",
            "uefi"
        ],
        "SupportedRootDeviceTypes": [
            "ebs"
        ],
        "SupportedUsageClasses": [
            "on-demand",
            "spot"
        ],
        "SupportedVirtualizationTypes": [
            "hvm"
        ],
        "VCpuInfo": {
            "DefaultCores": 1,
            "DefaultThreadsPerCore": 2,
            "DefaultVCpus": 2,
            "ValidCores": [
                1
            ],
            "ValidThreadsPerCore": [
                1,
                2
            ]
        },
        "OndemandPricePerHour": 0.096,
        "SpotPrice": 0.02064770118269803
    }

@bwagner5 bwagner5 added the good first issue Good for newcomers label Nov 20, 2022
@hassaanakram
Copy link

Hey @bwagner5
I'd like to take a stab at it. I've gone through the relevant code, and I have a fair idea about the implementation. However, I'll need some pointers for writing test cases.

Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

2 participants