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

Remove WorkflowData API #650

Merged

Conversation

chrisdoherty4
Copy link
Member

The WorkflowData API relates to multi worker workflows that aren't implemented in the Kube backend. With the move to the Kube backend we're taking the opportunity to reset and implement this sanely when its needed, not before.

@chrisdoherty4 chrisdoherty4 force-pushed the feat/remove-workflow-data branch from 56f3cee to 08552a8 Compare November 11, 2022 23:21
@codecov
Copy link

codecov bot commented Nov 11, 2022

Codecov Report

Merging #650 (527294f) into main (9df0dad) will decrease coverage by 0.03%.
The diff coverage is n/a.

❗ Current head 527294f differs from pull request most recent head 6d7dea7. Consider uploading reports for the commit 6d7dea7 to get more accurate results

@@            Coverage Diff             @@
##             main     #650      +/-   ##
==========================================
- Coverage   45.09%   45.06%   -0.04%     
==========================================
  Files          62       61       -1     
  Lines        3559     3557       -2     
==========================================
- Hits         1605     1603       -2     
  Misses       1872     1872              
  Partials       82       82              

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@chrisdoherty4 chrisdoherty4 force-pushed the feat/remove-workflow-data branch 2 times, most recently from 582a594 to b9a72a1 Compare November 12, 2022 00:05
@chrisdoherty4 chrisdoherty4 force-pushed the feat/remove-workflow-data branch from f0bba8f to 35dc781 Compare November 12, 2022 00:10
@chrisdoherty4 chrisdoherty4 marked this pull request as draft November 14, 2022 18:00
@chrisdoherty4 chrisdoherty4 force-pushed the feat/remove-workflow-data branch from f11fa40 to 56f3cee Compare November 18, 2022 02:09
ci-checks.sh Outdated Show resolved Hide resolved
@chrisdoherty4 chrisdoherty4 force-pushed the feat/remove-workflow-data branch from 527294f to a459ded Compare November 18, 2022 19:27
The WorkflowData API relates to multi worker workflows that aren't
implemented in the Kube backend. With the move to the Kube backend we're
taking the opportunity to reset and implement this sanely when its
needed, not before.

Signed-off-by: Chris Doherty <chris.doherty4@gmail.com>
@chrisdoherty4 chrisdoherty4 force-pushed the feat/remove-workflow-data branch from a459ded to 6d7dea7 Compare November 18, 2022 19:27
@chrisdoherty4 chrisdoherty4 marked this pull request as ready for review November 18, 2022 19:34
@chrisdoherty4 chrisdoherty4 removed the request for review from mmlb November 18, 2022 20:22
@chrisdoherty4 chrisdoherty4 added the ready-to-merge Signal to Mergify to merge the PR. label Nov 21, 2022
@chrisdoherty4 chrisdoherty4 merged commit fe2177f into tinkerbell:main Nov 21, 2022
@chrisdoherty4 chrisdoherty4 deleted the feat/remove-workflow-data branch November 21, 2022 13:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge Signal to Mergify to merge the PR.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants