This repository was archived by the owner on Sep 20, 2023. It is now read-only.
[DO NOT MERGE] FluxMonitor implementation #100
Draft
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Initial FluxMonitor implementation * Distinct new vs old interfaces * Polling and median working * Deviation checking * Working with channels * Job Spec parsing working + values being passed to FM service * FM Spec persisted on DB * Sending Job triggers on every answer update * Log fixes and job run returning result * Cleanup and Substrate changes (#96) * Substrate updates (#97) * Fix decoding issues * Use new structs * Simplify decoding * Adapter timeout set. General Event for Job run. Testing Stop() method of FM * Restore cleaned files, but make sure they're not used (#98) * Testing with continuous changes in FA state * Distinct blockchain events * Clean up failing tests * FM implementation updates (#99) * Clean up duplicated definitions * More cleanup * Connect Substrate to FM service and subscriber * Fix cyclic import * Remove references of old interfaces * Fix import order * Make sure type cast is error-handled * Remove outdated test Co-authored-by: Evangelos Barakos <e.mparakos@gmail.com>
* Get job run payload from Blockchain manager * Remove outdated integrations * Expect new type
* Clean up FM service and references * Remove test until it can be refactored * More cleanup
ebarakos
force-pushed
the
feature/fluxmonitor
branch
4 times, most recently
from
1e6d734 to
5b23662
Compare
Co-authored-by: Jonas Hals <contact@jonashals.me>
* Progress integration of WS subscriber * Update mock subscriber
* Disable ONT due to dependency issues * When initiating a new round, the round ID should increase * Don't consider round 0 as submitted * Move job run listener to before FM starts
* Misc cleanup * Prevent shadowing declaration
Co-authored-by: Jonas Hals <contact@jonashals.me> Co-authored-by: boxhock <jonas@smartcontract.com>
…ys (#128) * Update Substrate implementation to use new pallet updates * Add static "adapter" to mock client * Partially prevent race condition on Chainlink node * FluxMonitor should send the requestData params instead of specific keys
* Fix FluxMonitor check if it can submit to new round * Fix Substrate event decoding
* Improve locking around polling and add retries * Fix issues with reflect.DeepEqual false negatives * Make FM adapter retries configurable * Potential change (#131) * Fix naming of new config vars * FM logger w/jobid and updated heartbeat resetting * Subscribe to jobs without pointer to avoid value being overwritten * Add job name to test FM Co-authored-by: Evangelos Barakos <evangelos@smartcontract.com>
* Return Substrate runlog service * FM integration test now expects 2 runs * Revert the change of the Event type * Cleanup * Update tests * Cleanup
* Return Substrate runlog service * Revert the change of the Event type * Cleanup * Refactor subscriber and BM managers * Modify other implementations to make project compile * More refactoring, mock client changes * Remove outdated tests * Fix failing tests * Prevent exhaustive cases * Avoid use of dot-imports * Cleanup * Increase test coverage * Start service directly instead of making a starter type * Make updated FM validation use the new spec path * Only use fluxmonitor config object when parsing it
* Return Substrate runlog service * Revert the change of the Event type * Cleanup * Fix failing tests * Refactor tier 2 * Make errors from saving the subscription more clear * Use correct Params struct * Finish CFX refactor * Refactor outdated tests * Remove unnecessary CFX static file * Fix old failing tests * Refactor eth manager * Refactor Harmony integration
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.