intel / CPU-Manager-for-Kubernetes

Kubernetes Core Manager for NFV workloads
Apache License 2.0
187 stars 83 forks source link

fix isolate.py #291

Closed kslee closed 2 years ago

kslee commented 3 years ago

isolate.py is currently broken as it attempts to create config without namespace

Traceback (most recent call last): File "./cmk.py", line 216, in main() File "./cmk.py", line 151, in main isolate.isolate(args["--pool"], File "/cmk/intel/isolate.py", line 151, in isolate c = config.Config(configmap_name, pod_name) TypeError: init() missing 1 required positional argument: 'cm_namespace'

adpempx commented 3 years ago

Hi,

there is a similar issue in nodereport, where you are using config.Config() without specify the namespace.

Thanks

DheerajDang commented 3 years ago

I have tested this change, it is working fine.