Skip to content

Review tags for elevation to the core spec #206

Closed

Description

@j616

Context
Tags are often used in TAMS to explore features before committing to baking them into the spec. Some tags in this class have been in use for some time now (e.g. flow_status). We should review our tags listing and elevate tags to the core spec where appropriate.

Actions

  • Review tags for potential elevation to the core spec
  • Write and ADR documenting the decision process for candidates for elevation
  • Review this with the TSC, and wider community as appropriate
  • Make the relevant spec changes, and changes to AppNote0003

Outcome
Tags have been reviewed for elevation to the core spec, and elevated as appropriate.

Activity

  1. added theissue type on May 28, 2026
  2. added
    tsc-discussDiscuss this issue at the next TSC meeting
    on May 28, 2026
  3. johnbilt commented on May 29, 2026

    @johnbilt
    Collaborator

    The flow_status is a good candidate for elevation as there are now multiple solutions using this to detect whether a flow is currently ingesting or closed. There are a few interesting extensions on this one that probably should be discussed including whether the store should actively do anything with this tag (eg close it after a period of time) or leave for the ingest tool to deal with. Also there has been request to roll the status up from the flows to an overall status at the top source level. So needs scoping and discussion and may be worth creating it's own ticket

  4. moved this from Triage to Backlog in TAMS Community Planningon May 29, 2026
  5. j616 commented on May 29, 2026

    @j616
    ContributorAuthor

    Another consideration on flow_status is where an ingester is being swapped out without interruption of the ingest. The clean shutdown of the initial ingester might currently result in a closed_complete. One option would be to drop the status back to awaiting_content. Or we could consider adding a further state to aid in that handover.

  6. removed
    tsc-discussDiscuss this issue at the next TSC meeting
    on Jun 12, 2026
  7. moved this from Backlog to In Progress in TAMS Community Planningon Jul 10, 2026
  8. self-assigned this
    on Jul 10, 2026
  9. moved this from In Progress to In review in TAMS Community Planningon Jul 10, 2026
  10. linked a pull request that will close this issueTag status review #227on Jul 14, 2026
  11. samdbmg commented on Jul 24, 2026

    @samdbmg
    Member

    TSC 2026-05-24
    JS: Finally got to going through this: reviewed in #227 and confirmed everything is (nearly) the right status - remaining ones tidied up in that PR.
    JB: Is flow status a separate PR or is that in here?
    JS: It's in here
    JS: Also added some wording into description pointing at the flow transform appnote on how you may want to automate managing Flow lifecycle, but it isn't in the core spec
    SMG: Can we merge this now?
    DJ: No dedicated endpoint to change it. Don't like having dedicated endpoint to change it, but the frequency of change might justify it?
    JS: I'm ok with that.

    Agreed to merge once changes reviewed

  12. removed
    tsc-discussDiscuss this issue at the next TSC meeting
    on Jul 24, 2026
  13. moved this from In review to Ready to merge in TAMS Community Planningon Aug 7, 2026
  14. moved this from Ready to merge to Done in TAMS Community Planningon Aug 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

Projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions