initiallyqq opened a new pull request, #177:
URL: https://github.com/apache/dolphinscheduler-sdk-python/pull/177

   … fix Kubernetes.yaml example
   
   The YAML workflow loader resolved task types by class name only, so 
task_type: K8S (TaskType.KUBERNETES) failed with
   PyDSTaskNoFoundException: cant not find task K8S. The shipped 
Kubernetes.yaml example also used camelCase fields rejected by the Kubernetes 
task constructor, and was the only example excluded from CI.
   
   - yaml_workflow.py: add _TASK_TYPE_ALIAS fallback so K8S resolves to the 
Kubernetes class; class-name matching is still tried first.
   - Kubernetes.yaml: minCpuCores/minMemorySpace -> snake_case.
   - test_yaml_workflow.py: cover K8S/k8s/Kubernetes in test_get_task and add 
Kubernetes.yaml to test_get_create_workflow.
   
   closes #176
   
   <!--Thanks for you contribute to Apache DolphinScheduler Python API, You can 
see more detail about contributing in 
https://github.com/apache/dolphinscheduler-sdk-python/DEVELOP.md .-->
   
   ## Brief Summary of The Change
   
   <!--Please include `fixes: #XXXX(ISSUE_NUMBER)` to automatically close any 
corresponding issue when the pull request is merged. Alternatively if not fully 
closed you can say `related: #XXXX(ISSUE_NUMBER)`.-->
   
   ## Pull Request checklist
   
   I confirm that the following checklist has been completed.
   
   - [ ] Add/Change **test cases** for the changes.
   - [ ] Add/Change the related **documentation**, should also change 
`docs/source/config.rst` when you change file `default_config.yaml`.
   - [ ] (Optional) Add your change to `UPDATING.md` when it is an incompatible 
change.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to