Uploaded image for project: 'Data Management'
  1. Data Management
  2. DM-34699

Patch the ap_pipe config reset hack

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Done
    • Resolution: Done
    • Fix Version/s: None
    • Component/s: ap_pipe
    • Labels:
      None
    • Story Points:
      2
    • Sprint:
      AP S22-6 (May)
    • Team:
      Alert Production
    • Urgent?:
      No

      Description

      While investigating why the solution to DM-34254 only needs config overrides in the root ApPipe pipeline, I discovered that the hack I'd introduced in DM-31063 blocks its own config settings, not the obs package config's. This is not what we want for cleaning up the AP pipeline configuration.

      Find an improved version of the hack that actually resets the pipeline's config. Preliminary experiments have gotten good results with Config.loadFromString and Config.saveToString, but I have not yet tested how this works with configs that may require imports or otherwise alter nested configs.

        Attachments

          Issue Links

            Activity

            Hide
            krzys Krzysztof Findeisen added a comment -

            Hi John Parejko, since this issue is relevant to both DM-31063 and DM-34254, would you be willing to review it? It's 70 lines total.

            Show
            krzys Krzysztof Findeisen added a comment - Hi John Parejko , since this issue is relevant to both DM-31063 and DM-34254 , would you be willing to review it? It's 70 lines total.
            Hide
            Parejkoj John Parejko added a comment -

            I'm not seeing the PRs for this linked?

            Show
            Parejkoj John Parejko added a comment - I'm not seeing the PRs for this linked?
            Hide
            krzys Krzysztof Findeisen added a comment -
            Show
            krzys Krzysztof Findeisen added a comment - It's ap_pipe#108 .

              People

              Assignee:
              krzys Krzysztof Findeisen
              Reporter:
              krzys Krzysztof Findeisen
              Reviewers:
              John Parejko
              Watchers:
              Eric Bellm, Ian Sullivan, John Parejko, Krzysztof Findeisen
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:

                  Jenkins

                  No builds found.