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.

Sign up for GitHub

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

Jump to bottom

Add TEP-0044: Decouple Task Composition from Scheduling ➰ #316

Merged
merged 5 commits into from
Mar 16, 2021
Merged

Add TEP-0044: Decouple Task Composition from Scheduling ➰ #316

merged 5 commits into from
Mar 16, 2021

Commits on Mar 16, 2021

  1. Add TEP-0044: Composing Tasks with Tasks ➰ 

    This TEP describes some difficulty folks have run into, especially when
trying to use Tekton Pipelines without PipelineResources: if you want to
use the functionality of multiple Tasks, you need to put them together
in a Pipeline and share data between them with volumes. It could be a
useful abstraction, and more efficient, to make it possible to combine
Tasks together without scheudling them on different pods.
    @bobcatfish
    bobcatfish committed Mar 16, 2021
    Configuration menu
    Copy the full SHA
    dbd8c70 View commit details
    Browse the repository at this point in the history

  2. Move the TaskRun use case to non-goals ⚽ 

    In the review @vdemeester pointed out that it feels like we are trying
to deal with 2 different things - for me the main reason for that is b/c
I have a feeling we WILL solve @mattmoor 's use case - since we'll
probably have to change TaskRuns to make it happen - but I agree with
@vdemeester that that's not the main thing we're targetting here.

Now TEP-0044 is more aligned with TEP-0046 than ever!
    @bobcatfish
    bobcatfish committed Mar 16, 2021
    Configuration menu
    Copy the full SHA
    15b51f9 View commit details
    Browse the repository at this point in the history

  3. Add comparison with TEP-0046 🔎 

    In the most recent API working group we decided to keep this TEP and
[TEP-0046](tektoncd#318) separate
because they are coming at a similar problem from different angles.

In the WG @jerop suggested that we update the TEPs with some info on
what the overlaps + differences are and that's what this TEP is adding!
    @bobcatfish
    bobcatfish committed Mar 16, 2021
    Configuration menu
    Copy the full SHA
    516f776 View commit details
    Browse the repository at this point in the history

  4. Rename TEP to "Decouple Task Composition from Scheduling" ✍️ 

    In today's API working group it was clear that I still haven't been able
to articulate what problem this TEP is trying to solve in a way that's
clear, so I'm trying this composition vs scheduling approach instead to
see if that is more clear. Also @skaegi pointed out that the overhead
required in running multiple pods is another consideration in addition
to the PVC overhead so I've tried to include this as well.

I'm going to follow up with a list of possible solutions so hopefully
that will help folks who are confused about what this is trying to
address.
    @bobcatfish
    bobcatfish committed Mar 16, 2021
    Configuration menu
    Copy the full SHA
    72d88c0 View commit details
    Browse the repository at this point in the history

  5. Start brainstorming options 🧠 

    This commit adds possible solutions to the problem described in
TEP-0044, including references to solutions in other TEPS (46 & 54).
I was hoping to merge the problem statement before starting to talk
about solutions but it seems like the problem statement is too abstract
to get enough traction, and meanwhile folks have been opening more TEPs
with related proposals in the meantime, so hopefully starting to list
the options here will help us move the discussion forward.

I'm hoping we can merge the problem + possible options without needing
to decide on which one(s) we want to pursue.
    @bobcatfish
    bobcatfish committed Mar 16, 2021
    Configuration menu
    Copy the full SHA
    a509831 View commit details
    Browse the repository at this point in the history