Elasticsearch-dsl refactor, unit test, config update

[shashank-boyapally]

Type of change

• Refactor
• New feature
• Bug fix
• Optimization
• Documentation Update

Description

I have integrated elasticsearch-dsl library such that there is better code readabilty and more abstraction to our queries. One change included in the following iteration is regarding config files which are sent by orion.

metrics : 
    - name:  podReadyLatency
      metricName: podLatencyQuantilesMeasurement
      quantileName: Ready
      metric_of_interest: P99
      not: 
      - jobConfig.name: "garbage-collection"

the above config when translated gives a "not" list, at the moment this list is a list of dictionaries, with each dictionary having only one key:value pair. The change implemented now takes in the not list as a dictionary with multiple key:value pairs

metrics : 
    - name:  podReadyLatency
      metricName: podLatencyQuantilesMeasurement
      quantileName: Ready
      metric_of_interest: P99
      not: 
          jobConfig.name: "garbage-collection"

Related Tickets & Documents

• Related Issue [RFE] Use elasticsearch-dsl in fmatch #17
• Closes #

Checklist before requesting a review

• I have performed a self-review of my code.
• If it is a core feature, I have added thorough tests.

Testing

• Please describe the System Under Test.
• Unit tests coverage of 100%
• Please provide detailed steps to perform tests related to this code change.
• How were the fix/results from this change verified? Please provide relevant screenshots or results.

[jtaleric]

minor nits in the test script, but overall looks good.

[jtaleric]

lgtm

[paigerube14]

didnt have chance to fully test, things look good to me from a code level