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

[BUG] EKS Module Deployment Issues #574

Open
malachi-constant opened this issue Aug 28, 2024 · 1 comment
Open

[BUG] EKS Module Deployment Issues #574

malachi-constant opened this issue Aug 28, 2024 · 1 comment
Assignees
Labels
bug Something isn't working

Comments

@malachi-constant
Copy link
Contributor

Describe the bug
Deploying the EKS Module yields an error

[Container] 2024/08/27 21:52:08.217370 Running command cdk deploy --require-approval never --progress events --app "python app.py" --outputs-file ./cdk-exports.json
--
626 | Traceback (most recent call last):
627 | File "/codebuild/output/src967942608/src/bundle/module/app.py", line 57, in <module>
628 | stack = Eks(
629 | ^^^^
630 | File "/root/.venv/lib/python3.11/site-packages/jsii/_runtime.py", line 118, in __call__
631 | inst = super(JSIIMeta, cast(JSIIMeta, cls)).__call__(*args, **kwargs)
632 | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
633 | File "/codebuild/output/src967942608/src/bundle/module/stack.py", line 215, in __init__
634 | self._create_managed_node_group(eks_cluster, eks_version, ng, node_capacity_type, vpc_cni_chart)
635 | File "/codebuild/output/src967942608/src/bundle/module/stack.py", line 452, in _create_managed_node_group
636 | taints=[
637 | ^
638 | File "/codebuild/output/src967942608/src/bundle/module/stack.py", line 456, in <listcomp>
639 | effect=eks.TaintEffect(taint.get("effect").upper()),
640 | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
641 | File "/root/.pyenv/versions/3.11.9/lib/python3.11/enum.py", line 714, in __call__
642 | return cls.__new__(cls, value)
643 | ^^^^^^^^^^^^^^^^^^^^^^^
644 | File "/root/.pyenv/versions/3.11.9/lib/python3.11/enum.py", line 1137, in __new__
645 | raise ve_exc
646 | ValueError: 'NOSCHEDULE' is not a valid TaintEffect

I believe this value should be NO_SCHEDULE with an underscore from the documentation. Please investigate.

To Reproduce
Deploy the latest version of the ml-training-on-eks manifest.

@malachi-constant malachi-constant added the bug Something isn't working label Aug 28, 2024
@srinivasreddych
Copy link
Contributor

I will be looking into it today

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants