Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Calculate acceptance test operator balance from USD #8957

Merged
merged 1 commit into from
Aug 6, 2024

Conversation

steven-sheehy
Copy link
Member

Description:

  • Add a feature flag for rejectToken so mirror node can deploy before services 0.52
  • Change operatorBalance property to store amount in USD
  • Change operator initial balance to convert USD into tinybar using transactionReceipt.ExchangeRate or falling back to /api/v1/network/exchangerate
  • Fix assertion not printing out correct contract call response
  • Fix regression caused by null immutable key
  • Fix startup probe topic not being deleted
  • Fix startup probe taking forever to recover when all nodes are down

Related issue(s):

Fixes #8953

Notes for reviewer:

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

Signed-off-by: Steven Sheehy <steven.sheehy@swirldslabs.com>
@steven-sheehy steven-sheehy added enhancement Type: New feature test Test infrastructure, automated tests required, etc labels Aug 6, 2024
@steven-sheehy steven-sheehy added this to the 0.112.0 milestone Aug 6, 2024
@steven-sheehy steven-sheehy requested a review from a team August 6, 2024 20:19
@steven-sheehy steven-sheehy self-assigned this Aug 6, 2024
Copy link

sonarcloud bot commented Aug 6, 2024

Copy link

codecov bot commented Aug 6, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.20%. Comparing base (beb49fa) to head (dca28cd).

Additional details and impacted files
@@            Coverage Diff            @@
##               main    #8957   +/-   ##
=========================================
  Coverage     92.20%   92.20%           
  Complexity     7624     7624           
=========================================
  Files           927      927           
  Lines         30503    30503           
  Branches       3721     3721           
=========================================
  Hits          28126    28126           
  Misses         1550     1550           
  Partials        827      827           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Collaborator

@xin-hedera xin-hedera left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@jnels124 jnels124 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@steven-sheehy steven-sheehy merged commit f72fe73 into main Aug 6, 2024
29 checks passed
@steven-sheehy steven-sheehy deleted the 8953-fiat-operator branch August 6, 2024 20:46
jnels124 pushed a commit that referenced this pull request Aug 6, 2024
* Add a feature flag for rejectToken so mirror node can deploy before services 0.52
* Change `operatorBalance` property to store amount in USD
* Change operator initial balance to convert USD into tinybar using `transactionReceipt.ExchangeRate` or falling back to `/api/v1/network/exchangerate`
* Fix assertion not printing out correct contract call response
* Fix regression caused by null immutable key
* Fix startup probe topic not being deleted
* Fix startup probe taking forever to recover when all nodes are down

Signed-off-by: Steven Sheehy <steven.sheehy@swirldslabs.com>
(cherry picked from commit f72fe73)
jnels124 pushed a commit that referenced this pull request Aug 6, 2024
* Add a feature flag for rejectToken so mirror node can deploy before services 0.52
* Change `operatorBalance` property to store amount in USD
* Change operator initial balance to convert USD into tinybar using `transactionReceipt.ExchangeRate` or falling back to `/api/v1/network/exchangerate`
* Fix assertion not printing out correct contract call response
* Fix regression caused by null immutable key
* Fix startup probe topic not being deleted
* Fix startup probe taking forever to recover when all nodes are down

Signed-off-by: Steven Sheehy <steven.sheehy@swirldslabs.com>
(cherry picked from commit f72fe73)
Signed-off-by: Jesse Nelson <jesse@hashgraph.com>
steven-sheehy added a commit that referenced this pull request Aug 6, 2024
Calculate acceptance test operator balance from USD (#8957)

* Add a feature flag for rejectToken so mirror node can deploy before services 0.52
* Change `operatorBalance` property to store amount in USD
* Change operator initial balance to convert USD into tinybar using `transactionReceipt.ExchangeRate` or falling back to `/api/v1/network/exchangerate`
* Fix assertion not printing out correct contract call response
* Fix regression caused by null immutable key
* Fix startup probe topic not being deleted
* Fix startup probe taking forever to recover when all nodes are down


(cherry picked from commit f72fe73)

Signed-off-by: Steven Sheehy <steven.sheehy@swirldslabs.com>
Signed-off-by: Jesse Nelson <jesse@hashgraph.com>
Co-authored-by: Steven Sheehy <17552371+steven-sheehy@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Type: New feature test Test infrastructure, automated tests required, etc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Calculate acceptance operator balance from fiat
3 participants