Skip to content
This repository has been archived by the owner on Feb 15, 2024. It is now read-only.

Upgrade sdk to gemini-3g-2024-jan-08 #237

Merged
merged 2 commits into from
Jan 17, 2024
Merged

Upgrade sdk to gemini-3g-2024-jan-08 #237

merged 2 commits into from
Jan 17, 2024

Conversation

ParthDesai
Copy link
Contributor

Description

This PR updates sdk to gemini-3g-2023-jan-08 snapshot.

Notable changes:

  • Farm arguments are changed to be in parity with monorepo farmer's cli arguments.

@ParthDesai
Copy link
Contributor Author

ParthDesai commented Jan 16, 2024

Update: @nazar-pc @jfrank-summit Fixed now.

Mac CI jobs are failing with the error: *** read jobs pipe: Resource temporarily unavailable. Investigating..

perl: warning: Falling back to the standard locale ("C").
configure.ac:44: installing './config/compile'
configure.ac:40: installing './config/missing'
contrib/misc/Makefile.am: installing './config/depcomp'
autoreconf: './config/config.sub' is updated
autoreconf: './config/config.guess' is updated
autoreconf: './config/install-sh' is updated
autoreconf: Leaving directory '.'
configure: WARNING: "-fvisibility=hidden" has been added to the hwloc CFLAGS
make[1]: *** read jobs pipe: Resource temporarily unavailable.  Stop.
make[1]: *** Waiting for unfinished jobs....
make: *** [install-recursive] Error 1

@ParthDesai ParthDesai force-pushed the upgrade-to-jan-08 branch 3 times, most recently from 1080df8 to 2764044 Compare January 17, 2024 04:23
Copy link
Member

@nazar-pc nazar-pc left a comment

Choose a reason for hiding this comment

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

I think many of the things don't need numa feature on them

Cargo.toml Outdated
Comment on lines 155 to 159
"sdk-farmer/numa",
"sdk-dsn/numa",
"sdk-utils/numa",
"sdk-node/numa",
"sdk-substrate/numa"
Copy link
Member

Choose a reason for hiding this comment

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

Why would so many features need to be aware of NUMA support? subspace-farmer is the only crate with NUMA awareness, is it really a dependency of every single of those crate AND makes a difference there?

Copy link
Contributor Author

@ParthDesai ParthDesai Jan 17, 2024

Choose a reason for hiding this comment

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

@nazar-pc Yes. Almost every package either uses something from subspace farmer (for example, sdk-utils uses NodeClient and sdk-dsn uses PieceCache) and/or uses the sdk-utils or sdk-dsn package.

Initial plan to have only sdk-farmer have numa feature, but there is enough inter-dependency that I need to add it everywhere.

Copy link
Contributor Author

@ParthDesai ParthDesai Jan 17, 2024

Choose a reason for hiding this comment

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

Hmm. Maybe I can mark default-features=false and try to simplify this.

Copy link
Member

Choose a reason for hiding this comment

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

Maybe I can mark default-features=false and try to simplify this

Yes, please do

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Simplified. Only sdk-farmer has numa dependency now.

@ParthDesai ParthDesai enabled auto-merge January 17, 2024 10:51
@ParthDesai ParthDesai merged commit 000c6c7 into main Jan 17, 2024
10 checks passed
@ParthDesai ParthDesai deleted the upgrade-to-jan-08 branch January 17, 2024 12:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants