volcano-sh / volcano

A Cloud Native Batch System (Project under CNCF)
https://volcano.sh
Apache License 2.0
4.12k stars 952 forks source link

JobFlow UT probabilistic failure #2851

Open wangyang0616 opened 1 year ago

wangyang0616 commented 1 year ago

What happened: After add jobflow controller code #2559 is integrated, the code verify of the backbone CI fails to run, which are goimports check and ut


**What you expected to happen**:
ut success
**How to reproduce it (as minimally and precisely as possible)**:
make unit-test

**Anything else we need to know?**:

**Environment**:
- Volcano Version: master
- Kubernetes version (use `kubectl version`):
- Cloud provider or hardware configuration:
- OS (e.g. from /etc/os-release):
- Kernel (e.g. `uname -a`):
- Install tools:
- Others:
wangyang0616 commented 1 year ago

In order to ensure that the integration of other PRs in the trunk is not affected, the case involving this issue is temporarily skipped, and the follow-up will be tracked and repaired through this issue.

For details, see: https://github.com/volcano-sh/volcano/blob/17370d7d6c3c992ef47eba77da45b5b24bc6215a/pkg/controllers/jobflow/jobflow_controller_action_test.go#L255-L258

lowang-bh commented 1 year ago

Becaulse of that there is no jobflow crd at all. See issue #2867

stale[bot] commented 1 year ago

Hello 👋 Looks like there was no activity on this issue for last 90 days. Do you mind updating us on the status? Is this still reproducible or needed? If yes, just comment on this PR or push a commit. Thanks! 🤗 If there will be no activity for 60 days, this issue will be closed (we can always reopen an issue if we need!).