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

Fix observer role registration #6037

Merged
merged 1 commit into from
Feb 6, 2025
Merged

Fix observer role registration #6037

merged 1 commit into from
Feb 6, 2025

Conversation

abukosek
Copy link
Contributor

@abukosek abukosek commented Feb 4, 2025

Closes #6019.

Copy link

netlify bot commented Feb 4, 2025

Deploy Preview for oasisprotocol-oasis-core canceled.

Name Link
🔨 Latest commit ec5fb33
🔍 Latest deploy log https://app.netlify.com/sites/oasisprotocol-oasis-core/deploys/67a4dbe80277f700080fb7f9

go/worker/client/worker.go Outdated Show resolved Hide resolved
@abukosek abukosek force-pushed the andrej/fix/observer-reg branch 4 times, most recently from cc55958 to 64d1512 Compare February 4, 2025 17:41
go/worker/client/worker.go Outdated Show resolved Hide resolved
@abukosek abukosek force-pushed the andrej/fix/observer-reg branch from 64d1512 to b9ba874 Compare February 4, 2025 18:07
@abukosek abukosek marked this pull request as ready for review February 4, 2025 18:07
@abukosek
Copy link
Contributor Author

abukosek commented Feb 4, 2025

OK, this should work now, please re-review :)

go/worker/client/worker.go Outdated Show resolved Hide resolved
go/worker/client/worker.go Outdated Show resolved Hide resolved
go/worker/client/worker.go Outdated Show resolved Hide resolved
go/worker/client/worker.go Outdated Show resolved Hide resolved
go/worker/client/worker.go Outdated Show resolved Hide resolved
@abukosek abukosek force-pushed the andrej/fix/observer-reg branch 2 times, most recently from 1fb5f44 to 1c34f91 Compare February 5, 2025 17:05
Copy link

codecov bot commented Feb 5, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 65.16%. Comparing base (9e138d8) to head (1c34f91).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #6037      +/-   ##
==========================================
- Coverage   65.57%   65.16%   -0.42%     
==========================================
  Files         635      635              
  Lines       64600    64610      +10     
==========================================
- Hits        42362    42103     -259     
- Misses      17378    17577     +199     
- Partials     4860     4930      +70     

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

Copy link
Member

@kostko kostko left a comment

Choose a reason for hiding this comment

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

Some nits, otherwise I think this looks good now.

go/worker/client/committee/node.go Outdated Show resolved Hide resolved
go/worker/client/committee/node.go Outdated Show resolved Hide resolved
go/worker/client/worker.go Outdated Show resolved Hide resolved
@abukosek abukosek force-pushed the andrej/fix/observer-reg branch from 1c34f91 to 3e602b4 Compare February 6, 2025 15:33
@abukosek abukosek force-pushed the andrej/fix/observer-reg branch from 3e602b4 to ec5fb33 Compare February 6, 2025 15:57
@abukosek abukosek merged commit 6609237 into master Feb 6, 2025
5 checks passed
@abukosek abukosek deleted the andrej/fix/observer-reg branch February 6, 2025 17:37
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.

Observer role registration either fails or is incomplete when multiple runtimes are enabled
3 participants