-
Notifications
You must be signed in to change notification settings - Fork 13
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
TypeError: '<' not supported between instances of 'NoneType' and 'str' #161
Comments
Hi @shealym, I've been looking into this issue and I haven't been able to reproduce the error you're seeing. To start, would you mind trying to run the deployment-report again? I just want to make sure there wasn't some temporary resource causing an issue that has since been cleaned up. If you are still seeing the issue after running the report again, we can try to narrow down the cause. Thanks! |
Hey Josh, I re-cloned the code again today, but I'm still getting the issue every run. Any debug I can put in on this? Thanks for the help! |
Hey @shealym, unfortunately there isn't a surefire debug option that would catch the issue. It stems from some key in the dictionary of data that the report collects being empty. It could be a key the report generates, but it could also just be something that's returned from K8s so we wouldn't be able to effectively detect it. There is a fairly simple modification to the code that can be made to help generate something that can be evaluated. On line 705 in data_json = json.dumps(self._report_data, cls=KubernetesObjectJSONEncoder, indent=4, sort_keys=True) to: data_json = json.dumps(self._report_data, cls=KubernetesObjectJSONEncoder, indent=4, sort_keys=False) Once the file is updated, you can rerun the report with the python3 viya-ark.py deployment-report -n devtech-viya-ns -d Once that file has been generated, a grep for |
Hey Josh, it's in this section:
Thanks! |
Hi @shealym, thank you for capturing that information. That is perfect. I'm going to dig into this today and I'll let you know when I have something that should work for this issue. Thanks again! |
Hi @shealym, sorry for the delay. I've pushed a change to a development branch that I'm hoping will resolve the issue you're seeing. Because I wasn't able to reproduce this issue, I was hoping you could run a test and make sure the issue is no longer encountered. When you have a chance, please pull or re-clone this project, checkout the development branch, |
Hey Josh, apologies for the late reply to this. I pulled the development branch earlier today, and the Deployment Report ran successfully. Thanks for the help! |
This Issue is addressed in Release 1.8.0 |
This seems related to #74 and #92 ... I'm getting the same error when trying to generate the Deployment Report, even if I repeat the command over and over:
The Pre-Install Requirements report runs without issue. Thoughts? Thanks!
The text was updated successfully, but these errors were encountered: