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

Add feature to disable code upgrades #1906

Closed
wants to merge 1 commit into from
Closed

Conversation

fridrik01
Copy link
Contributor

Adding feature flag to have the code upgrade syscall disabled until we officially release it

@codecov-commenter
Copy link

Codecov Report

Merging #1906 (7fd7df8) into actor-upgrades (22d00c5) will decrease coverage by 0.30%.
The diff coverage is n/a.

Additional details and impacted files

Impacted file tree graph

@@                Coverage Diff                 @@
##           actor-upgrades    #1906      +/-   ##
==================================================
- Coverage           75.91%   75.61%   -0.30%     
==================================================
  Files                 153      153              
  Lines               15220    15146      -74     
==================================================
- Hits                11554    11453     -101     
- Misses               3666     3693      +27     
Files Coverage Δ
fvm/src/kernel/default.rs 59.42% <ø> (-2.08%) ⬇️
fvm/src/syscalls/actor.rs 95.50% <ø> (-0.38%) ⬇️
fvm/src/syscalls/mod.rs 97.81% <ø> (-1.10%) ⬇️

... and 7 files with indirect coverage changes

Stebalien added a commit that referenced this pull request Oct 26, 2023
This is a minimal version of #1906 that still compiles the relevent
code, just disables it at runtime.
@Stebalien
Copy link
Member

I've implemented a minimal version of this in the main PR that just disables the syscall, but still compiles the code.

@Stebalien Stebalien closed this Oct 26, 2023
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.

3 participants