-
Notifications
You must be signed in to change notification settings - Fork 14
process: IP Whitelisting #175
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
Conversation
|
@nikowatari can you review this SV process please? |
|
I slacked and emailed @nikowatari the reminder of this one. |
|
How is this different from https://github.com/global-synchronizer-foundation/configs/blob/main/processes/new-whitelisted-ip.md ? |
waynecollier-da
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@isegall-da Can you take a look? I'm not sure I know exactly where to say both SV Sponsor and Node as a Service provider, versus just SV Sponsor
|
@hythloda I agree with @stas-sbi that seems to have too much intersection with https://github.com/global-synchronizer-foundation/configs/blob/main/processes/new-whitelisted-ip.md, seems like the two should be merged. On SV sponsor vs node operator, I'm thinking maybe we just need to make the distinction at the highest level, so something like: |
Co-authored-by: Wayne Collier <wayne.collier@digitalasset.com> Signed-off-by: Amanda L Martin <hythloda@gmail.com>
Signed-off-by: Amanda L Martin <hythloda@gmail.com>
isegall-da
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good, thank you
No description provided.