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

[API-78] Implement the Hazelcast Cloud Discovery #832

Merged
merged 10 commits into from
Mar 19, 2021

Conversation

ihsandemir
Copy link
Collaborator

Implemented the Hazelcast cloud discovery.

@ihsandemir ihsandemir added this to the 4.1 milestone Mar 15, 2021
@ihsandemir ihsandemir self-assigned this Mar 15, 2021
@devOpsHazelcast
Copy link
Contributor

Windows test FAILed.

1 similar comment
@devOpsHazelcast
Copy link
Contributor

Windows test FAILed.

@devOpsHazelcast
Copy link
Contributor

Linux test FAILed.

1 similar comment
@devOpsHazelcast
Copy link
Contributor

Linux test FAILed.

@devOpsHazelcast
Copy link
Contributor

Windows test FAILed.

@devOpsHazelcast
Copy link
Contributor

Linux test PASSed.

@devOpsHazelcast
Copy link
Contributor

Windows test PASSed.

@devOpsHazelcast
Copy link
Contributor

Linux test PASSed.

@devOpsHazelcast
Copy link
Contributor

Windows test PASSed.

@ihsandemir ihsandemir marked this pull request as ready for review March 16, 2021 13:02
@devOpsHazelcast
Copy link
Contributor

Linux test FAILed.

@ihsandemir
Copy link
Collaborator Author

verify-linux

@devOpsHazelcast
Copy link
Contributor

Windows test PASSed.

@devOpsHazelcast
Copy link
Contributor

Linux test PASSed.

1 similar comment
@devOpsHazelcast
Copy link
Contributor

Linux test PASSed.

…nabled.

A minor fix for printing the connected server ip.
@devOpsHazelcast
Copy link
Contributor

Windows test PASSed.

@devOpsHazelcast
Copy link
Contributor

Linux test PASSed.

sancar
sancar previously approved these changes Mar 17, 2021
@devOpsHazelcast
Copy link
Contributor

Windows test PASSed.

@devOpsHazelcast
Copy link
Contributor

Linux test PASSed.

Please check the code sample at `examples/cloud-discovery/connect-cloud.cpp` for a full featured cloud discovery example.

## 5.8.1. Cloud Discovery With SSL Enabled
You can create a Hazelcast cluster in the cloud with "[Enable Encryption](https://docs.cloud.hazelcast.com/docs/encryption)" option. When this option is selected the cluster requires the clients to connect using the SSL connection and the client should be configured to do [mutual authentication](#6122-mutual-authentication-two-way-authentication). The required certificate authority file, client certificate, client key file, client key file and the key file PEM pass phrase are located at the Hazelcast cloud web site cluster configuration `Configure Clients` page. Download the keystore file and unzip it (it will be folder such as `hzcloud_xxx_keys` where `xxx` is the cluster number), this zip includes all the required files. Also, copy the `Keystore and truststore password` which is the client key PEM file pass phrase. Once, you have all this information in hand, you can configure the client as in the following code snippet:
Copy link
Contributor

Choose a reason for hiding this comment

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

client key file, client key file

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

done.

hazelcast/src/hazelcast/client/discovery.cpp Outdated Show resolved Hide resolved
ihsandemir and others added 2 commits March 19, 2021 15:31
Co-authored-by: yemreinci <18687880+yemreinci@users.noreply.github.com>
@devOpsHazelcast
Copy link
Contributor

Windows test PASSed.

@devOpsHazelcast
Copy link
Contributor

Linux test PASSed.

1 similar comment
@devOpsHazelcast
Copy link
Contributor

Linux test PASSed.

@devOpsHazelcast
Copy link
Contributor

Windows test PASSed.

@ihsandemir ihsandemir merged commit cd439bd into hazelcast:master Mar 19, 2021
@ihsandemir ihsandemir deleted the cloud_discovery branch March 19, 2021 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants