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

aeon: add connection from the etcd/tcs #1108

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

AlexandrLitkevich
Copy link
Contributor

@AlexandrLitkevich AlexandrLitkevich commented Feb 20, 2025

aeon: add connection from the etcd/tcs

@@@@Tarantool_box document
Title: add connection from the etcd/tcs config

Added ability to connect to gRpc server used a configuration etcd/tcs

tt aeon connect URI INSTANCE_NAME

Closes #1052

@AlexandrLitkevich AlexandrLitkevich added the full-ci Enables full ci tests label Feb 20, 2025
@AlexandrLitkevich AlexandrLitkevich force-pushed the TNTP-1074 branch 4 times, most recently from 46ee573 to 6c997fd Compare February 25, 2025 06:10
@AlexandrLitkevich AlexandrLitkevich changed the title Tntp 1074 aeon: add connection from the etcd/tcs Feb 25, 2025
@AlexandrLitkevich AlexandrLitkevich force-pushed the TNTP-1074 branch 2 times, most recently from d9a0ee8 to fdacd23 Compare February 25, 2025 07:46
@oleg-jukovec
Copy link
Contributor

oleg-jukovec commented Feb 25, 2025

@@@@Tarantool_box document -> @TarantoolBot document
gRpc -> gRPC

@@ -64,7 +64,6 @@ func getTlsConfig(args cmd.Ssl) (*tls.Config, error) {
}
}
// Else if RootCAs is nil, TLS uses the host's root CA set.

Copy link
Contributor

Choose a reason for hiding this comment

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

Please, revert the change.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

Key string
// Instance is an instance name.
Instance string
// Username is a user name for authorization
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
// Username is a user name for authorization
// Username is a user name for authorization.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

Instance string
// Username is a user name for authorization
Username string
// Password is a password for authorization
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
// Password is a password for authorization
// Password is a password for authorization.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

@TarantoolBot document
Title: add connection from the etcd/tcs config

Added ability to connect to gRPC server used a configuration etcd/tcs

tt aeon connect URI INSTANCE_NAME

Closes #1052
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
full-ci Enables full ci tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

aeon: connect from a config in a cluster config storage
3 participants