Uploaded image for project: 'PuppetDB'
  1. PuppetDB
  2. PDB-800

Move /metric endpoint to prepare for modularization

    Details

    • Type: Task
    • Status: Closed
    • Priority: Normal
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: PDB 3.0.0
    • Component/s: None
    • Labels:
      None
    • Template:
    • Story Points:
      2
    • Sprint:
      PuppetDB 2015-01-14, PuppetDB 2015-01-28

      Description

      The metrics application should be a TK service in the future, we should move it out of the main URL path to prepare for this.

      One idea is to move it into a place that might be a permanent location for other generic http services, considerations for the future of where the main api then lives is also worthwhile. Not to mention that metrics itself probably should be versioned for the future.

      As a strawman something like this perhaps:

      pdb/v4/...
      metrics/v1/mbeans/...

      But we need to take into account other API evolutions, like where does the sync code live, or non-core endpoints that extend PuppetDB in the future etc. etc.

      Perhaps we ponder a more complex layout to handle this:

      pdb/query/v4/...
      pdb/commands/v1/...
      pdb/sync/v1/...
      pdb/pe/v1/...
      metrics/v1/...

      Anyway, interesting to ponder this problem. Our ultimate goal is to separate out the metrics tool into its own re-usable service for other TK services, so ultimately as long as we don't constrain ourselves from this goal, we are good.

      Another consideration, is that perhaps just moving the endpoint is an unnecessary intermediary step, and perhaps its only mildly harder to just make the TK service. Hohum.

        Attachments

          Activity

            jsd-sla-details-panel

              People

              • Assignee:
                andrew.roetker AJ Roetker
                Reporter:
                ken Ken Barber
              • Votes:
                0 Vote for this issue
                Watchers:
                1 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: