Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@arinda-arif
Copy link
Contributor

Resolving #490

@coveralls
Copy link

coveralls commented Aug 2, 2022

Pull Request Test Coverage Report for Build 2780310876

  • 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
  • 2 unchanged lines in 1 file lost coverage.
  • Overall coverage decreased (-0.02%) to 77.66%

Files with Coverage Reduction New Missed Lines %
ext/notify/pagerduty/pagerdutynotifier.go 2 93.75%
Totals Coverage Status
Change from base Build 2779521532: -0.02%
Covered Lines: 7700
Relevant Lines: 9915

💛 - Coveralls

@arinda-arif arinda-arif changed the base branch from main to release-v0.2.6 August 2, 2022 07:46
@arinda-arif arinda-arif force-pushed the change-cross-optimus-sensor-name branch from d9ec72d to 9607890 Compare August 2, 2022 07:55
@arinda-arif arinda-arif merged commit 5b3896d into release-v0.2.6 Aug 2, 2022
@arinda-arif arinda-arif deleted the change-cross-optimus-sensor-name branch August 2, 2022 09:56
sravankorumilli pushed a commit that referenced this pull request Aug 5, 2022
…l sensor task id (#491)

* refactor: change external optimus sensor task name similar with internal sensor name

* fix: cross optimus task dependencies in airflow is not properly

* refactor: revert cross optimus sensor variable name

Co-authored-by: Anwar Hidayat <[email protected]>
arinda-arif added a commit that referenced this pull request Aug 9, 2022
* refactor: change external optimus sensor task id similar with internal sensor task id (#491)

* refactor: change external optimus sensor task name similar with internal sensor name

* fix: cross optimus task dependencies in airflow is not properly

* refactor: revert cross optimus sensor variable name

Co-authored-by: Anwar Hidayat <[email protected]>

* fix: bug caused by error when getting dependencies from external resource manager

* fix: bug on unknown dependencies being updated

Co-authored-by: Arinda Arif <[email protected]>
arinda-arif added a commit that referenced this pull request Aug 9, 2022
* refactor: change external optimus sensor task id similar with internal sensor task id (#491)

* refactor: change external optimus sensor task name similar with internal sensor name

* fix: cross optimus task dependencies in airflow is not properly

* refactor: revert cross optimus sensor variable name

Co-authored-by: Anwar Hidayat <[email protected]>

* fix: bug caused by error when getting dependencies from external resource manager

* fix: bug on unknown dependencies being updated

Co-authored-by: Arinda Arif <[email protected]>
sravankorumilli pushed a commit that referenced this pull request Aug 9, 2022
* refactor: change external optimus sensor task id similar with internal sensor task id (#491)

* refactor: change external optimus sensor task name similar with internal sensor name

* fix: cross optimus task dependencies in airflow is not properly

* refactor: revert cross optimus sensor variable name

Co-authored-by: Anwar Hidayat <[email protected]>

* fix: bug caused by error when getting dependencies from external resource manager

* fix: bug on unknown dependencies being updated

Co-authored-by: Arinda Arif <[email protected]>

Co-authored-by: Anwar Hidayat <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Preserve history of Airflow sensor task by having the same sensor name as internal sensor task

6 participants