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

configuration files templates updated #3270

Merged
merged 3 commits into from
Sep 3, 2024

Conversation

Niks988
Copy link
Collaborator

@Niks988 Niks988 commented Aug 22, 2024

Description

Upgraded .origintrail_noderc template files (mainnet and testnet)

Fixes # (issue)
None

Type of change

Improvement

@@ -57,7 +76,7 @@
},
"publicCurrent":{
"url":"http://localhost:9999",
"name":"public-current",
"name":"kb",
Copy link
Collaborator

Choose a reason for hiding this comment

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

@Mihajlo-Pavlovic double check this change

Copy link
Collaborator

Choose a reason for hiding this comment

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

@Niks988 we moved to using public-current here it should stay that way

@@ -57,7 +76,7 @@
},
"publicCurrent":{
"url":"http://localhost:9999",
"name":"public-current",
"name":"kb",
Copy link
Collaborator

Choose a reason for hiding this comment

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

@Niks988 this shouldn't be changed also

@@ -5,34 +5,53 @@
"otp:2043":{
"enabled":true,
Copy link
Collaborator

Choose a reason for hiding this comment

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

Not sure if this should also be parametrised, if someone doesn't want to run node on some network

@Mihajlo-Pavlovic
Copy link
Collaborator

There also should be configuraton for libp2p
"network": {
"implementation": {
"libp2p-service": {
"config": {
"nat": {
"enabled": true,
"externalIp": // Or smth like that
}
}
}
}

@Mihajlo-Pavlovic Mihajlo-Pavlovic merged commit 1bda008 into v6/develop Sep 3, 2024
2 of 3 checks passed
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