Skip to content
This repository has been archived by the owner on Jul 22, 2020. It is now read-only.

Mapper package refactoring #216

Merged
merged 5 commits into from
Jan 22, 2018
Merged

Mapper package refactoring #216

merged 5 commits into from
Jan 22, 2018

Commits on Jan 22, 2018

  1. Refactor mapper package responsibilities 

    Mapper should only take input, decode it and map onto internal model instances
    @prymitive
    prymitive committed Jan 22, 2018
    Configuration menu
    Copy the full SHA
    0b30a23 View commit details
    Browse the repository at this point in the history

  2. Refactor transport package 

    With this change we'll initialize Transport object for each Alertmanager and just call Read() on it when we need to use this transport to read from upstream Alertmanager
    @prymitive
    prymitive committed Jan 22, 2018
    Configuration menu
    Copy the full SHA
    2cf9253 View commit details
    Browse the repository at this point in the history

  3. Finish refactoring mapper package 

    Fix bugs, add docstrings and let each mapper give us full url since it doesn't handle any http requests now (it just maps text to objects now)
    @prymitive
    prymitive committed Jan 22, 2018
    Configuration menu
    Copy the full SHA
    6124196 View commit details
    Browse the repository at this point in the history

  4. Migrate rest of the code to new mapper and transport packages

    @prymitive
    prymitive committed Jan 22, 2018
    Configuration menu
    Copy the full SHA
    1f89ba0 View commit details
    Browse the repository at this point in the history

  5. Drop Release() from SilenceMapper 

    Not used
    @prymitive
    prymitive committed Jan 22, 2018
    Configuration menu
    Copy the full SHA
    c7fb8db View commit details
    Browse the repository at this point in the history