Atri,
Let's discuss the subj together with the community. Ignite already supports AWS [1] and GCE [2] IP Finders out of the box, but the Azure one is still missing. I can confirm that the demand exists, and rather frequently, I see developers asking for an Azure-native IP finder for Ignite. Atri, could you please research how to implement the IP finder and suggest a solution in this discussion thread? See how it was done for AWS and GCE, we might go the same route or use a more contemporary and easy-to-configure approach for Azure. [1] https://ignite.apache.org/docs/latest/clustering/discovery-in-the-cloud#amazon-s3-ip-finder [2] https://ignite.apache.org/docs/latest/clustering/discovery-in-the-cloud#google-compute-discovery - Denis |
Thanks Denis.
I have raised a PR for the same: https://github.com/apache/ignite/pull/8897 Regards, Atri On Wed, Mar 10, 2021 at 1:21 AM Denis Magda <[hidden email]> wrote: > > Atri, > > Let's discuss the subj together with the community. Ignite already supports > AWS [1] and GCE [2] IP Finders out of the box, but the Azure one is still > missing. I can confirm that the demand exists, and rather frequently, I see > developers asking for an Azure-native IP finder for Ignite. > > Atri, could you please research how to implement the IP finder and suggest > a solution in this discussion thread? See how it was done for AWS and GCE, > we might go the same route or use a more contemporary and easy-to-configure > approach for Azure. > > [1] > https://ignite.apache.org/docs/latest/clustering/discovery-in-the-cloud#amazon-s3-ip-finder > [2] > https://ignite.apache.org/docs/latest/clustering/discovery-in-the-cloud#google-compute-discovery > > - > Denis -- Regards, Atri Apache Concerted |
Explanation of the design:
The design follows a similar model to existing IP finders -- it either takes a pre-created container or creates a new container and creates entries in it for each IP address. I have tested it with a test account and the operation is fine. On Fri, Mar 19, 2021 at 10:23 AM Atri Sharma <[hidden email]> wrote: > > Thanks Denis. > > I have raised a PR for the same: > > https://github.com/apache/ignite/pull/8897 > > Regards, > > Atri > > On Wed, Mar 10, 2021 at 1:21 AM Denis Magda <[hidden email]> wrote: > > > > Atri, > > > > Let's discuss the subj together with the community. Ignite already supports > > AWS [1] and GCE [2] IP Finders out of the box, but the Azure one is still > > missing. I can confirm that the demand exists, and rather frequently, I see > > developers asking for an Azure-native IP finder for Ignite. > > > > Atri, could you please research how to implement the IP finder and suggest > > a solution in this discussion thread? See how it was done for AWS and GCE, > > we might go the same route or use a more contemporary and easy-to-configure > > approach for Azure. > > > > [1] > > https://ignite.apache.org/docs/latest/clustering/discovery-in-the-cloud#amazon-s3-ip-finder > > [2] > > https://ignite.apache.org/docs/latest/clustering/discovery-in-the-cloud#google-compute-discovery > > > > - > > Denis > > > > -- > Regards, > > Atri > Apache Concerted -- Regards, Atri Apache Concerted |
In reply to this post by Atri Sharma-2
Gentle reminder on this -- please help in reviewing this.
On Fri, Mar 19, 2021 at 10:23 AM Atri Sharma <[hidden email]> wrote: > > Thanks Denis. > > I have raised a PR for the same: > > https://github.com/apache/ignite/pull/8897 > > Regards, > > Atri > > On Wed, Mar 10, 2021 at 1:21 AM Denis Magda <[hidden email]> wrote: > > > > Atri, > > > > Let's discuss the subj together with the community. Ignite already supports > > AWS [1] and GCE [2] IP Finders out of the box, but the Azure one is still > > missing. I can confirm that the demand exists, and rather frequently, I see > > developers asking for an Azure-native IP finder for Ignite. > > > > Atri, could you please research how to implement the IP finder and suggest > > a solution in this discussion thread? See how it was done for AWS and GCE, > > we might go the same route or use a more contemporary and easy-to-configure > > approach for Azure. > > > > [1] > > https://ignite.apache.org/docs/latest/clustering/discovery-in-the-cloud#amazon-s3-ip-finder > > [2] > > https://ignite.apache.org/docs/latest/clustering/discovery-in-the-cloud#google-compute-discovery > > > > - > > Denis > > > > -- > Regards, > > Atri > Apache Concerted -- Regards, Atri Apache Concerted |
Hello!
I am going to check this change out when I have time, using my Azure account. Regards, -- Ilya Kasnacheev вт, 23 мар. 2021 г. в 07:20, Atri Sharma <[hidden email]>: > Gentle reminder on this -- please help in reviewing this. > > On Fri, Mar 19, 2021 at 10:23 AM Atri Sharma <[hidden email]> wrote: > > > > Thanks Denis. > > > > I have raised a PR for the same: > > > > https://github.com/apache/ignite/pull/8897 > > > > Regards, > > > > Atri > > > > On Wed, Mar 10, 2021 at 1:21 AM Denis Magda <[hidden email]> wrote: > > > > > > Atri, > > > > > > Let's discuss the subj together with the community. Ignite already > supports > > > AWS [1] and GCE [2] IP Finders out of the box, but the Azure one is > still > > > missing. I can confirm that the demand exists, and rather frequently, > I see > > > developers asking for an Azure-native IP finder for Ignite. > > > > > > Atri, could you please research how to implement the IP finder and > suggest > > > a solution in this discussion thread? See how it was done for AWS and > GCE, > > > we might go the same route or use a more contemporary and > easy-to-configure > > > approach for Azure. > > > > > > [1] > > > > https://ignite.apache.org/docs/latest/clustering/discovery-in-the-cloud#amazon-s3-ip-finder > > > [2] > > > > https://ignite.apache.org/docs/latest/clustering/discovery-in-the-cloud#google-compute-discovery > > > > > > - > > > Denis > > > > > > > > -- > > Regards, > > > > Atri > > Apache Concerted > > > > -- > Regards, > > Atri > Apache Concerted > |
Thank you!
I have updated the PR. Please see and let me know. On Tue, Mar 23, 2021 at 4:27 PM Ilya Kasnacheev <[hidden email]> wrote: > > Hello! > > I am going to check this change out when I have time, using my Azure > account. > > Regards, > -- > Ilya Kasnacheev > > > вт, 23 мар. 2021 г. в 07:20, Atri Sharma <[hidden email]>: > > > Gentle reminder on this -- please help in reviewing this. > > > > On Fri, Mar 19, 2021 at 10:23 AM Atri Sharma <[hidden email]> wrote: > > > > > > Thanks Denis. > > > > > > I have raised a PR for the same: > > > > > > https://github.com/apache/ignite/pull/8897 > > > > > > Regards, > > > > > > Atri > > > > > > On Wed, Mar 10, 2021 at 1:21 AM Denis Magda <[hidden email]> wrote: > > > > > > > > Atri, > > > > > > > > Let's discuss the subj together with the community. Ignite already > > supports > > > > AWS [1] and GCE [2] IP Finders out of the box, but the Azure one is > > still > > > > missing. I can confirm that the demand exists, and rather frequently, > > I see > > > > developers asking for an Azure-native IP finder for Ignite. > > > > > > > > Atri, could you please research how to implement the IP finder and > > suggest > > > > a solution in this discussion thread? See how it was done for AWS and > > GCE, > > > > we might go the same route or use a more contemporary and > > easy-to-configure > > > > approach for Azure. > > > > > > > > [1] > > > > > > https://ignite.apache.org/docs/latest/clustering/discovery-in-the-cloud#amazon-s3-ip-finder > > > > [2] > > > > > > https://ignite.apache.org/docs/latest/clustering/discovery-in-the-cloud#google-compute-discovery > > > > > > > > - > > > > Denis > > > > > > > > > > > > -- > > > Regards, > > > > > > Atri > > > Apache Concerted > > > > > > > > -- > > Regards, > > > > Atri > > Apache Concerted > > -- Regards, Atri Apache Concerted |
Hello!
I have left some more comments after trying this change on a real Azure cluster. Regards, -- Ilya Kasnacheev вт, 23 мар. 2021 г. в 18:32, Atri Sharma <[hidden email]>: > Thank you! > > I have updated the PR. Please see and let me know. > > On Tue, Mar 23, 2021 at 4:27 PM Ilya Kasnacheev > <[hidden email]> wrote: > > > > Hello! > > > > I am going to check this change out when I have time, using my Azure > > account. > > > > Regards, > > -- > > Ilya Kasnacheev > > > > > > вт, 23 мар. 2021 г. в 07:20, Atri Sharma <[hidden email]>: > > > > > Gentle reminder on this -- please help in reviewing this. > > > > > > On Fri, Mar 19, 2021 at 10:23 AM Atri Sharma <[hidden email]> wrote: > > > > > > > > Thanks Denis. > > > > > > > > I have raised a PR for the same: > > > > > > > > https://github.com/apache/ignite/pull/8897 > > > > > > > > Regards, > > > > > > > > Atri > > > > > > > > On Wed, Mar 10, 2021 at 1:21 AM Denis Magda <[hidden email]> > wrote: > > > > > > > > > > Atri, > > > > > > > > > > Let's discuss the subj together with the community. Ignite already > > > supports > > > > > AWS [1] and GCE [2] IP Finders out of the box, but the Azure one is > > > still > > > > > missing. I can confirm that the demand exists, and rather > frequently, > > > I see > > > > > developers asking for an Azure-native IP finder for Ignite. > > > > > > > > > > Atri, could you please research how to implement the IP finder and > > > suggest > > > > > a solution in this discussion thread? See how it was done for AWS > and > > > GCE, > > > > > we might go the same route or use a more contemporary and > > > easy-to-configure > > > > > approach for Azure. > > > > > > > > > > [1] > > > > > > > > > https://ignite.apache.org/docs/latest/clustering/discovery-in-the-cloud#amazon-s3-ip-finder > > > > > [2] > > > > > > > > > https://ignite.apache.org/docs/latest/clustering/discovery-in-the-cloud#google-compute-discovery > > > > > > > > > > - > > > > > Denis > > > > > > > > > > > > > > > > -- > > > > Regards, > > > > > > > > Atri > > > > Apache Concerted > > > > > > > > > > > > -- > > > Regards, > > > > > > Atri > > > Apache Concerted > > > > > -- > Regards, > > Atri > Apache Concerted > |
Thank you for the review.
I have updated the PR. Please see. On Mon, Mar 29, 2021 at 8:27 PM Ilya Kasnacheev <[hidden email]> wrote: > > Hello! > > I have left some more comments after trying this change on a real Azure > cluster. > > Regards, > -- > Ilya Kasnacheev > > > вт, 23 мар. 2021 г. в 18:32, Atri Sharma <[hidden email]>: > > > Thank you! > > > > I have updated the PR. Please see and let me know. > > > > On Tue, Mar 23, 2021 at 4:27 PM Ilya Kasnacheev > > <[hidden email]> wrote: > > > > > > Hello! > > > > > > I am going to check this change out when I have time, using my Azure > > > account. > > > > > > Regards, > > > -- > > > Ilya Kasnacheev > > > > > > > > > вт, 23 мар. 2021 г. в 07:20, Atri Sharma <[hidden email]>: > > > > > > > Gentle reminder on this -- please help in reviewing this. > > > > > > > > On Fri, Mar 19, 2021 at 10:23 AM Atri Sharma <[hidden email]> wrote: > > > > > > > > > > Thanks Denis. > > > > > > > > > > I have raised a PR for the same: > > > > > > > > > > https://github.com/apache/ignite/pull/8897 > > > > > > > > > > Regards, > > > > > > > > > > Atri > > > > > > > > > > On Wed, Mar 10, 2021 at 1:21 AM Denis Magda <[hidden email]> > > wrote: > > > > > > > > > > > > Atri, > > > > > > > > > > > > Let's discuss the subj together with the community. Ignite already > > > > supports > > > > > > AWS [1] and GCE [2] IP Finders out of the box, but the Azure one is > > > > still > > > > > > missing. I can confirm that the demand exists, and rather > > frequently, > > > > I see > > > > > > developers asking for an Azure-native IP finder for Ignite. > > > > > > > > > > > > Atri, could you please research how to implement the IP finder and > > > > suggest > > > > > > a solution in this discussion thread? See how it was done for AWS > > and > > > > GCE, > > > > > > we might go the same route or use a more contemporary and > > > > easy-to-configure > > > > > > approach for Azure. > > > > > > > > > > > > [1] > > > > > > > > > > > > https://ignite.apache.org/docs/latest/clustering/discovery-in-the-cloud#amazon-s3-ip-finder > > > > > > [2] > > > > > > > > > > > > https://ignite.apache.org/docs/latest/clustering/discovery-in-the-cloud#google-compute-discovery > > > > > > > > > > > > - > > > > > > Denis > > > > > > > > > > > > > > > > > > > > -- > > > > > Regards, > > > > > > > > > > Atri > > > > > Apache Concerted > > > > > > > > > > > > > > > > -- > > > > Regards, > > > > > > > > Atri > > > > Apache Concerted > > > > > > > > -- > > Regards, > > > > Atri > > Apache Concerted > > -- Regards, Atri Apache Concerted |
Hello!
Were you successful with using it from the deliverable of mvn initialize -Prelease? Please refer for example to aws module dependencies section: <dependency> <groupId>com.amazonaws</groupId> <artifactId>aws-java-sdk-core</artifactId> <version>${aws.sdk.version}</version> </dependency> <dependency> <groupId>com.amazonaws</groupId> <artifactId>aws-java-sdk-s3</artifactId> <version>${aws.sdk.version}</version> </dependency> <dependency> <groupId>com.amazonaws</groupId> <artifactId>aws-java-sdk-ec2</artifactId> <version>${aws.sdk.version}</version> </dependency> <dependency> <groupId>com.amazonaws</groupId> <artifactId>aws-java-sdk-elasticloadbalancing</artifactId> <version>${aws.sdk.version}</version> </dependency> <dependency> <groupId>com.amazonaws</groupId> <artifactId>aws-java-sdk-elasticloadbalancingv2</artifactId> <version>${aws.sdk.version}</version> </dependency> <dependency> <groupId>com.amazonaws</groupId> <artifactId>aws-java-sdk-kms</artifactId> <version>${aws.sdk.version}</version> </dependency> <!-- Do not remove com.fasterxml.jackson.core:jackson-core it is required at runtime --> <dependency> <groupId>com.fasterxml.jackson.core</groupId> <artifactId>jackson-core</artifactId> <version>${jackson.version}</version> </dependency> <!-- Do not remove com.fasterxml.jackson.core:jackson-annotations it is required at runtime --> <dependency> <groupId>com.fasterxml.jackson.core</groupId> <artifactId>jackson-annotations</artifactId> <version>${jackson.version}</version> </dependency> <!-- Do not remove com.fasterxml.jackson.core:jackson-databind it is required at runtime --> <dependency> <groupId>com.fasterxml.jackson.core</groupId> <artifactId>jackson-databind</artifactId> <version>${jackson.version}</version> </dependency> <dependency> <groupId>com.amazonaws</groupId> <artifactId>aws-encryption-sdk-java</artifactId> <version>${aws.encryption.sdk.version}</version> <exclusions> <exclusion> <groupId>org.bouncycastle</groupId> <artifactId>bcprov-ext-jdk15on</artifactId> </exclusion> </exclusions> </dependency> <dependency> <groupId>org.bouncycastle</groupId> <artifactId>bcprov-ext-jdk15on</artifactId> <version>${bouncycastle.version}</version> </dependency> <dependency> <groupId>joda-time</groupId> <artifactId>joda-time</artifactId> <version>2.8.1</version> </dependency> <dependency> <groupId>org.apache.httpcomponents</groupId> <artifactId>httpclient</artifactId> <version>${httpclient.version}</version> </dependency> <dependency> <groupId>org.apache.httpcomponents</groupId> <artifactId>httpcore</artifactId> <version>${httpcore.version}</version> </dependency> azure module should likewise list all of its actual dependencies. Right now the module directory is empty so I'm not even going to test to run it in stand-alone mode: ~/Downloads/apache-ignite-2.11.0-SNAPSHOT-bin% ls libs/optional/ignite-azure azure-storage-blob-12.0.0.jar ignite-azure-2.11.0-SNAPSHOT.jar README.txt Please address this issue along with other things which I have also commented on the PR. Regards, -- Ilya Kasnacheev ср, 31 мар. 2021 г. в 08:17, Atri Sharma <[hidden email]>: > Thank you for the review. > > I have updated the PR. Please see. > > On Mon, Mar 29, 2021 at 8:27 PM Ilya Kasnacheev > <[hidden email]> wrote: > > > > Hello! > > > > I have left some more comments after trying this change on a real Azure > > cluster. > > > > Regards, > > -- > > Ilya Kasnacheev > > > > > > вт, 23 мар. 2021 г. в 18:32, Atri Sharma <[hidden email]>: > > > > > Thank you! > > > > > > I have updated the PR. Please see and let me know. > > > > > > On Tue, Mar 23, 2021 at 4:27 PM Ilya Kasnacheev > > > <[hidden email]> wrote: > > > > > > > > Hello! > > > > > > > > I am going to check this change out when I have time, using my Azure > > > > account. > > > > > > > > Regards, > > > > -- > > > > Ilya Kasnacheev > > > > > > > > > > > > вт, 23 мар. 2021 г. в 07:20, Atri Sharma <[hidden email]>: > > > > > > > > > Gentle reminder on this -- please help in reviewing this. > > > > > > > > > > On Fri, Mar 19, 2021 at 10:23 AM Atri Sharma <[hidden email]> > wrote: > > > > > > > > > > > > Thanks Denis. > > > > > > > > > > > > I have raised a PR for the same: > > > > > > > > > > > > https://github.com/apache/ignite/pull/8897 > > > > > > > > > > > > Regards, > > > > > > > > > > > > Atri > > > > > > > > > > > > On Wed, Mar 10, 2021 at 1:21 AM Denis Magda <[hidden email]> > > > wrote: > > > > > > > > > > > > > > Atri, > > > > > > > > > > > > > > Let's discuss the subj together with the community. Ignite > already > > > > > supports > > > > > > > AWS [1] and GCE [2] IP Finders out of the box, but the Azure > one is > > > > > still > > > > > > > missing. I can confirm that the demand exists, and rather > > > frequently, > > > > > I see > > > > > > > developers asking for an Azure-native IP finder for Ignite. > > > > > > > > > > > > > > Atri, could you please research how to implement the IP finder > and > > > > > suggest > > > > > > > a solution in this discussion thread? See how it was done for > AWS > > > and > > > > > GCE, > > > > > > > we might go the same route or use a more contemporary and > > > > > easy-to-configure > > > > > > > approach for Azure. > > > > > > > > > > > > > > [1] > > > > > > > > > > > > > > > > https://ignite.apache.org/docs/latest/clustering/discovery-in-the-cloud#amazon-s3-ip-finder > > > > > > > [2] > > > > > > > > > > > > > > > > https://ignite.apache.org/docs/latest/clustering/discovery-in-the-cloud#google-compute-discovery > > > > > > > > > > > > > > - > > > > > > > Denis > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > Regards, > > > > > > > > > > > > Atri > > > > > > Apache Concerted > > > > > > > > > > > > > > > > > > > > -- > > > > > Regards, > > > > > > > > > > Atri > > > > > Apache Concerted > > > > > > > > > > > -- > > > Regards, > > > > > > Atri > > > Apache Concerted > > > > > -- > Regards, > > Atri > Apache Concerted > |
Hello,
Thank you for taking a look. I am not sure what the confusion is. On the current iteration of PR, I am able to build and release and looking in the location you mentioned, I see: Atris-MacBook-Pro-15:libs atrisharma$ cd optional/ Atris-MacBook-Pro-15:optional atrisharma$ cd ignite-azure/ Atris-MacBook-Pro-15:ignite-azure atrisharma$ ls README.txt azure-storage-blob-12.0.0.jar ignite-azure-2.11.0-SNAPSHOT.jar Attached is the screenshot. I have fixed the comments on the PR, please see and let me know. Atri On Thu, Apr 1, 2021 at 8:21 PM Ilya Kasnacheev <[hidden email]> wrote: > > Hello! > > Were you successful with using it from the deliverable of mvn initialize > -Prelease? > > Please refer for example to aws module dependencies section: > > <dependency> > <groupId>com.amazonaws</groupId> > <artifactId>aws-java-sdk-core</artifactId> > <version>${aws.sdk.version}</version> > </dependency> > > <dependency> > <groupId>com.amazonaws</groupId> > <artifactId>aws-java-sdk-s3</artifactId> > <version>${aws.sdk.version}</version> > </dependency> > > <dependency> > <groupId>com.amazonaws</groupId> > <artifactId>aws-java-sdk-ec2</artifactId> > <version>${aws.sdk.version}</version> > </dependency> > > <dependency> > <groupId>com.amazonaws</groupId> > <artifactId>aws-java-sdk-elasticloadbalancing</artifactId> > <version>${aws.sdk.version}</version> > </dependency> > > <dependency> > <groupId>com.amazonaws</groupId> > <artifactId>aws-java-sdk-elasticloadbalancingv2</artifactId> > <version>${aws.sdk.version}</version> > </dependency> > > <dependency> > <groupId>com.amazonaws</groupId> > <artifactId>aws-java-sdk-kms</artifactId> > <version>${aws.sdk.version}</version> > </dependency> > > <!-- Do not remove com.fasterxml.jackson.core:jackson-core it is > required at runtime --> > <dependency> > <groupId>com.fasterxml.jackson.core</groupId> > <artifactId>jackson-core</artifactId> > <version>${jackson.version}</version> > </dependency> > > <!-- Do not remove com.fasterxml.jackson.core:jackson-annotations it > is required at runtime --> > <dependency> > <groupId>com.fasterxml.jackson.core</groupId> > <artifactId>jackson-annotations</artifactId> > <version>${jackson.version}</version> > </dependency> > > <!-- Do not remove com.fasterxml.jackson.core:jackson-databind it is > required at runtime --> > <dependency> > <groupId>com.fasterxml.jackson.core</groupId> > <artifactId>jackson-databind</artifactId> > <version>${jackson.version}</version> > </dependency> > > <dependency> > <groupId>com.amazonaws</groupId> > <artifactId>aws-encryption-sdk-java</artifactId> > <version>${aws.encryption.sdk.version}</version> > <exclusions> > <exclusion> > <groupId>org.bouncycastle</groupId> > <artifactId>bcprov-ext-jdk15on</artifactId> > </exclusion> > </exclusions> > </dependency> > > <dependency> > <groupId>org.bouncycastle</groupId> > <artifactId>bcprov-ext-jdk15on</artifactId> > <version>${bouncycastle.version}</version> > </dependency> > > <dependency> > <groupId>joda-time</groupId> > <artifactId>joda-time</artifactId> > <version>2.8.1</version> > </dependency> > > <dependency> > <groupId>org.apache.httpcomponents</groupId> > <artifactId>httpclient</artifactId> > <version>${httpclient.version}</version> > </dependency> > > <dependency> > <groupId>org.apache.httpcomponents</groupId> > <artifactId>httpcore</artifactId> > <version>${httpcore.version}</version> > </dependency> > > azure module should likewise list all of its actual dependencies. Right now > the module directory is empty so I'm not even going to test to run it in > stand-alone mode: > ~/Downloads/apache-ignite-2.11.0-SNAPSHOT-bin% ls libs/optional/ignite-azure > azure-storage-blob-12.0.0.jar ignite-azure-2.11.0-SNAPSHOT.jar README.txt > > Please address this issue along with other things which I have also > commented on the PR. > > Regards, > -- > Ilya Kasnacheev > > > ср, 31 мар. 2021 г. в 08:17, Atri Sharma <[hidden email]>: > > > Thank you for the review. > > > > I have updated the PR. Please see. > > > > On Mon, Mar 29, 2021 at 8:27 PM Ilya Kasnacheev > > <[hidden email]> wrote: > > > > > > Hello! > > > > > > I have left some more comments after trying this change on a real Azure > > > cluster. > > > > > > Regards, > > > -- > > > Ilya Kasnacheev > > > > > > > > > вт, 23 мар. 2021 г. в 18:32, Atri Sharma <[hidden email]>: > > > > > > > Thank you! > > > > > > > > I have updated the PR. Please see and let me know. > > > > > > > > On Tue, Mar 23, 2021 at 4:27 PM Ilya Kasnacheev > > > > <[hidden email]> wrote: > > > > > > > > > > Hello! > > > > > > > > > > I am going to check this change out when I have time, using my Azure > > > > > account. > > > > > > > > > > Regards, > > > > > -- > > > > > Ilya Kasnacheev > > > > > > > > > > > > > > > вт, 23 мар. 2021 г. в 07:20, Atri Sharma <[hidden email]>: > > > > > > > > > > > Gentle reminder on this -- please help in reviewing this. > > > > > > > > > > > > On Fri, Mar 19, 2021 at 10:23 AM Atri Sharma <[hidden email]> > > wrote: > > > > > > > > > > > > > > Thanks Denis. > > > > > > > > > > > > > > I have raised a PR for the same: > > > > > > > > > > > > > > https://github.com/apache/ignite/pull/8897 > > > > > > > > > > > > > > Regards, > > > > > > > > > > > > > > Atri > > > > > > > > > > > > > > On Wed, Mar 10, 2021 at 1:21 AM Denis Magda <[hidden email]> > > > > wrote: > > > > > > > > > > > > > > > > Atri, > > > > > > > > > > > > > > > > Let's discuss the subj together with the community. Ignite > > already > > > > > > supports > > > > > > > > AWS [1] and GCE [2] IP Finders out of the box, but the Azure > > one is > > > > > > still > > > > > > > > missing. I can confirm that the demand exists, and rather > > > > frequently, > > > > > > I see > > > > > > > > developers asking for an Azure-native IP finder for Ignite. > > > > > > > > > > > > > > > > Atri, could you please research how to implement the IP finder > > and > > > > > > suggest > > > > > > > > a solution in this discussion thread? See how it was done for > > AWS > > > > and > > > > > > GCE, > > > > > > > > we might go the same route or use a more contemporary and > > > > > > easy-to-configure > > > > > > > > approach for Azure. > > > > > > > > > > > > > > > > [1] > > > > > > > > > > > > > > > > > > > > https://ignite.apache.org/docs/latest/clustering/discovery-in-the-cloud#amazon-s3-ip-finder > > > > > > > > [2] > > > > > > > > > > > > > > > > > > > > https://ignite.apache.org/docs/latest/clustering/discovery-in-the-cloud#google-compute-discovery > > > > > > > > > > > > > > > > - > > > > > > > > Denis > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > > Regards, > > > > > > > > > > > > > > Atri > > > > > > > Apache Concerted > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > Regards, > > > > > > > > > > > > Atri > > > > > > Apache Concerted > > > > > > > > > > > > > > -- > > > > Regards, > > > > > > > > Atri > > > > Apache Concerted > > > > > > > > -- > > Regards, > > > > Atri > > Apache Concerted > > Regards, Atri Apache Concerted |
Hello!
But are you successful in running a node with Azure IP finder enabled in configuration, starting from that release directory? I amn't. Regards, -- Ilya Kasnacheev пт, 2 апр. 2021 г. в 07:42, Atri Sharma <[hidden email]>: > Hello, > > Thank you for taking a look. > > I am not sure what the confusion is. On the current iteration of PR, I > am able to build and release and looking in the location you > mentioned, I see: > > Atris-MacBook-Pro-15:libs atrisharma$ cd optional/ > > Atris-MacBook-Pro-15:optional atrisharma$ cd ignite-azure/ > > Atris-MacBook-Pro-15:ignite-azure atrisharma$ ls > > README.txt azure-storage-blob-12.0.0.jar ignite-azure-2.11.0-SNAPSHOT.jar > > > Attached is the screenshot. > > I have fixed the comments on the PR, please see and let me know. > > Atri > > On Thu, Apr 1, 2021 at 8:21 PM Ilya Kasnacheev > <[hidden email]> wrote: > > > > Hello! > > > > Were you successful with using it from the deliverable of mvn initialize > > -Prelease? > > > > Please refer for example to aws module dependencies section: > > > > <dependency> > > <groupId>com.amazonaws</groupId> > > <artifactId>aws-java-sdk-core</artifactId> > > <version>${aws.sdk.version}</version> > > </dependency> > > > > <dependency> > > <groupId>com.amazonaws</groupId> > > <artifactId>aws-java-sdk-s3</artifactId> > > <version>${aws.sdk.version}</version> > > </dependency> > > > > <dependency> > > <groupId>com.amazonaws</groupId> > > <artifactId>aws-java-sdk-ec2</artifactId> > > <version>${aws.sdk.version}</version> > > </dependency> > > > > <dependency> > > <groupId>com.amazonaws</groupId> > > <artifactId>aws-java-sdk-elasticloadbalancing</artifactId> > > <version>${aws.sdk.version}</version> > > </dependency> > > > > <dependency> > > <groupId>com.amazonaws</groupId> > > <artifactId>aws-java-sdk-elasticloadbalancingv2</artifactId> > > <version>${aws.sdk.version}</version> > > </dependency> > > > > <dependency> > > <groupId>com.amazonaws</groupId> > > <artifactId>aws-java-sdk-kms</artifactId> > > <version>${aws.sdk.version}</version> > > </dependency> > > > > <!-- Do not remove com.fasterxml.jackson.core:jackson-core it is > > required at runtime --> > > <dependency> > > <groupId>com.fasterxml.jackson.core</groupId> > > <artifactId>jackson-core</artifactId> > > <version>${jackson.version}</version> > > </dependency> > > > > <!-- Do not remove com.fasterxml.jackson.core:jackson-annotations it > > is required at runtime --> > > <dependency> > > <groupId>com.fasterxml.jackson.core</groupId> > > <artifactId>jackson-annotations</artifactId> > > <version>${jackson.version}</version> > > </dependency> > > > > <!-- Do not remove com.fasterxml.jackson.core:jackson-databind it is > > required at runtime --> > > <dependency> > > <groupId>com.fasterxml.jackson.core</groupId> > > <artifactId>jackson-databind</artifactId> > > <version>${jackson.version}</version> > > </dependency> > > > > <dependency> > > <groupId>com.amazonaws</groupId> > > <artifactId>aws-encryption-sdk-java</artifactId> > > <version>${aws.encryption.sdk.version}</version> > > <exclusions> > > <exclusion> > > <groupId>org.bouncycastle</groupId> > > <artifactId>bcprov-ext-jdk15on</artifactId> > > </exclusion> > > </exclusions> > > </dependency> > > > > <dependency> > > <groupId>org.bouncycastle</groupId> > > <artifactId>bcprov-ext-jdk15on</artifactId> > > <version>${bouncycastle.version}</version> > > </dependency> > > > > <dependency> > > <groupId>joda-time</groupId> > > <artifactId>joda-time</artifactId> > > <version>2.8.1</version> > > </dependency> > > > > <dependency> > > <groupId>org.apache.httpcomponents</groupId> > > <artifactId>httpclient</artifactId> > > <version>${httpclient.version}</version> > > </dependency> > > > > <dependency> > > <groupId>org.apache.httpcomponents</groupId> > > <artifactId>httpcore</artifactId> > > <version>${httpcore.version}</version> > > </dependency> > > > > azure module should likewise list all of its actual dependencies. Right > now > > the module directory is empty so I'm not even going to test to run it in > > stand-alone mode: > > ~/Downloads/apache-ignite-2.11.0-SNAPSHOT-bin% ls > libs/optional/ignite-azure > > azure-storage-blob-12.0.0.jar ignite-azure-2.11.0-SNAPSHOT.jar > README.txt > > > > Please address this issue along with other things which I have also > > commented on the PR. > > > > Regards, > > -- > > Ilya Kasnacheev > > > > > > ср, 31 мар. 2021 г. в 08:17, Atri Sharma <[hidden email]>: > > > > > Thank you for the review. > > > > > > I have updated the PR. Please see. > > > > > > On Mon, Mar 29, 2021 at 8:27 PM Ilya Kasnacheev > > > <[hidden email]> wrote: > > > > > > > > Hello! > > > > > > > > I have left some more comments after trying this change on a real > Azure > > > > cluster. > > > > > > > > Regards, > > > > -- > > > > Ilya Kasnacheev > > > > > > > > > > > > вт, 23 мар. 2021 г. в 18:32, Atri Sharma <[hidden email]>: > > > > > > > > > Thank you! > > > > > > > > > > I have updated the PR. Please see and let me know. > > > > > > > > > > On Tue, Mar 23, 2021 at 4:27 PM Ilya Kasnacheev > > > > > <[hidden email]> wrote: > > > > > > > > > > > > Hello! > > > > > > > > > > > > I am going to check this change out when I have time, using my > Azure > > > > > > account. > > > > > > > > > > > > Regards, > > > > > > -- > > > > > > Ilya Kasnacheev > > > > > > > > > > > > > > > > > > вт, 23 мар. 2021 г. в 07:20, Atri Sharma <[hidden email]>: > > > > > > > > > > > > > Gentle reminder on this -- please help in reviewing this. > > > > > > > > > > > > > > On Fri, Mar 19, 2021 at 10:23 AM Atri Sharma <[hidden email]> > > > wrote: > > > > > > > > > > > > > > > > Thanks Denis. > > > > > > > > > > > > > > > > I have raised a PR for the same: > > > > > > > > > > > > > > > > https://github.com/apache/ignite/pull/8897 > > > > > > > > > > > > > > > > Regards, > > > > > > > > > > > > > > > > Atri > > > > > > > > > > > > > > > > On Wed, Mar 10, 2021 at 1:21 AM Denis Magda < > [hidden email]> > > > > > wrote: > > > > > > > > > > > > > > > > > > Atri, > > > > > > > > > > > > > > > > > > Let's discuss the subj together with the community. Ignite > > > already > > > > > > > supports > > > > > > > > > AWS [1] and GCE [2] IP Finders out of the box, but the > Azure > > > one is > > > > > > > still > > > > > > > > > missing. I can confirm that the demand exists, and rather > > > > > frequently, > > > > > > > I see > > > > > > > > > developers asking for an Azure-native IP finder for Ignite. > > > > > > > > > > > > > > > > > > Atri, could you please research how to implement the IP > finder > > > and > > > > > > > suggest > > > > > > > > > a solution in this discussion thread? See how it was done > for > > > AWS > > > > > and > > > > > > > GCE, > > > > > > > > > we might go the same route or use a more contemporary and > > > > > > > easy-to-configure > > > > > > > > > approach for Azure. > > > > > > > > > > > > > > > > > > [1] > > > > > > > > > > > > > > > > > > > > > > > > > https://ignite.apache.org/docs/latest/clustering/discovery-in-the-cloud#amazon-s3-ip-finder > > > > > > > > > [2] > > > > > > > > > > > > > > > > > > > > > > > > > https://ignite.apache.org/docs/latest/clustering/discovery-in-the-cloud#google-compute-discovery > > > > > > > > > > > > > > > > > > - > > > > > > > > > Denis > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > > > Regards, > > > > > > > > > > > > > > > > Atri > > > > > > > > Apache Concerted > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > > Regards, > > > > > > > > > > > > > > Atri > > > > > > > Apache Concerted > > > > > > > > > > > > > > > > > -- > > > > > Regards, > > > > > > > > > > Atri > > > > > Apache Concerted > > > > > > > > > > > -- > > > Regards, > > > > > > Atri > > > Apache Concerted > > > > > -- > Regards, > > Atri > Apache Concerted > |
I was able to, but then, it might be that something is cached locally.
What errors did you run into? (I am assuming you are trying with a build from the latest iteration of the PR). Atri On Fri, Apr 2, 2021 at 2:19 PM Ilya Kasnacheev <[hidden email]> wrote: > > Hello! > > But are you successful in running a node with Azure IP finder enabled in > configuration, starting from that release directory? > > I amn't. > > Regards, > -- > Ilya Kasnacheev > > > пт, 2 апр. 2021 г. в 07:42, Atri Sharma <[hidden email]>: > > > Hello, > > > > Thank you for taking a look. > > > > I am not sure what the confusion is. On the current iteration of PR, I > > am able to build and release and looking in the location you > > mentioned, I see: > > > > Atris-MacBook-Pro-15:libs atrisharma$ cd optional/ > > > > Atris-MacBook-Pro-15:optional atrisharma$ cd ignite-azure/ > > > > Atris-MacBook-Pro-15:ignite-azure atrisharma$ ls > > > > README.txt azure-storage-blob-12.0.0.jar ignite-azure-2.11.0-SNAPSHOT.jar > > > > > > Attached is the screenshot. > > > > I have fixed the comments on the PR, please see and let me know. > > > > Atri > > > > On Thu, Apr 1, 2021 at 8:21 PM Ilya Kasnacheev > > <[hidden email]> wrote: > > > > > > Hello! > > > > > > Were you successful with using it from the deliverable of mvn initialize > > > -Prelease? > > > > > > Please refer for example to aws module dependencies section: > > > > > > <dependency> > > > <groupId>com.amazonaws</groupId> > > > <artifactId>aws-java-sdk-core</artifactId> > > > <version>${aws.sdk.version}</version> > > > </dependency> > > > > > > <dependency> > > > <groupId>com.amazonaws</groupId> > > > <artifactId>aws-java-sdk-s3</artifactId> > > > <version>${aws.sdk.version}</version> > > > </dependency> > > > > > > <dependency> > > > <groupId>com.amazonaws</groupId> > > > <artifactId>aws-java-sdk-ec2</artifactId> > > > <version>${aws.sdk.version}</version> > > > </dependency> > > > > > > <dependency> > > > <groupId>com.amazonaws</groupId> > > > <artifactId>aws-java-sdk-elasticloadbalancing</artifactId> > > > <version>${aws.sdk.version}</version> > > > </dependency> > > > > > > <dependency> > > > <groupId>com.amazonaws</groupId> > > > <artifactId>aws-java-sdk-elasticloadbalancingv2</artifactId> > > > <version>${aws.sdk.version}</version> > > > </dependency> > > > > > > <dependency> > > > <groupId>com.amazonaws</groupId> > > > <artifactId>aws-java-sdk-kms</artifactId> > > > <version>${aws.sdk.version}</version> > > > </dependency> > > > > > > <!-- Do not remove com.fasterxml.jackson.core:jackson-core it is > > > required at runtime --> > > > <dependency> > > > <groupId>com.fasterxml.jackson.core</groupId> > > > <artifactId>jackson-core</artifactId> > > > <version>${jackson.version}</version> > > > </dependency> > > > > > > <!-- Do not remove com.fasterxml.jackson.core:jackson-annotations it > > > is required at runtime --> > > > <dependency> > > > <groupId>com.fasterxml.jackson.core</groupId> > > > <artifactId>jackson-annotations</artifactId> > > > <version>${jackson.version}</version> > > > </dependency> > > > > > > <!-- Do not remove com.fasterxml.jackson.core:jackson-databind it is > > > required at runtime --> > > > <dependency> > > > <groupId>com.fasterxml.jackson.core</groupId> > > > <artifactId>jackson-databind</artifactId> > > > <version>${jackson.version}</version> > > > </dependency> > > > > > > <dependency> > > > <groupId>com.amazonaws</groupId> > > > <artifactId>aws-encryption-sdk-java</artifactId> > > > <version>${aws.encryption.sdk.version}</version> > > > <exclusions> > > > <exclusion> > > > <groupId>org.bouncycastle</groupId> > > > <artifactId>bcprov-ext-jdk15on</artifactId> > > > </exclusion> > > > </exclusions> > > > </dependency> > > > > > > <dependency> > > > <groupId>org.bouncycastle</groupId> > > > <artifactId>bcprov-ext-jdk15on</artifactId> > > > <version>${bouncycastle.version}</version> > > > </dependency> > > > > > > <dependency> > > > <groupId>joda-time</groupId> > > > <artifactId>joda-time</artifactId> > > > <version>2.8.1</version> > > > </dependency> > > > > > > <dependency> > > > <groupId>org.apache.httpcomponents</groupId> > > > <artifactId>httpclient</artifactId> > > > <version>${httpclient.version}</version> > > > </dependency> > > > > > > <dependency> > > > <groupId>org.apache.httpcomponents</groupId> > > > <artifactId>httpcore</artifactId> > > > <version>${httpcore.version}</version> > > > </dependency> > > > > > > azure module should likewise list all of its actual dependencies. Right > > now > > > the module directory is empty so I'm not even going to test to run it in > > > stand-alone mode: > > > ~/Downloads/apache-ignite-2.11.0-SNAPSHOT-bin% ls > > libs/optional/ignite-azure > > > azure-storage-blob-12.0.0.jar ignite-azure-2.11.0-SNAPSHOT.jar > > README.txt > > > > > > Please address this issue along with other things which I have also > > > commented on the PR. > > > > > > Regards, > > > -- > > > Ilya Kasnacheev > > > > > > > > > ср, 31 мар. 2021 г. в 08:17, Atri Sharma <[hidden email]>: > > > > > > > Thank you for the review. > > > > > > > > I have updated the PR. Please see. > > > > > > > > On Mon, Mar 29, 2021 at 8:27 PM Ilya Kasnacheev > > > > <[hidden email]> wrote: > > > > > > > > > > Hello! > > > > > > > > > > I have left some more comments after trying this change on a real > > Azure > > > > > cluster. > > > > > > > > > > Regards, > > > > > -- > > > > > Ilya Kasnacheev > > > > > > > > > > > > > > > вт, 23 мар. 2021 г. в 18:32, Atri Sharma <[hidden email]>: > > > > > > > > > > > Thank you! > > > > > > > > > > > > I have updated the PR. Please see and let me know. > > > > > > > > > > > > On Tue, Mar 23, 2021 at 4:27 PM Ilya Kasnacheev > > > > > > <[hidden email]> wrote: > > > > > > > > > > > > > > Hello! > > > > > > > > > > > > > > I am going to check this change out when I have time, using my > > Azure > > > > > > > account. > > > > > > > > > > > > > > Regards, > > > > > > > -- > > > > > > > Ilya Kasnacheev > > > > > > > > > > > > > > > > > > > > > вт, 23 мар. 2021 г. в 07:20, Atri Sharma <[hidden email]>: > > > > > > > > > > > > > > > Gentle reminder on this -- please help in reviewing this. > > > > > > > > > > > > > > > > On Fri, Mar 19, 2021 at 10:23 AM Atri Sharma <[hidden email]> > > > > wrote: > > > > > > > > > > > > > > > > > > Thanks Denis. > > > > > > > > > > > > > > > > > > I have raised a PR for the same: > > > > > > > > > > > > > > > > > > https://github.com/apache/ignite/pull/8897 > > > > > > > > > > > > > > > > > > Regards, > > > > > > > > > > > > > > > > > > Atri > > > > > > > > > > > > > > > > > > On Wed, Mar 10, 2021 at 1:21 AM Denis Magda < > > [hidden email]> > > > > > > wrote: > > > > > > > > > > > > > > > > > > > > Atri, > > > > > > > > > > > > > > > > > > > > Let's discuss the subj together with the community. Ignite > > > > already > > > > > > > > supports > > > > > > > > > > AWS [1] and GCE [2] IP Finders out of the box, but the > > Azure > > > > one is > > > > > > > > still > > > > > > > > > > missing. I can confirm that the demand exists, and rather > > > > > > frequently, > > > > > > > > I see > > > > > > > > > > developers asking for an Azure-native IP finder for Ignite. > > > > > > > > > > > > > > > > > > > > Atri, could you please research how to implement the IP > > finder > > > > and > > > > > > > > suggest > > > > > > > > > > a solution in this discussion thread? See how it was done > > for > > > > AWS > > > > > > and > > > > > > > > GCE, > > > > > > > > > > we might go the same route or use a more contemporary and > > > > > > > > easy-to-configure > > > > > > > > > > approach for Azure. > > > > > > > > > > > > > > > > > > > > [1] > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > https://ignite.apache.org/docs/latest/clustering/discovery-in-the-cloud#amazon-s3-ip-finder > > > > > > > > > > [2] > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > https://ignite.apache.org/docs/latest/clustering/discovery-in-the-cloud#google-compute-discovery > > > > > > > > > > > > > > > > > > > > - > > > > > > > > > > Denis > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > > > > Regards, > > > > > > > > > > > > > > > > > > Atri > > > > > > > > > Apache Concerted > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > > > Regards, > > > > > > > > > > > > > > > > Atri > > > > > > > > Apache Concerted > > > > > > > > > > > > > > > > > > > > -- > > > > > > Regards, > > > > > > > > > > > > Atri > > > > > > Apache Concerted > > > > > > > > > > > > > > -- > > > > Regards, > > > > > > > > Atri > > > > Apache Concerted > > > > > > > > -- > > Regards, > > > > Atri > > Apache Concerted > > -- Regards, Atri Apache Concerted |
Hello! Please find attached the log file of errors. This is yesterday's (Apr 1) build. Regards, -- Ilya Kasnacheev пт, 2 апр. 2021 г. в 11:52, Atri Sharma <[hidden email]>: I was able to, but then, it might be that something is cached locally. |
Hello,
Thank you for sharing. I was finally able to replicate the issue. However, I tried with other IPFinders and ran into the same problem (attached are the logs). Not sure what is causing this? Atri On Fri, Apr 2, 2021 at 2:29 PM Ilya Kasnacheev <[hidden email]> wrote: > > Hello! > > Please find attached the log file of errors. This is yesterday's (Apr 1) build. > > Regards, > -- > Ilya Kasnacheev > > > пт, 2 апр. 2021 г. в 11:52, Atri Sharma <[hidden email]>: >> >> I was able to, but then, it might be that something is cached locally. >> >> What errors did you run into? (I am assuming you are trying with a >> build from the latest iteration of the PR). >> >> Atri >> >> On Fri, Apr 2, 2021 at 2:19 PM Ilya Kasnacheev >> <[hidden email]> wrote: >> > >> > Hello! >> > >> > But are you successful in running a node with Azure IP finder enabled in >> > configuration, starting from that release directory? >> > >> > I amn't. >> > >> > Regards, >> > -- >> > Ilya Kasnacheev >> > >> > >> > пт, 2 апр. 2021 г. в 07:42, Atri Sharma <[hidden email]>: >> > >> > > Hello, >> > > >> > > Thank you for taking a look. >> > > >> > > I am not sure what the confusion is. On the current iteration of PR, I >> > > am able to build and release and looking in the location you >> > > mentioned, I see: >> > > >> > > Atris-MacBook-Pro-15:libs atrisharma$ cd optional/ >> > > >> > > Atris-MacBook-Pro-15:optional atrisharma$ cd ignite-azure/ >> > > >> > > Atris-MacBook-Pro-15:ignite-azure atrisharma$ ls >> > > >> > > README.txt azure-storage-blob-12.0.0.jar ignite-azure-2.11.0-SNAPSHOT.jar >> > > >> > > >> > > Attached is the screenshot. >> > > >> > > I have fixed the comments on the PR, please see and let me know. >> > > >> > > Atri >> > > >> > > On Thu, Apr 1, 2021 at 8:21 PM Ilya Kasnacheev >> > > <[hidden email]> wrote: >> > > > >> > > > Hello! >> > > > >> > > > Were you successful with using it from the deliverable of mvn initialize >> > > > -Prelease? >> > > > >> > > > Please refer for example to aws module dependencies section: >> > > > >> > > > <dependency> >> > > > <groupId>com.amazonaws</groupId> >> > > > <artifactId>aws-java-sdk-core</artifactId> >> > > > <version>${aws.sdk.version}</version> >> > > > </dependency> >> > > > >> > > > <dependency> >> > > > <groupId>com.amazonaws</groupId> >> > > > <artifactId>aws-java-sdk-s3</artifactId> >> > > > <version>${aws.sdk.version}</version> >> > > > </dependency> >> > > > >> > > > <dependency> >> > > > <groupId>com.amazonaws</groupId> >> > > > <artifactId>aws-java-sdk-ec2</artifactId> >> > > > <version>${aws.sdk.version}</version> >> > > > </dependency> >> > > > >> > > > <dependency> >> > > > <groupId>com.amazonaws</groupId> >> > > > <artifactId>aws-java-sdk-elasticloadbalancing</artifactId> >> > > > <version>${aws.sdk.version}</version> >> > > > </dependency> >> > > > >> > > > <dependency> >> > > > <groupId>com.amazonaws</groupId> >> > > > <artifactId>aws-java-sdk-elasticloadbalancingv2</artifactId> >> > > > <version>${aws.sdk.version}</version> >> > > > </dependency> >> > > > >> > > > <dependency> >> > > > <groupId>com.amazonaws</groupId> >> > > > <artifactId>aws-java-sdk-kms</artifactId> >> > > > <version>${aws.sdk.version}</version> >> > > > </dependency> >> > > > >> > > > <!-- Do not remove com.fasterxml.jackson.core:jackson-core it is >> > > > required at runtime --> >> > > > <dependency> >> > > > <groupId>com.fasterxml.jackson.core</groupId> >> > > > <artifactId>jackson-core</artifactId> >> > > > <version>${jackson.version}</version> >> > > > </dependency> >> > > > >> > > > <!-- Do not remove com.fasterxml.jackson.core:jackson-annotations it >> > > > is required at runtime --> >> > > > <dependency> >> > > > <groupId>com.fasterxml.jackson.core</groupId> >> > > > <artifactId>jackson-annotations</artifactId> >> > > > <version>${jackson.version}</version> >> > > > </dependency> >> > > > >> > > > <!-- Do not remove com.fasterxml.jackson.core:jackson-databind it is >> > > > required at runtime --> >> > > > <dependency> >> > > > <groupId>com.fasterxml.jackson.core</groupId> >> > > > <artifactId>jackson-databind</artifactId> >> > > > <version>${jackson.version}</version> >> > > > </dependency> >> > > > >> > > > <dependency> >> > > > <groupId>com.amazonaws</groupId> >> > > > <artifactId>aws-encryption-sdk-java</artifactId> >> > > > <version>${aws.encryption.sdk.version}</version> >> > > > <exclusions> >> > > > <exclusion> >> > > > <groupId>org.bouncycastle</groupId> >> > > > <artifactId>bcprov-ext-jdk15on</artifactId> >> > > > </exclusion> >> > > > </exclusions> >> > > > </dependency> >> > > > >> > > > <dependency> >> > > > <groupId>org.bouncycastle</groupId> >> > > > <artifactId>bcprov-ext-jdk15on</artifactId> >> > > > <version>${bouncycastle.version}</version> >> > > > </dependency> >> > > > >> > > > <dependency> >> > > > <groupId>joda-time</groupId> >> > > > <artifactId>joda-time</artifactId> >> > > > <version>2.8.1</version> >> > > > </dependency> >> > > > >> > > > <dependency> >> > > > <groupId>org.apache.httpcomponents</groupId> >> > > > <artifactId>httpclient</artifactId> >> > > > <version>${httpclient.version}</version> >> > > > </dependency> >> > > > >> > > > <dependency> >> > > > <groupId>org.apache.httpcomponents</groupId> >> > > > <artifactId>httpcore</artifactId> >> > > > <version>${httpcore.version}</version> >> > > > </dependency> >> > > > >> > > > azure module should likewise list all of its actual dependencies. Right >> > > now >> > > > the module directory is empty so I'm not even going to test to run it in >> > > > stand-alone mode: >> > > > ~/Downloads/apache-ignite-2.11.0-SNAPSHOT-bin% ls >> > > libs/optional/ignite-azure >> > > > azure-storage-blob-12.0.0.jar ignite-azure-2.11.0-SNAPSHOT.jar >> > > README.txt >> > > > >> > > > Please address this issue along with other things which I have also >> > > > commented on the PR. >> > > > >> > > > Regards, >> > > > -- >> > > > Ilya Kasnacheev >> > > > >> > > > >> > > > ср, 31 мар. 2021 г. в 08:17, Atri Sharma <[hidden email]>: >> > > > >> > > > > Thank you for the review. >> > > > > >> > > > > I have updated the PR. Please see. >> > > > > >> > > > > On Mon, Mar 29, 2021 at 8:27 PM Ilya Kasnacheev >> > > > > <[hidden email]> wrote: >> > > > > > >> > > > > > Hello! >> > > > > > >> > > > > > I have left some more comments after trying this change on a real >> > > Azure >> > > > > > cluster. >> > > > > > >> > > > > > Regards, >> > > > > > -- >> > > > > > Ilya Kasnacheev >> > > > > > >> > > > > > >> > > > > > вт, 23 мар. 2021 г. в 18:32, Atri Sharma <[hidden email]>: >> > > > > > >> > > > > > > Thank you! >> > > > > > > >> > > > > > > I have updated the PR. Please see and let me know. >> > > > > > > >> > > > > > > On Tue, Mar 23, 2021 at 4:27 PM Ilya Kasnacheev >> > > > > > > <[hidden email]> wrote: >> > > > > > > > >> > > > > > > > Hello! >> > > > > > > > >> > > > > > > > I am going to check this change out when I have time, using my >> > > Azure >> > > > > > > > account. >> > > > > > > > >> > > > > > > > Regards, >> > > > > > > > -- >> > > > > > > > Ilya Kasnacheev >> > > > > > > > >> > > > > > > > >> > > > > > > > вт, 23 мар. 2021 г. в 07:20, Atri Sharma <[hidden email]>: >> > > > > > > > >> > > > > > > > > Gentle reminder on this -- please help in reviewing this. >> > > > > > > > > >> > > > > > > > > On Fri, Mar 19, 2021 at 10:23 AM Atri Sharma <[hidden email]> >> > > > > wrote: >> > > > > > > > > > >> > > > > > > > > > Thanks Denis. >> > > > > > > > > > >> > > > > > > > > > I have raised a PR for the same: >> > > > > > > > > > >> > > > > > > > > > https://github.com/apache/ignite/pull/8897 >> > > > > > > > > > >> > > > > > > > > > Regards, >> > > > > > > > > > >> > > > > > > > > > Atri >> > > > > > > > > > >> > > > > > > > > > On Wed, Mar 10, 2021 at 1:21 AM Denis Magda < >> > > [hidden email]> >> > > > > > > wrote: >> > > > > > > > > > > >> > > > > > > > > > > Atri, >> > > > > > > > > > > >> > > > > > > > > > > Let's discuss the subj together with the community. Ignite >> > > > > already >> > > > > > > > > supports >> > > > > > > > > > > AWS [1] and GCE [2] IP Finders out of the box, but the >> > > Azure >> > > > > one is >> > > > > > > > > still >> > > > > > > > > > > missing. I can confirm that the demand exists, and rather >> > > > > > > frequently, >> > > > > > > > > I see >> > > > > > > > > > > developers asking for an Azure-native IP finder for Ignite. >> > > > > > > > > > > >> > > > > > > > > > > Atri, could you please research how to implement the IP >> > > finder >> > > > > and >> > > > > > > > > suggest >> > > > > > > > > > > a solution in this discussion thread? See how it was done >> > > for >> > > > > AWS >> > > > > > > and >> > > > > > > > > GCE, >> > > > > > > > > > > we might go the same route or use a more contemporary and >> > > > > > > > > easy-to-configure >> > > > > > > > > > > approach for Azure. >> > > > > > > > > > > >> > > > > > > > > > > [1] >> > > > > > > > > > > >> > > > > > > > > >> > > > > > > >> > > > > >> > > https://ignite.apache.org/docs/latest/clustering/discovery-in-the-cloud#amazon-s3-ip-finder >> > > > > > > > > > > [2] >> > > > > > > > > > > >> > > > > > > > > >> > > > > > > >> > > > > >> > > https://ignite.apache.org/docs/latest/clustering/discovery-in-the-cloud#google-compute-discovery >> > > > > > > > > > > >> > > > > > > > > > > - >> > > > > > > > > > > Denis >> > > > > > > > > > >> > > > > > > > > > >> > > > > > > > > > >> > > > > > > > > > -- >> > > > > > > > > > Regards, >> > > > > > > > > > >> > > > > > > > > > Atri >> > > > > > > > > > Apache Concerted >> > > > > > > > > >> > > > > > > > > >> > > > > > > > > >> > > > > > > > > -- >> > > > > > > > > Regards, >> > > > > > > > > >> > > > > > > > > Atri >> > > > > > > > > Apache Concerted >> > > > > > > > > >> > > > > > > >> > > > > > > -- >> > > > > > > Regards, >> > > > > > > >> > > > > > > Atri >> > > > > > > Apache Concerted >> > > > > > > >> > > > > >> > > > > -- >> > > > > Regards, >> > > > > >> > > > > Atri >> > > > > Apache Concerted >> > > > > >> > > >> > > -- >> > > Regards, >> > > >> > > Atri >> > > Apache Concerted >> > > >> >> -- >> Regards, >> >> Atri >> Apache Concerted Regards, Atri Apache Concerted |
Hello!
I'm not sure that I can see any attachment to your e-mail. Can you please re-send? We could have broken some of those, I guess, since we seem to not run integration tests for them. Regards, -- Ilya Kasnacheev пт, 2 апр. 2021 г. в 12:59, Atri Sharma <[hidden email]>: > Hello, > > Thank you for sharing. > > I was finally able to replicate the issue. However, I tried with other > IPFinders and ran into the same problem (attached are the logs). > > Not sure what is causing this? > > Atri > > On Fri, Apr 2, 2021 at 2:29 PM Ilya Kasnacheev > <[hidden email]> wrote: > > > > Hello! > > > > Please find attached the log file of errors. This is yesterday's (Apr 1) > build. > > > > Regards, > > -- > > Ilya Kasnacheev > > > > > > пт, 2 апр. 2021 г. в 11:52, Atri Sharma <[hidden email]>: > >> > >> I was able to, but then, it might be that something is cached locally. > >> > >> What errors did you run into? (I am assuming you are trying with a > >> build from the latest iteration of the PR). > >> > >> Atri > >> > >> On Fri, Apr 2, 2021 at 2:19 PM Ilya Kasnacheev > >> <[hidden email]> wrote: > >> > > >> > Hello! > >> > > >> > But are you successful in running a node with Azure IP finder enabled > in > >> > configuration, starting from that release directory? > >> > > >> > I amn't. > >> > > >> > Regards, > >> > -- > >> > Ilya Kasnacheev > >> > > >> > > >> > пт, 2 апр. 2021 г. в 07:42, Atri Sharma <[hidden email]>: > >> > > >> > > Hello, > >> > > > >> > > Thank you for taking a look. > >> > > > >> > > I am not sure what the confusion is. On the current iteration of > PR, I > >> > > am able to build and release and looking in the location you > >> > > mentioned, I see: > >> > > > >> > > Atris-MacBook-Pro-15:libs atrisharma$ cd optional/ > >> > > > >> > > Atris-MacBook-Pro-15:optional atrisharma$ cd ignite-azure/ > >> > > > >> > > Atris-MacBook-Pro-15:ignite-azure atrisharma$ ls > >> > > > >> > > README.txt azure-storage-blob-12.0.0.jar > ignite-azure-2.11.0-SNAPSHOT.jar > >> > > > >> > > > >> > > Attached is the screenshot. > >> > > > >> > > I have fixed the comments on the PR, please see and let me know. > >> > > > >> > > Atri > >> > > > >> > > On Thu, Apr 1, 2021 at 8:21 PM Ilya Kasnacheev > >> > > <[hidden email]> wrote: > >> > > > > >> > > > Hello! > >> > > > > >> > > > Were you successful with using it from the deliverable of mvn > initialize > >> > > > -Prelease? > >> > > > > >> > > > Please refer for example to aws module dependencies section: > >> > > > > >> > > > <dependency> > >> > > > <groupId>com.amazonaws</groupId> > >> > > > <artifactId>aws-java-sdk-core</artifactId> > >> > > > <version>${aws.sdk.version}</version> > >> > > > </dependency> > >> > > > > >> > > > <dependency> > >> > > > <groupId>com.amazonaws</groupId> > >> > > > <artifactId>aws-java-sdk-s3</artifactId> > >> > > > <version>${aws.sdk.version}</version> > >> > > > </dependency> > >> > > > > >> > > > <dependency> > >> > > > <groupId>com.amazonaws</groupId> > >> > > > <artifactId>aws-java-sdk-ec2</artifactId> > >> > > > <version>${aws.sdk.version}</version> > >> > > > </dependency> > >> > > > > >> > > > <dependency> > >> > > > <groupId>com.amazonaws</groupId> > >> > > > <artifactId>aws-java-sdk-elasticloadbalancing</artifactId> > >> > > > <version>${aws.sdk.version}</version> > >> > > > </dependency> > >> > > > > >> > > > <dependency> > >> > > > <groupId>com.amazonaws</groupId> > >> > > > <artifactId>aws-java-sdk-elasticloadbalancingv2</artifactId> > >> > > > <version>${aws.sdk.version}</version> > >> > > > </dependency> > >> > > > > >> > > > <dependency> > >> > > > <groupId>com.amazonaws</groupId> > >> > > > <artifactId>aws-java-sdk-kms</artifactId> > >> > > > <version>${aws.sdk.version}</version> > >> > > > </dependency> > >> > > > > >> > > > <!-- Do not remove com.fasterxml.jackson.core:jackson-core it is > >> > > > required at runtime --> > >> > > > <dependency> > >> > > > <groupId>com.fasterxml.jackson.core</groupId> > >> > > > <artifactId>jackson-core</artifactId> > >> > > > <version>${jackson.version}</version> > >> > > > </dependency> > >> > > > > >> > > > <!-- Do not remove com.fasterxml.jackson.core:jackson-annotations > it > >> > > > is required at runtime --> > >> > > > <dependency> > >> > > > <groupId>com.fasterxml.jackson.core</groupId> > >> > > > <artifactId>jackson-annotations</artifactId> > >> > > > <version>${jackson.version}</version> > >> > > > </dependency> > >> > > > > >> > > > <!-- Do not remove com.fasterxml.jackson.core:jackson-databind it > is > >> > > > required at runtime --> > >> > > > <dependency> > >> > > > <groupId>com.fasterxml.jackson.core</groupId> > >> > > > <artifactId>jackson-databind</artifactId> > >> > > > <version>${jackson.version}</version> > >> > > > </dependency> > >> > > > > >> > > > <dependency> > >> > > > <groupId>com.amazonaws</groupId> > >> > > > <artifactId>aws-encryption-sdk-java</artifactId> > >> > > > <version>${aws.encryption.sdk.version}</version> > >> > > > <exclusions> > >> > > > <exclusion> > >> > > > <groupId>org.bouncycastle</groupId> > >> > > > <artifactId>bcprov-ext-jdk15on</artifactId> > >> > > > </exclusion> > >> > > > </exclusions> > >> > > > </dependency> > >> > > > > >> > > > <dependency> > >> > > > <groupId>org.bouncycastle</groupId> > >> > > > <artifactId>bcprov-ext-jdk15on</artifactId> > >> > > > <version>${bouncycastle.version}</version> > >> > > > </dependency> > >> > > > > >> > > > <dependency> > >> > > > <groupId>joda-time</groupId> > >> > > > <artifactId>joda-time</artifactId> > >> > > > <version>2.8.1</version> > >> > > > </dependency> > >> > > > > >> > > > <dependency> > >> > > > <groupId>org.apache.httpcomponents</groupId> > >> > > > <artifactId>httpclient</artifactId> > >> > > > <version>${httpclient.version}</version> > >> > > > </dependency> > >> > > > > >> > > > <dependency> > >> > > > <groupId>org.apache.httpcomponents</groupId> > >> > > > <artifactId>httpcore</artifactId> > >> > > > <version>${httpcore.version}</version> > >> > > > </dependency> > >> > > > > >> > > > azure module should likewise list all of its actual dependencies. > Right > >> > > now > >> > > > the module directory is empty so I'm not even going to test to > run it in > >> > > > stand-alone mode: > >> > > > ~/Downloads/apache-ignite-2.11.0-SNAPSHOT-bin% ls > >> > > libs/optional/ignite-azure > >> > > > azure-storage-blob-12.0.0.jar ignite-azure-2.11.0-SNAPSHOT.jar > >> > > README.txt > >> > > > > >> > > > Please address this issue along with other things which I have > also > >> > > > commented on the PR. > >> > > > > >> > > > Regards, > >> > > > -- > >> > > > Ilya Kasnacheev > >> > > > > >> > > > > >> > > > ср, 31 мар. 2021 г. в 08:17, Atri Sharma <[hidden email]>: > >> > > > > >> > > > > Thank you for the review. > >> > > > > > >> > > > > I have updated the PR. Please see. > >> > > > > > >> > > > > On Mon, Mar 29, 2021 at 8:27 PM Ilya Kasnacheev > >> > > > > <[hidden email]> wrote: > >> > > > > > > >> > > > > > Hello! > >> > > > > > > >> > > > > > I have left some more comments after trying this change on a > real > >> > > Azure > >> > > > > > cluster. > >> > > > > > > >> > > > > > Regards, > >> > > > > > -- > >> > > > > > Ilya Kasnacheev > >> > > > > > > >> > > > > > > >> > > > > > вт, 23 мар. 2021 г. в 18:32, Atri Sharma <[hidden email]>: > >> > > > > > > >> > > > > > > Thank you! > >> > > > > > > > >> > > > > > > I have updated the PR. Please see and let me know. > >> > > > > > > > >> > > > > > > On Tue, Mar 23, 2021 at 4:27 PM Ilya Kasnacheev > >> > > > > > > <[hidden email]> wrote: > >> > > > > > > > > >> > > > > > > > Hello! > >> > > > > > > > > >> > > > > > > > I am going to check this change out when I have time, > using my > >> > > Azure > >> > > > > > > > account. > >> > > > > > > > > >> > > > > > > > Regards, > >> > > > > > > > -- > >> > > > > > > > Ilya Kasnacheev > >> > > > > > > > > >> > > > > > > > > >> > > > > > > > вт, 23 мар. 2021 г. в 07:20, Atri Sharma <[hidden email] > >: > >> > > > > > > > > >> > > > > > > > > Gentle reminder on this -- please help in reviewing > this. > >> > > > > > > > > > >> > > > > > > > > On Fri, Mar 19, 2021 at 10:23 AM Atri Sharma < > [hidden email]> > >> > > > > wrote: > >> > > > > > > > > > > >> > > > > > > > > > Thanks Denis. > >> > > > > > > > > > > >> > > > > > > > > > I have raised a PR for the same: > >> > > > > > > > > > > >> > > > > > > > > > https://github.com/apache/ignite/pull/8897 > >> > > > > > > > > > > >> > > > > > > > > > Regards, > >> > > > > > > > > > > >> > > > > > > > > > Atri > >> > > > > > > > > > > >> > > > > > > > > > On Wed, Mar 10, 2021 at 1:21 AM Denis Magda < > >> > > [hidden email]> > >> > > > > > > wrote: > >> > > > > > > > > > > > >> > > > > > > > > > > Atri, > >> > > > > > > > > > > > >> > > > > > > > > > > Let's discuss the subj together with the community. > Ignite > >> > > > > already > >> > > > > > > > > supports > >> > > > > > > > > > > AWS [1] and GCE [2] IP Finders out of the box, but > the > >> > > Azure > >> > > > > one is > >> > > > > > > > > still > >> > > > > > > > > > > missing. I can confirm that the demand exists, and > rather > >> > > > > > > frequently, > >> > > > > > > > > I see > >> > > > > > > > > > > developers asking for an Azure-native IP finder for > Ignite. > >> > > > > > > > > > > > >> > > > > > > > > > > Atri, could you please research how to implement > the IP > >> > > finder > >> > > > > and > >> > > > > > > > > suggest > >> > > > > > > > > > > a solution in this discussion thread? See how it > was done > >> > > for > >> > > > > AWS > >> > > > > > > and > >> > > > > > > > > GCE, > >> > > > > > > > > > > we might go the same route or use a more > contemporary and > >> > > > > > > > > easy-to-configure > >> > > > > > > > > > > approach for Azure. > >> > > > > > > > > > > > >> > > > > > > > > > > [1] > >> > > > > > > > > > > > >> > > > > > > > > > >> > > > > > > > >> > > > > > >> > > > https://ignite.apache.org/docs/latest/clustering/discovery-in-the-cloud#amazon-s3-ip-finder > >> > > > > > > > > > > [2] > >> > > > > > > > > > > > >> > > > > > > > > > >> > > > > > > > >> > > > > > >> > > > https://ignite.apache.org/docs/latest/clustering/discovery-in-the-cloud#google-compute-discovery > >> > > > > > > > > > > > >> > > > > > > > > > > - > >> > > > > > > > > > > Denis > >> > > > > > > > > > > >> > > > > > > > > > > >> > > > > > > > > > > >> > > > > > > > > > -- > >> > > > > > > > > > Regards, > >> > > > > > > > > > > >> > > > > > > > > > Atri > >> > > > > > > > > > Apache Concerted > >> > > > > > > > > > >> > > > > > > > > > >> > > > > > > > > > >> > > > > > > > > -- > >> > > > > > > > > Regards, > >> > > > > > > > > > >> > > > > > > > > Atri > >> > > > > > > > > Apache Concerted > >> > > > > > > > > > >> > > > > > > > >> > > > > > > -- > >> > > > > > > Regards, > >> > > > > > > > >> > > > > > > Atri > >> > > > > > > Apache Concerted > >> > > > > > > > >> > > > > > >> > > > > -- > >> > > > > Regards, > >> > > > > > >> > > > > Atri > >> > > > > Apache Concerted > >> > > > > > >> > > > >> > > -- > >> > > Regards, > >> > > > >> > > Atri > >> > > Apache Concerted > >> > > > >> > >> -- > >> Regards, > >> > >> Atri > >> Apache Concerted > > -- > Regards, > > Atri > Apache Concerted > |
Hello again!
I re-checked our cloud discovery options by moving ignite-aws, ignite-gce, ignite-cloud directories from lib/optional to lib/ and trying to run Ignite with simple config taken from examples. The results are the following: ignite-aws seems to work (complains about unknown key) ignite-gce seems to work too (complains about zero length key, this is before it tries any network access so maybe there are other issues down the path) ignite-cloud (jclouds) DOES NOT work. Filed https://issues.apache.org/jira/browse/IGNITE-14481 . I guess this discovery finder is not widely used. ignite-azure, presently, will not work too. Please use ignite-aws as an example since it sees the most usage. Regards, -- Ilya Kasnacheev пн, 5 апр. 2021 г. в 16:05, Ilya Kasnacheev <[hidden email]>: > Hello! > > I'm not sure that I can see any attachment to your e-mail. Can you please > re-send? > > We could have broken some of those, I guess, since we seem to not run > integration tests for them. > > Regards, > -- > Ilya Kasnacheev > > > пт, 2 апр. 2021 г. в 12:59, Atri Sharma <[hidden email]>: > >> Hello, >> >> Thank you for sharing. >> >> I was finally able to replicate the issue. However, I tried with other >> IPFinders and ran into the same problem (attached are the logs). >> >> Not sure what is causing this? >> >> Atri >> >> On Fri, Apr 2, 2021 at 2:29 PM Ilya Kasnacheev >> <[hidden email]> wrote: >> > >> > Hello! >> > >> > Please find attached the log file of errors. This is yesterday's (Apr >> 1) build. >> > >> > Regards, >> > -- >> > Ilya Kasnacheev >> > >> > >> > пт, 2 апр. 2021 г. в 11:52, Atri Sharma <[hidden email]>: >> >> >> >> I was able to, but then, it might be that something is cached locally. >> >> >> >> What errors did you run into? (I am assuming you are trying with a >> >> build from the latest iteration of the PR). >> >> >> >> Atri >> >> >> >> On Fri, Apr 2, 2021 at 2:19 PM Ilya Kasnacheev >> >> <[hidden email]> wrote: >> >> > >> >> > Hello! >> >> > >> >> > But are you successful in running a node with Azure IP finder >> enabled in >> >> > configuration, starting from that release directory? >> >> > >> >> > I amn't. >> >> > >> >> > Regards, >> >> > -- >> >> > Ilya Kasnacheev >> >> > >> >> > >> >> > пт, 2 апр. 2021 г. в 07:42, Atri Sharma <[hidden email]>: >> >> > >> >> > > Hello, >> >> > > >> >> > > Thank you for taking a look. >> >> > > >> >> > > I am not sure what the confusion is. On the current iteration of >> PR, I >> >> > > am able to build and release and looking in the location you >> >> > > mentioned, I see: >> >> > > >> >> > > Atris-MacBook-Pro-15:libs atrisharma$ cd optional/ >> >> > > >> >> > > Atris-MacBook-Pro-15:optional atrisharma$ cd ignite-azure/ >> >> > > >> >> > > Atris-MacBook-Pro-15:ignite-azure atrisharma$ ls >> >> > > >> >> > > README.txt azure-storage-blob-12.0.0.jar >> ignite-azure-2.11.0-SNAPSHOT.jar >> >> > > >> >> > > >> >> > > Attached is the screenshot. >> >> > > >> >> > > I have fixed the comments on the PR, please see and let me know. >> >> > > >> >> > > Atri >> >> > > >> >> > > On Thu, Apr 1, 2021 at 8:21 PM Ilya Kasnacheev >> >> > > <[hidden email]> wrote: >> >> > > > >> >> > > > Hello! >> >> > > > >> >> > > > Were you successful with using it from the deliverable of mvn >> initialize >> >> > > > -Prelease? >> >> > > > >> >> > > > Please refer for example to aws module dependencies section: >> >> > > > >> >> > > > <dependency> >> >> > > > <groupId>com.amazonaws</groupId> >> >> > > > <artifactId>aws-java-sdk-core</artifactId> >> >> > > > <version>${aws.sdk.version}</version> >> >> > > > </dependency> >> >> > > > >> >> > > > <dependency> >> >> > > > <groupId>com.amazonaws</groupId> >> >> > > > <artifactId>aws-java-sdk-s3</artifactId> >> >> > > > <version>${aws.sdk.version}</version> >> >> > > > </dependency> >> >> > > > >> >> > > > <dependency> >> >> > > > <groupId>com.amazonaws</groupId> >> >> > > > <artifactId>aws-java-sdk-ec2</artifactId> >> >> > > > <version>${aws.sdk.version}</version> >> >> > > > </dependency> >> >> > > > >> >> > > > <dependency> >> >> > > > <groupId>com.amazonaws</groupId> >> >> > > > <artifactId>aws-java-sdk-elasticloadbalancing</artifactId> >> >> > > > <version>${aws.sdk.version}</version> >> >> > > > </dependency> >> >> > > > >> >> > > > <dependency> >> >> > > > <groupId>com.amazonaws</groupId> >> >> > > > <artifactId>aws-java-sdk-elasticloadbalancingv2</artifactId> >> >> > > > <version>${aws.sdk.version}</version> >> >> > > > </dependency> >> >> > > > >> >> > > > <dependency> >> >> > > > <groupId>com.amazonaws</groupId> >> >> > > > <artifactId>aws-java-sdk-kms</artifactId> >> >> > > > <version>${aws.sdk.version}</version> >> >> > > > </dependency> >> >> > > > >> >> > > > <!-- Do not remove com.fasterxml.jackson.core:jackson-core it is >> >> > > > required at runtime --> >> >> > > > <dependency> >> >> > > > <groupId>com.fasterxml.jackson.core</groupId> >> >> > > > <artifactId>jackson-core</artifactId> >> >> > > > <version>${jackson.version}</version> >> >> > > > </dependency> >> >> > > > >> >> > > > <!-- Do not remove >> com.fasterxml.jackson.core:jackson-annotations it >> >> > > > is required at runtime --> >> >> > > > <dependency> >> >> > > > <groupId>com.fasterxml.jackson.core</groupId> >> >> > > > <artifactId>jackson-annotations</artifactId> >> >> > > > <version>${jackson.version}</version> >> >> > > > </dependency> >> >> > > > >> >> > > > <!-- Do not remove com.fasterxml.jackson.core:jackson-databind >> it is >> >> > > > required at runtime --> >> >> > > > <dependency> >> >> > > > <groupId>com.fasterxml.jackson.core</groupId> >> >> > > > <artifactId>jackson-databind</artifactId> >> >> > > > <version>${jackson.version}</version> >> >> > > > </dependency> >> >> > > > >> >> > > > <dependency> >> >> > > > <groupId>com.amazonaws</groupId> >> >> > > > <artifactId>aws-encryption-sdk-java</artifactId> >> >> > > > <version>${aws.encryption.sdk.version}</version> >> >> > > > <exclusions> >> >> > > > <exclusion> >> >> > > > <groupId>org.bouncycastle</groupId> >> >> > > > <artifactId>bcprov-ext-jdk15on</artifactId> >> >> > > > </exclusion> >> >> > > > </exclusions> >> >> > > > </dependency> >> >> > > > >> >> > > > <dependency> >> >> > > > <groupId>org.bouncycastle</groupId> >> >> > > > <artifactId>bcprov-ext-jdk15on</artifactId> >> >> > > > <version>${bouncycastle.version}</version> >> >> > > > </dependency> >> >> > > > >> >> > > > <dependency> >> >> > > > <groupId>joda-time</groupId> >> >> > > > <artifactId>joda-time</artifactId> >> >> > > > <version>2.8.1</version> >> >> > > > </dependency> >> >> > > > >> >> > > > <dependency> >> >> > > > <groupId>org.apache.httpcomponents</groupId> >> >> > > > <artifactId>httpclient</artifactId> >> >> > > > <version>${httpclient.version}</version> >> >> > > > </dependency> >> >> > > > >> >> > > > <dependency> >> >> > > > <groupId>org.apache.httpcomponents</groupId> >> >> > > > <artifactId>httpcore</artifactId> >> >> > > > <version>${httpcore.version}</version> >> >> > > > </dependency> >> >> > > > >> >> > > > azure module should likewise list all of its actual >> dependencies. Right >> >> > > now >> >> > > > the module directory is empty so I'm not even going to test to >> run it in >> >> > > > stand-alone mode: >> >> > > > ~/Downloads/apache-ignite-2.11.0-SNAPSHOT-bin% ls >> >> > > libs/optional/ignite-azure >> >> > > > azure-storage-blob-12.0.0.jar ignite-azure-2.11.0-SNAPSHOT.jar >> >> > > README.txt >> >> > > > >> >> > > > Please address this issue along with other things which I have >> also >> >> > > > commented on the PR. >> >> > > > >> >> > > > Regards, >> >> > > > -- >> >> > > > Ilya Kasnacheev >> >> > > > >> >> > > > >> >> > > > ср, 31 мар. 2021 г. в 08:17, Atri Sharma <[hidden email]>: >> >> > > > >> >> > > > > Thank you for the review. >> >> > > > > >> >> > > > > I have updated the PR. Please see. >> >> > > > > >> >> > > > > On Mon, Mar 29, 2021 at 8:27 PM Ilya Kasnacheev >> >> > > > > <[hidden email]> wrote: >> >> > > > > > >> >> > > > > > Hello! >> >> > > > > > >> >> > > > > > I have left some more comments after trying this change on a >> real >> >> > > Azure >> >> > > > > > cluster. >> >> > > > > > >> >> > > > > > Regards, >> >> > > > > > -- >> >> > > > > > Ilya Kasnacheev >> >> > > > > > >> >> > > > > > >> >> > > > > > вт, 23 мар. 2021 г. в 18:32, Atri Sharma <[hidden email]>: >> >> > > > > > >> >> > > > > > > Thank you! >> >> > > > > > > >> >> > > > > > > I have updated the PR. Please see and let me know. >> >> > > > > > > >> >> > > > > > > On Tue, Mar 23, 2021 at 4:27 PM Ilya Kasnacheev >> >> > > > > > > <[hidden email]> wrote: >> >> > > > > > > > >> >> > > > > > > > Hello! >> >> > > > > > > > >> >> > > > > > > > I am going to check this change out when I have time, >> using my >> >> > > Azure >> >> > > > > > > > account. >> >> > > > > > > > >> >> > > > > > > > Regards, >> >> > > > > > > > -- >> >> > > > > > > > Ilya Kasnacheev >> >> > > > > > > > >> >> > > > > > > > >> >> > > > > > > > вт, 23 мар. 2021 г. в 07:20, Atri Sharma < >> [hidden email]>: >> >> > > > > > > > >> >> > > > > > > > > Gentle reminder on this -- please help in reviewing >> this. >> >> > > > > > > > > >> >> > > > > > > > > On Fri, Mar 19, 2021 at 10:23 AM Atri Sharma < >> [hidden email]> >> >> > > > > wrote: >> >> > > > > > > > > > >> >> > > > > > > > > > Thanks Denis. >> >> > > > > > > > > > >> >> > > > > > > > > > I have raised a PR for the same: >> >> > > > > > > > > > >> >> > > > > > > > > > https://github.com/apache/ignite/pull/8897 >> >> > > > > > > > > > >> >> > > > > > > > > > Regards, >> >> > > > > > > > > > >> >> > > > > > > > > > Atri >> >> > > > > > > > > > >> >> > > > > > > > > > On Wed, Mar 10, 2021 at 1:21 AM Denis Magda < >> >> > > [hidden email]> >> >> > > > > > > wrote: >> >> > > > > > > > > > > >> >> > > > > > > > > > > Atri, >> >> > > > > > > > > > > >> >> > > > > > > > > > > Let's discuss the subj together with the >> community. Ignite >> >> > > > > already >> >> > > > > > > > > supports >> >> > > > > > > > > > > AWS [1] and GCE [2] IP Finders out of the box, but >> the >> >> > > Azure >> >> > > > > one is >> >> > > > > > > > > still >> >> > > > > > > > > > > missing. I can confirm that the demand exists, and >> rather >> >> > > > > > > frequently, >> >> > > > > > > > > I see >> >> > > > > > > > > > > developers asking for an Azure-native IP finder >> for Ignite. >> >> > > > > > > > > > > >> >> > > > > > > > > > > Atri, could you please research how to implement >> the IP >> >> > > finder >> >> > > > > and >> >> > > > > > > > > suggest >> >> > > > > > > > > > > a solution in this discussion thread? See how it >> was done >> >> > > for >> >> > > > > AWS >> >> > > > > > > and >> >> > > > > > > > > GCE, >> >> > > > > > > > > > > we might go the same route or use a more >> contemporary and >> >> > > > > > > > > easy-to-configure >> >> > > > > > > > > > > approach for Azure. >> >> > > > > > > > > > > >> >> > > > > > > > > > > [1] >> >> > > > > > > > > > > >> >> > > > > > > > > >> >> > > > > > > >> >> > > > > >> >> > > >> https://ignite.apache.org/docs/latest/clustering/discovery-in-the-cloud#amazon-s3-ip-finder >> >> > > > > > > > > > > [2] >> >> > > > > > > > > > > >> >> > > > > > > > > >> >> > > > > > > >> >> > > > > >> >> > > >> https://ignite.apache.org/docs/latest/clustering/discovery-in-the-cloud#google-compute-discovery >> >> > > > > > > > > > > >> >> > > > > > > > > > > - >> >> > > > > > > > > > > Denis >> >> > > > > > > > > > >> >> > > > > > > > > > >> >> > > > > > > > > > >> >> > > > > > > > > > -- >> >> > > > > > > > > > Regards, >> >> > > > > > > > > > >> >> > > > > > > > > > Atri >> >> > > > > > > > > > Apache Concerted >> >> > > > > > > > > >> >> > > > > > > > > >> >> > > > > > > > > >> >> > > > > > > > > -- >> >> > > > > > > > > Regards, >> >> > > > > > > > > >> >> > > > > > > > > Atri >> >> > > > > > > > > Apache Concerted >> >> > > > > > > > > >> >> > > > > > > >> >> > > > > > > -- >> >> > > > > > > Regards, >> >> > > > > > > >> >> > > > > > > Atri >> >> > > > > > > Apache Concerted >> >> > > > > > > >> >> > > > > >> >> > > > > -- >> >> > > > > Regards, >> >> > > > > >> >> > > > > Atri >> >> > > > > Apache Concerted >> >> > > > > >> >> > > >> >> > > -- >> >> > > Regards, >> >> > > >> >> > > Atri >> >> > > Apache Concerted >> >> > > >> >> >> >> -- >> >> Regards, >> >> >> >> Atri >> >> Apache Concerted >> >> -- >> Regards, >> >> Atri >> Apache Concerted >> > |
Hi Ilya,
Thanks for taking a look. I was able to resolve dependencies (Thanks, Sam!) and have updated the PR. Copying the jars from ignite-azure to libs works for me. Please see and let me know your thoughts. Regards, Atri On Mon, Apr 5, 2021 at 9:24 PM Ilya Kasnacheev <[hidden email]> wrote: > > Hello again! > > I re-checked our cloud discovery options by moving ignite-aws, ignite-gce, > ignite-cloud directories from lib/optional to lib/ and trying to run Ignite > with simple config taken from examples. > > The results are the following: > ignite-aws seems to work (complains about unknown key) > ignite-gce seems to work too (complains about zero length key, this is > before it tries any network access so maybe there are other issues down the > path) > ignite-cloud (jclouds) DOES NOT work. Filed > https://issues.apache.org/jira/browse/IGNITE-14481 . I guess this discovery > finder is not widely used. > ignite-azure, presently, will not work too. Please use ignite-aws as an > example since it sees the most usage. > > Regards, > -- > Ilya Kasnacheev > > > пн, 5 апр. 2021 г. в 16:05, Ilya Kasnacheev <[hidden email]>: > > > Hello! > > > > I'm not sure that I can see any attachment to your e-mail. Can you please > > re-send? > > > > We could have broken some of those, I guess, since we seem to not run > > integration tests for them. > > > > Regards, > > -- > > Ilya Kasnacheev > > > > > > пт, 2 апр. 2021 г. в 12:59, Atri Sharma <[hidden email]>: > > > >> Hello, > >> > >> Thank you for sharing. > >> > >> I was finally able to replicate the issue. However, I tried with other > >> IPFinders and ran into the same problem (attached are the logs). > >> > >> Not sure what is causing this? > >> > >> Atri > >> > >> On Fri, Apr 2, 2021 at 2:29 PM Ilya Kasnacheev > >> <[hidden email]> wrote: > >> > > >> > Hello! > >> > > >> > Please find attached the log file of errors. This is yesterday's (Apr > >> 1) build. > >> > > >> > Regards, > >> > -- > >> > Ilya Kasnacheev > >> > > >> > > >> > пт, 2 апр. 2021 г. в 11:52, Atri Sharma <[hidden email]>: > >> >> > >> >> I was able to, but then, it might be that something is cached locally. > >> >> > >> >> What errors did you run into? (I am assuming you are trying with a > >> >> build from the latest iteration of the PR). > >> >> > >> >> Atri > >> >> > >> >> On Fri, Apr 2, 2021 at 2:19 PM Ilya Kasnacheev > >> >> <[hidden email]> wrote: > >> >> > > >> >> > Hello! > >> >> > > >> >> > But are you successful in running a node with Azure IP finder > >> enabled in > >> >> > configuration, starting from that release directory? > >> >> > > >> >> > I amn't. > >> >> > > >> >> > Regards, > >> >> > -- > >> >> > Ilya Kasnacheev > >> >> > > >> >> > > >> >> > пт, 2 апр. 2021 г. в 07:42, Atri Sharma <[hidden email]>: > >> >> > > >> >> > > Hello, > >> >> > > > >> >> > > Thank you for taking a look. > >> >> > > > >> >> > > I am not sure what the confusion is. On the current iteration of > >> PR, I > >> >> > > am able to build and release and looking in the location you > >> >> > > mentioned, I see: > >> >> > > > >> >> > > Atris-MacBook-Pro-15:libs atrisharma$ cd optional/ > >> >> > > > >> >> > > Atris-MacBook-Pro-15:optional atrisharma$ cd ignite-azure/ > >> >> > > > >> >> > > Atris-MacBook-Pro-15:ignite-azure atrisharma$ ls > >> >> > > > >> >> > > README.txt azure-storage-blob-12.0.0.jar > >> ignite-azure-2.11.0-SNAPSHOT.jar > >> >> > > > >> >> > > > >> >> > > Attached is the screenshot. > >> >> > > > >> >> > > I have fixed the comments on the PR, please see and let me know. > >> >> > > > >> >> > > Atri > >> >> > > > >> >> > > On Thu, Apr 1, 2021 at 8:21 PM Ilya Kasnacheev > >> >> > > <[hidden email]> wrote: > >> >> > > > > >> >> > > > Hello! > >> >> > > > > >> >> > > > Were you successful with using it from the deliverable of mvn > >> initialize > >> >> > > > -Prelease? > >> >> > > > > >> >> > > > Please refer for example to aws module dependencies section: > >> >> > > > > >> >> > > > <dependency> > >> >> > > > <groupId>com.amazonaws</groupId> > >> >> > > > <artifactId>aws-java-sdk-core</artifactId> > >> >> > > > <version>${aws.sdk.version}</version> > >> >> > > > </dependency> > >> >> > > > > >> >> > > > <dependency> > >> >> > > > <groupId>com.amazonaws</groupId> > >> >> > > > <artifactId>aws-java-sdk-s3</artifactId> > >> >> > > > <version>${aws.sdk.version}</version> > >> >> > > > </dependency> > >> >> > > > > >> >> > > > <dependency> > >> >> > > > <groupId>com.amazonaws</groupId> > >> >> > > > <artifactId>aws-java-sdk-ec2</artifactId> > >> >> > > > <version>${aws.sdk.version}</version> > >> >> > > > </dependency> > >> >> > > > > >> >> > > > <dependency> > >> >> > > > <groupId>com.amazonaws</groupId> > >> >> > > > <artifactId>aws-java-sdk-elasticloadbalancing</artifactId> > >> >> > > > <version>${aws.sdk.version}</version> > >> >> > > > </dependency> > >> >> > > > > >> >> > > > <dependency> > >> >> > > > <groupId>com.amazonaws</groupId> > >> >> > > > <artifactId>aws-java-sdk-elasticloadbalancingv2</artifactId> > >> >> > > > <version>${aws.sdk.version}</version> > >> >> > > > </dependency> > >> >> > > > > >> >> > > > <dependency> > >> >> > > > <groupId>com.amazonaws</groupId> > >> >> > > > <artifactId>aws-java-sdk-kms</artifactId> > >> >> > > > <version>${aws.sdk.version}</version> > >> >> > > > </dependency> > >> >> > > > > >> >> > > > <!-- Do not remove com.fasterxml.jackson.core:jackson-core it is > >> >> > > > required at runtime --> > >> >> > > > <dependency> > >> >> > > > <groupId>com.fasterxml.jackson.core</groupId> > >> >> > > > <artifactId>jackson-core</artifactId> > >> >> > > > <version>${jackson.version}</version> > >> >> > > > </dependency> > >> >> > > > > >> >> > > > <!-- Do not remove > >> com.fasterxml.jackson.core:jackson-annotations it > >> >> > > > is required at runtime --> > >> >> > > > <dependency> > >> >> > > > <groupId>com.fasterxml.jackson.core</groupId> > >> >> > > > <artifactId>jackson-annotations</artifactId> > >> >> > > > <version>${jackson.version}</version> > >> >> > > > </dependency> > >> >> > > > > >> >> > > > <!-- Do not remove com.fasterxml.jackson.core:jackson-databind > >> it is > >> >> > > > required at runtime --> > >> >> > > > <dependency> > >> >> > > > <groupId>com.fasterxml.jackson.core</groupId> > >> >> > > > <artifactId>jackson-databind</artifactId> > >> >> > > > <version>${jackson.version}</version> > >> >> > > > </dependency> > >> >> > > > > >> >> > > > <dependency> > >> >> > > > <groupId>com.amazonaws</groupId> > >> >> > > > <artifactId>aws-encryption-sdk-java</artifactId> > >> >> > > > <version>${aws.encryption.sdk.version}</version> > >> >> > > > <exclusions> > >> >> > > > <exclusion> > >> >> > > > <groupId>org.bouncycastle</groupId> > >> >> > > > <artifactId>bcprov-ext-jdk15on</artifactId> > >> >> > > > </exclusion> > >> >> > > > </exclusions> > >> >> > > > </dependency> > >> >> > > > > >> >> > > > <dependency> > >> >> > > > <groupId>org.bouncycastle</groupId> > >> >> > > > <artifactId>bcprov-ext-jdk15on</artifactId> > >> >> > > > <version>${bouncycastle.version}</version> > >> >> > > > </dependency> > >> >> > > > > >> >> > > > <dependency> > >> >> > > > <groupId>joda-time</groupId> > >> >> > > > <artifactId>joda-time</artifactId> > >> >> > > > <version>2.8.1</version> > >> >> > > > </dependency> > >> >> > > > > >> >> > > > <dependency> > >> >> > > > <groupId>org.apache.httpcomponents</groupId> > >> >> > > > <artifactId>httpclient</artifactId> > >> >> > > > <version>${httpclient.version}</version> > >> >> > > > </dependency> > >> >> > > > > >> >> > > > <dependency> > >> >> > > > <groupId>org.apache.httpcomponents</groupId> > >> >> > > > <artifactId>httpcore</artifactId> > >> >> > > > <version>${httpcore.version}</version> > >> >> > > > </dependency> > >> >> > > > > >> >> > > > azure module should likewise list all of its actual > >> dependencies. Right > >> >> > > now > >> >> > > > the module directory is empty so I'm not even going to test to > >> run it in > >> >> > > > stand-alone mode: > >> >> > > > ~/Downloads/apache-ignite-2.11.0-SNAPSHOT-bin% ls > >> >> > > libs/optional/ignite-azure > >> >> > > > azure-storage-blob-12.0.0.jar ignite-azure-2.11.0-SNAPSHOT.jar > >> >> > > README.txt > >> >> > > > > >> >> > > > Please address this issue along with other things which I have > >> also > >> >> > > > commented on the PR. > >> >> > > > > >> >> > > > Regards, > >> >> > > > -- > >> >> > > > Ilya Kasnacheev > >> >> > > > > >> >> > > > > >> >> > > > ср, 31 мар. 2021 г. в 08:17, Atri Sharma <[hidden email]>: > >> >> > > > > >> >> > > > > Thank you for the review. > >> >> > > > > > >> >> > > > > I have updated the PR. Please see. > >> >> > > > > > >> >> > > > > On Mon, Mar 29, 2021 at 8:27 PM Ilya Kasnacheev > >> >> > > > > <[hidden email]> wrote: > >> >> > > > > > > >> >> > > > > > Hello! > >> >> > > > > > > >> >> > > > > > I have left some more comments after trying this change on a > >> real > >> >> > > Azure > >> >> > > > > > cluster. > >> >> > > > > > > >> >> > > > > > Regards, > >> >> > > > > > -- > >> >> > > > > > Ilya Kasnacheev > >> >> > > > > > > >> >> > > > > > > >> >> > > > > > вт, 23 мар. 2021 г. в 18:32, Atri Sharma <[hidden email]>: > >> >> > > > > > > >> >> > > > > > > Thank you! > >> >> > > > > > > > >> >> > > > > > > I have updated the PR. Please see and let me know. > >> >> > > > > > > > >> >> > > > > > > On Tue, Mar 23, 2021 at 4:27 PM Ilya Kasnacheev > >> >> > > > > > > <[hidden email]> wrote: > >> >> > > > > > > > > >> >> > > > > > > > Hello! > >> >> > > > > > > > > >> >> > > > > > > > I am going to check this change out when I have time, > >> using my > >> >> > > Azure > >> >> > > > > > > > account. > >> >> > > > > > > > > >> >> > > > > > > > Regards, > >> >> > > > > > > > -- > >> >> > > > > > > > Ilya Kasnacheev > >> >> > > > > > > > > >> >> > > > > > > > > >> >> > > > > > > > вт, 23 мар. 2021 г. в 07:20, Atri Sharma < > >> [hidden email]>: > >> >> > > > > > > > > >> >> > > > > > > > > Gentle reminder on this -- please help in reviewing > >> this. > >> >> > > > > > > > > > >> >> > > > > > > > > On Fri, Mar 19, 2021 at 10:23 AM Atri Sharma < > >> [hidden email]> > >> >> > > > > wrote: > >> >> > > > > > > > > > > >> >> > > > > > > > > > Thanks Denis. > >> >> > > > > > > > > > > >> >> > > > > > > > > > I have raised a PR for the same: > >> >> > > > > > > > > > > >> >> > > > > > > > > > https://github.com/apache/ignite/pull/8897 > >> >> > > > > > > > > > > >> >> > > > > > > > > > Regards, > >> >> > > > > > > > > > > >> >> > > > > > > > > > Atri > >> >> > > > > > > > > > > >> >> > > > > > > > > > On Wed, Mar 10, 2021 at 1:21 AM Denis Magda < > >> >> > > [hidden email]> > >> >> > > > > > > wrote: > >> >> > > > > > > > > > > > >> >> > > > > > > > > > > Atri, > >> >> > > > > > > > > > > > >> >> > > > > > > > > > > Let's discuss the subj together with the > >> community. Ignite > >> >> > > > > already > >> >> > > > > > > > > supports > >> >> > > > > > > > > > > AWS [1] and GCE [2] IP Finders out of the box, but > >> the > >> >> > > Azure > >> >> > > > > one is > >> >> > > > > > > > > still > >> >> > > > > > > > > > > missing. I can confirm that the demand exists, and > >> rather > >> >> > > > > > > frequently, > >> >> > > > > > > > > I see > >> >> > > > > > > > > > > developers asking for an Azure-native IP finder > >> for Ignite. > >> >> > > > > > > > > > > > >> >> > > > > > > > > > > Atri, could you please research how to implement > >> the IP > >> >> > > finder > >> >> > > > > and > >> >> > > > > > > > > suggest > >> >> > > > > > > > > > > a solution in this discussion thread? See how it > >> was done > >> >> > > for > >> >> > > > > AWS > >> >> > > > > > > and > >> >> > > > > > > > > GCE, > >> >> > > > > > > > > > > we might go the same route or use a more > >> contemporary and > >> >> > > > > > > > > easy-to-configure > >> >> > > > > > > > > > > approach for Azure. > >> >> > > > > > > > > > > > >> >> > > > > > > > > > > [1] > >> >> > > > > > > > > > > > >> >> > > > > > > > > > >> >> > > > > > > > >> >> > > > > > >> >> > > > >> https://ignite.apache.org/docs/latest/clustering/discovery-in-the-cloud#amazon-s3-ip-finder > >> >> > > > > > > > > > > [2] > >> >> > > > > > > > > > > > >> >> > > > > > > > > > >> >> > > > > > > > >> >> > > > > > >> >> > > > >> https://ignite.apache.org/docs/latest/clustering/discovery-in-the-cloud#google-compute-discovery > >> >> > > > > > > > > > > > >> >> > > > > > > > > > > - > >> >> > > > > > > > > > > Denis > >> >> > > > > > > > > > > >> >> > > > > > > > > > > >> >> > > > > > > > > > > >> >> > > > > > > > > > -- > >> >> > > > > > > > > > Regards, > >> >> > > > > > > > > > > >> >> > > > > > > > > > Atri > >> >> > > > > > > > > > Apache Concerted > >> >> > > > > > > > > > >> >> > > > > > > > > > >> >> > > > > > > > > > >> >> > > > > > > > > -- > >> >> > > > > > > > > Regards, > >> >> > > > > > > > > > >> >> > > > > > > > > Atri > >> >> > > > > > > > > Apache Concerted > >> >> > > > > > > > > > >> >> > > > > > > > >> >> > > > > > > -- > >> >> > > > > > > Regards, > >> >> > > > > > > > >> >> > > > > > > Atri > >> >> > > > > > > Apache Concerted > >> >> > > > > > > > >> >> > > > > > >> >> > > > > -- > >> >> > > > > Regards, > >> >> > > > > > >> >> > > > > Atri > >> >> > > > > Apache Concerted > >> >> > > > > > >> >> > > > >> >> > > -- > >> >> > > Regards, > >> >> > > > >> >> > > Atri > >> >> > > Apache Concerted > >> >> > > > >> >> > >> >> -- > >> >> Regards, > >> >> > >> >> Atri > >> >> Apache Concerted > >> > >> -- > >> Regards, > >> > >> Atri > >> Apache Concerted > >> > > -- Regards, Atri Apache Concerted |
Hi Ilya,
Please let me know if I can help with any further iterations on the PR. Regards, Atri On Wed, Apr 7, 2021 at 5:04 PM Atri Sharma <[hidden email]> wrote: > > Hi Ilya, > > Thanks for taking a look. I was able to resolve dependencies (Thanks, > Sam!) and have updated the PR. > > Copying the jars from ignite-azure to libs works for me. > > Please see and let me know your thoughts. > > Regards, > > Atri > > On Mon, Apr 5, 2021 at 9:24 PM Ilya Kasnacheev > <[hidden email]> wrote: > > > > Hello again! > > > > I re-checked our cloud discovery options by moving ignite-aws, ignite-gce, > > ignite-cloud directories from lib/optional to lib/ and trying to run Ignite > > with simple config taken from examples. > > > > The results are the following: > > ignite-aws seems to work (complains about unknown key) > > ignite-gce seems to work too (complains about zero length key, this is > > before it tries any network access so maybe there are other issues down the > > path) > > ignite-cloud (jclouds) DOES NOT work. Filed > > https://issues.apache.org/jira/browse/IGNITE-14481 . I guess this discovery > > finder is not widely used. > > ignite-azure, presently, will not work too. Please use ignite-aws as an > > example since it sees the most usage. > > > > Regards, > > -- > > Ilya Kasnacheev > > > > > > пн, 5 апр. 2021 г. в 16:05, Ilya Kasnacheev <[hidden email]>: > > > > > Hello! > > > > > > I'm not sure that I can see any attachment to your e-mail. Can you please > > > re-send? > > > > > > We could have broken some of those, I guess, since we seem to not run > > > integration tests for them. > > > > > > Regards, > > > -- > > > Ilya Kasnacheev > > > > > > > > > пт, 2 апр. 2021 г. в 12:59, Atri Sharma <[hidden email]>: > > > > > >> Hello, > > >> > > >> Thank you for sharing. > > >> > > >> I was finally able to replicate the issue. However, I tried with other > > >> IPFinders and ran into the same problem (attached are the logs). > > >> > > >> Not sure what is causing this? > > >> > > >> Atri > > >> > > >> On Fri, Apr 2, 2021 at 2:29 PM Ilya Kasnacheev > > >> <[hidden email]> wrote: > > >> > > > >> > Hello! > > >> > > > >> > Please find attached the log file of errors. This is yesterday's (Apr > > >> 1) build. > > >> > > > >> > Regards, > > >> > -- > > >> > Ilya Kasnacheev > > >> > > > >> > > > >> > пт, 2 апр. 2021 г. в 11:52, Atri Sharma <[hidden email]>: > > >> >> > > >> >> I was able to, but then, it might be that something is cached locally. > > >> >> > > >> >> What errors did you run into? (I am assuming you are trying with a > > >> >> build from the latest iteration of the PR). > > >> >> > > >> >> Atri > > >> >> > > >> >> On Fri, Apr 2, 2021 at 2:19 PM Ilya Kasnacheev > > >> >> <[hidden email]> wrote: > > >> >> > > > >> >> > Hello! > > >> >> > > > >> >> > But are you successful in running a node with Azure IP finder > > >> enabled in > > >> >> > configuration, starting from that release directory? > > >> >> > > > >> >> > I amn't. > > >> >> > > > >> >> > Regards, > > >> >> > -- > > >> >> > Ilya Kasnacheev > > >> >> > > > >> >> > > > >> >> > пт, 2 апр. 2021 г. в 07:42, Atri Sharma <[hidden email]>: > > >> >> > > > >> >> > > Hello, > > >> >> > > > > >> >> > > Thank you for taking a look. > > >> >> > > > > >> >> > > I am not sure what the confusion is. On the current iteration of > > >> PR, I > > >> >> > > am able to build and release and looking in the location you > > >> >> > > mentioned, I see: > > >> >> > > > > >> >> > > Atris-MacBook-Pro-15:libs atrisharma$ cd optional/ > > >> >> > > > > >> >> > > Atris-MacBook-Pro-15:optional atrisharma$ cd ignite-azure/ > > >> >> > > > > >> >> > > Atris-MacBook-Pro-15:ignite-azure atrisharma$ ls > > >> >> > > > > >> >> > > README.txt azure-storage-blob-12.0.0.jar > > >> ignite-azure-2.11.0-SNAPSHOT.jar > > >> >> > > > > >> >> > > > > >> >> > > Attached is the screenshot. > > >> >> > > > > >> >> > > I have fixed the comments on the PR, please see and let me know. > > >> >> > > > > >> >> > > Atri > > >> >> > > > > >> >> > > On Thu, Apr 1, 2021 at 8:21 PM Ilya Kasnacheev > > >> >> > > <[hidden email]> wrote: > > >> >> > > > > > >> >> > > > Hello! > > >> >> > > > > > >> >> > > > Were you successful with using it from the deliverable of mvn > > >> initialize > > >> >> > > > -Prelease? > > >> >> > > > > > >> >> > > > Please refer for example to aws module dependencies section: > > >> >> > > > > > >> >> > > > <dependency> > > >> >> > > > <groupId>com.amazonaws</groupId> > > >> >> > > > <artifactId>aws-java-sdk-core</artifactId> > > >> >> > > > <version>${aws.sdk.version}</version> > > >> >> > > > </dependency> > > >> >> > > > > > >> >> > > > <dependency> > > >> >> > > > <groupId>com.amazonaws</groupId> > > >> >> > > > <artifactId>aws-java-sdk-s3</artifactId> > > >> >> > > > <version>${aws.sdk.version}</version> > > >> >> > > > </dependency> > > >> >> > > > > > >> >> > > > <dependency> > > >> >> > > > <groupId>com.amazonaws</groupId> > > >> >> > > > <artifactId>aws-java-sdk-ec2</artifactId> > > >> >> > > > <version>${aws.sdk.version}</version> > > >> >> > > > </dependency> > > >> >> > > > > > >> >> > > > <dependency> > > >> >> > > > <groupId>com.amazonaws</groupId> > > >> >> > > > <artifactId>aws-java-sdk-elasticloadbalancing</artifactId> > > >> >> > > > <version>${aws.sdk.version}</version> > > >> >> > > > </dependency> > > >> >> > > > > > >> >> > > > <dependency> > > >> >> > > > <groupId>com.amazonaws</groupId> > > >> >> > > > <artifactId>aws-java-sdk-elasticloadbalancingv2</artifactId> > > >> >> > > > <version>${aws.sdk.version}</version> > > >> >> > > > </dependency> > > >> >> > > > > > >> >> > > > <dependency> > > >> >> > > > <groupId>com.amazonaws</groupId> > > >> >> > > > <artifactId>aws-java-sdk-kms</artifactId> > > >> >> > > > <version>${aws.sdk.version}</version> > > >> >> > > > </dependency> > > >> >> > > > > > >> >> > > > <!-- Do not remove com.fasterxml.jackson.core:jackson-core it is > > >> >> > > > required at runtime --> > > >> >> > > > <dependency> > > >> >> > > > <groupId>com.fasterxml.jackson.core</groupId> > > >> >> > > > <artifactId>jackson-core</artifactId> > > >> >> > > > <version>${jackson.version}</version> > > >> >> > > > </dependency> > > >> >> > > > > > >> >> > > > <!-- Do not remove > > >> com.fasterxml.jackson.core:jackson-annotations it > > >> >> > > > is required at runtime --> > > >> >> > > > <dependency> > > >> >> > > > <groupId>com.fasterxml.jackson.core</groupId> > > >> >> > > > <artifactId>jackson-annotations</artifactId> > > >> >> > > > <version>${jackson.version}</version> > > >> >> > > > </dependency> > > >> >> > > > > > >> >> > > > <!-- Do not remove com.fasterxml.jackson.core:jackson-databind > > >> it is > > >> >> > > > required at runtime --> > > >> >> > > > <dependency> > > >> >> > > > <groupId>com.fasterxml.jackson.core</groupId> > > >> >> > > > <artifactId>jackson-databind</artifactId> > > >> >> > > > <version>${jackson.version}</version> > > >> >> > > > </dependency> > > >> >> > > > > > >> >> > > > <dependency> > > >> >> > > > <groupId>com.amazonaws</groupId> > > >> >> > > > <artifactId>aws-encryption-sdk-java</artifactId> > > >> >> > > > <version>${aws.encryption.sdk.version}</version> > > >> >> > > > <exclusions> > > >> >> > > > <exclusion> > > >> >> > > > <groupId>org.bouncycastle</groupId> > > >> >> > > > <artifactId>bcprov-ext-jdk15on</artifactId> > > >> >> > > > </exclusion> > > >> >> > > > </exclusions> > > >> >> > > > </dependency> > > >> >> > > > > > >> >> > > > <dependency> > > >> >> > > > <groupId>org.bouncycastle</groupId> > > >> >> > > > <artifactId>bcprov-ext-jdk15on</artifactId> > > >> >> > > > <version>${bouncycastle.version}</version> > > >> >> > > > </dependency> > > >> >> > > > > > >> >> > > > <dependency> > > >> >> > > > <groupId>joda-time</groupId> > > >> >> > > > <artifactId>joda-time</artifactId> > > >> >> > > > <version>2.8.1</version> > > >> >> > > > </dependency> > > >> >> > > > > > >> >> > > > <dependency> > > >> >> > > > <groupId>org.apache.httpcomponents</groupId> > > >> >> > > > <artifactId>httpclient</artifactId> > > >> >> > > > <version>${httpclient.version}</version> > > >> >> > > > </dependency> > > >> >> > > > > > >> >> > > > <dependency> > > >> >> > > > <groupId>org.apache.httpcomponents</groupId> > > >> >> > > > <artifactId>httpcore</artifactId> > > >> >> > > > <version>${httpcore.version}</version> > > >> >> > > > </dependency> > > >> >> > > > > > >> >> > > > azure module should likewise list all of its actual > > >> dependencies. Right > > >> >> > > now > > >> >> > > > the module directory is empty so I'm not even going to test to > > >> run it in > > >> >> > > > stand-alone mode: > > >> >> > > > ~/Downloads/apache-ignite-2.11.0-SNAPSHOT-bin% ls > > >> >> > > libs/optional/ignite-azure > > >> >> > > > azure-storage-blob-12.0.0.jar ignite-azure-2.11.0-SNAPSHOT.jar > > >> >> > > README.txt > > >> >> > > > > > >> >> > > > Please address this issue along with other things which I have > > >> also > > >> >> > > > commented on the PR. > > >> >> > > > > > >> >> > > > Regards, > > >> >> > > > -- > > >> >> > > > Ilya Kasnacheev > > >> >> > > > > > >> >> > > > > > >> >> > > > ср, 31 мар. 2021 г. в 08:17, Atri Sharma <[hidden email]>: > > >> >> > > > > > >> >> > > > > Thank you for the review. > > >> >> > > > > > > >> >> > > > > I have updated the PR. Please see. > > >> >> > > > > > > >> >> > > > > On Mon, Mar 29, 2021 at 8:27 PM Ilya Kasnacheev > > >> >> > > > > <[hidden email]> wrote: > > >> >> > > > > > > > >> >> > > > > > Hello! > > >> >> > > > > > > > >> >> > > > > > I have left some more comments after trying this change on a > > >> real > > >> >> > > Azure > > >> >> > > > > > cluster. > > >> >> > > > > > > > >> >> > > > > > Regards, > > >> >> > > > > > -- > > >> >> > > > > > Ilya Kasnacheev > > >> >> > > > > > > > >> >> > > > > > > > >> >> > > > > > вт, 23 мар. 2021 г. в 18:32, Atri Sharma <[hidden email]>: > > >> >> > > > > > > > >> >> > > > > > > Thank you! > > >> >> > > > > > > > > >> >> > > > > > > I have updated the PR. Please see and let me know. > > >> >> > > > > > > > > >> >> > > > > > > On Tue, Mar 23, 2021 at 4:27 PM Ilya Kasnacheev > > >> >> > > > > > > <[hidden email]> wrote: > > >> >> > > > > > > > > > >> >> > > > > > > > Hello! > > >> >> > > > > > > > > > >> >> > > > > > > > I am going to check this change out when I have time, > > >> using my > > >> >> > > Azure > > >> >> > > > > > > > account. > > >> >> > > > > > > > > > >> >> > > > > > > > Regards, > > >> >> > > > > > > > -- > > >> >> > > > > > > > Ilya Kasnacheev > > >> >> > > > > > > > > > >> >> > > > > > > > > > >> >> > > > > > > > вт, 23 мар. 2021 г. в 07:20, Atri Sharma < > > >> [hidden email]>: > > >> >> > > > > > > > > > >> >> > > > > > > > > Gentle reminder on this -- please help in reviewing > > >> this. > > >> >> > > > > > > > > > > >> >> > > > > > > > > On Fri, Mar 19, 2021 at 10:23 AM Atri Sharma < > > >> [hidden email]> > > >> >> > > > > wrote: > > >> >> > > > > > > > > > > > >> >> > > > > > > > > > Thanks Denis. > > >> >> > > > > > > > > > > > >> >> > > > > > > > > > I have raised a PR for the same: > > >> >> > > > > > > > > > > > >> >> > > > > > > > > > https://github.com/apache/ignite/pull/8897 > > >> >> > > > > > > > > > > > >> >> > > > > > > > > > Regards, > > >> >> > > > > > > > > > > > >> >> > > > > > > > > > Atri > > >> >> > > > > > > > > > > > >> >> > > > > > > > > > On Wed, Mar 10, 2021 at 1:21 AM Denis Magda < > > >> >> > > [hidden email]> > > >> >> > > > > > > wrote: > > >> >> > > > > > > > > > > > > >> >> > > > > > > > > > > Atri, > > >> >> > > > > > > > > > > > > >> >> > > > > > > > > > > Let's discuss the subj together with the > > >> community. Ignite > > >> >> > > > > already > > >> >> > > > > > > > > supports > > >> >> > > > > > > > > > > AWS [1] and GCE [2] IP Finders out of the box, but > > >> the > > >> >> > > Azure > > >> >> > > > > one is > > >> >> > > > > > > > > still > > >> >> > > > > > > > > > > missing. I can confirm that the demand exists, and > > >> rather > > >> >> > > > > > > frequently, > > >> >> > > > > > > > > I see > > >> >> > > > > > > > > > > developers asking for an Azure-native IP finder > > >> for Ignite. > > >> >> > > > > > > > > > > > > >> >> > > > > > > > > > > Atri, could you please research how to implement > > >> the IP > > >> >> > > finder > > >> >> > > > > and > > >> >> > > > > > > > > suggest > > >> >> > > > > > > > > > > a solution in this discussion thread? See how it > > >> was done > > >> >> > > for > > >> >> > > > > AWS > > >> >> > > > > > > and > > >> >> > > > > > > > > GCE, > > >> >> > > > > > > > > > > we might go the same route or use a more > > >> contemporary and > > >> >> > > > > > > > > easy-to-configure > > >> >> > > > > > > > > > > approach for Azure. > > >> >> > > > > > > > > > > > > >> >> > > > > > > > > > > [1] > > >> >> > > > > > > > > > > > > >> >> > > > > > > > > > > >> >> > > > > > > > > >> >> > > > > > > >> >> > > > > >> https://ignite.apache.org/docs/latest/clustering/discovery-in-the-cloud#amazon-s3-ip-finder > > >> >> > > > > > > > > > > [2] > > >> >> > > > > > > > > > > > > >> >> > > > > > > > > > > >> >> > > > > > > > > >> >> > > > > > > >> >> > > > > >> https://ignite.apache.org/docs/latest/clustering/discovery-in-the-cloud#google-compute-discovery > > >> >> > > > > > > > > > > > > >> >> > > > > > > > > > > - > > >> >> > > > > > > > > > > Denis > > >> >> > > > > > > > > > > > >> >> > > > > > > > > > > > >> >> > > > > > > > > > > > >> >> > > > > > > > > > -- > > >> >> > > > > > > > > > Regards, > > >> >> > > > > > > > > > > > >> >> > > > > > > > > > Atri > > >> >> > > > > > > > > > Apache Concerted > > >> >> > > > > > > > > > > >> >> > > > > > > > > > > >> >> > > > > > > > > > > >> >> > > > > > > > > -- > > >> >> > > > > > > > > Regards, > > >> >> > > > > > > > > > > >> >> > > > > > > > > Atri > > >> >> > > > > > > > > Apache Concerted > > >> >> > > > > > > > > > > >> >> > > > > > > > > >> >> > > > > > > -- > > >> >> > > > > > > Regards, > > >> >> > > > > > > > > >> >> > > > > > > Atri > > >> >> > > > > > > Apache Concerted > > >> >> > > > > > > > > >> >> > > > > > > >> >> > > > > -- > > >> >> > > > > Regards, > > >> >> > > > > > > >> >> > > > > Atri > > >> >> > > > > Apache Concerted > > >> >> > > > > > > >> >> > > > > >> >> > > -- > > >> >> > > Regards, > > >> >> > > > > >> >> > > Atri > > >> >> > > Apache Concerted > > >> >> > > > > >> >> > > >> >> -- > > >> >> Regards, > > >> >> > > >> >> Atri > > >> >> Apache Concerted > > >> > > >> -- > > >> Regards, > > >> > > >> Atri > > >> Apache Concerted > > >> > > > > > -- > Regards, > > Atri > Apache Concerted -- Regards, Atri Apache Concerted |
Hello!
I have responded to the ticket after testing on live Azure. Regards, -- Ilya Kasnacheev пт, 9 апр. 2021 г. в 08:37, Atri Sharma <[hidden email]>: > Hi Ilya, > > Please let me know if I can help with any further iterations on the PR. > > Regards, > > Atri > > On Wed, Apr 7, 2021 at 5:04 PM Atri Sharma <[hidden email]> wrote: > > > > Hi Ilya, > > > > Thanks for taking a look. I was able to resolve dependencies (Thanks, > > Sam!) and have updated the PR. > > > > Copying the jars from ignite-azure to libs works for me. > > > > Please see and let me know your thoughts. > > > > Regards, > > > > Atri > > > > On Mon, Apr 5, 2021 at 9:24 PM Ilya Kasnacheev > > <[hidden email]> wrote: > > > > > > Hello again! > > > > > > I re-checked our cloud discovery options by moving ignite-aws, > ignite-gce, > > > ignite-cloud directories from lib/optional to lib/ and trying to run > Ignite > > > with simple config taken from examples. > > > > > > The results are the following: > > > ignite-aws seems to work (complains about unknown key) > > > ignite-gce seems to work too (complains about zero length key, this is > > > before it tries any network access so maybe there are other issues > down the > > > path) > > > ignite-cloud (jclouds) DOES NOT work. Filed > > > https://issues.apache.org/jira/browse/IGNITE-14481 . I guess this > discovery > > > finder is not widely used. > > > ignite-azure, presently, will not work too. Please use ignite-aws as an > > > example since it sees the most usage. > > > > > > Regards, > > > -- > > > Ilya Kasnacheev > > > > > > > > > пн, 5 апр. 2021 г. в 16:05, Ilya Kasnacheev <[hidden email] > >: > > > > > > > Hello! > > > > > > > > I'm not sure that I can see any attachment to your e-mail. Can you > please > > > > re-send? > > > > > > > > We could have broken some of those, I guess, since we seem to not run > > > > integration tests for them. > > > > > > > > Regards, > > > > -- > > > > Ilya Kasnacheev > > > > > > > > > > > > пт, 2 апр. 2021 г. в 12:59, Atri Sharma <[hidden email]>: > > > > > > > >> Hello, > > > >> > > > >> Thank you for sharing. > > > >> > > > >> I was finally able to replicate the issue. However, I tried with > other > > > >> IPFinders and ran into the same problem (attached are the logs). > > > >> > > > >> Not sure what is causing this? > > > >> > > > >> Atri > > > >> > > > >> On Fri, Apr 2, 2021 at 2:29 PM Ilya Kasnacheev > > > >> <[hidden email]> wrote: > > > >> > > > > >> > Hello! > > > >> > > > > >> > Please find attached the log file of errors. This is yesterday's > (Apr > > > >> 1) build. > > > >> > > > > >> > Regards, > > > >> > -- > > > >> > Ilya Kasnacheev > > > >> > > > > >> > > > > >> > пт, 2 апр. 2021 г. в 11:52, Atri Sharma <[hidden email]>: > > > >> >> > > > >> >> I was able to, but then, it might be that something is cached > locally. > > > >> >> > > > >> >> What errors did you run into? (I am assuming you are trying with > a > > > >> >> build from the latest iteration of the PR). > > > >> >> > > > >> >> Atri > > > >> >> > > > >> >> On Fri, Apr 2, 2021 at 2:19 PM Ilya Kasnacheev > > > >> >> <[hidden email]> wrote: > > > >> >> > > > > >> >> > Hello! > > > >> >> > > > > >> >> > But are you successful in running a node with Azure IP finder > > > >> enabled in > > > >> >> > configuration, starting from that release directory? > > > >> >> > > > > >> >> > I amn't. > > > >> >> > > > > >> >> > Regards, > > > >> >> > -- > > > >> >> > Ilya Kasnacheev > > > >> >> > > > > >> >> > > > > >> >> > пт, 2 апр. 2021 г. в 07:42, Atri Sharma <[hidden email]>: > > > >> >> > > > > >> >> > > Hello, > > > >> >> > > > > > >> >> > > Thank you for taking a look. > > > >> >> > > > > > >> >> > > I am not sure what the confusion is. On the current > iteration of > > > >> PR, I > > > >> >> > > am able to build and release and looking in the location you > > > >> >> > > mentioned, I see: > > > >> >> > > > > > >> >> > > Atris-MacBook-Pro-15:libs atrisharma$ cd optional/ > > > >> >> > > > > > >> >> > > Atris-MacBook-Pro-15:optional atrisharma$ cd ignite-azure/ > > > >> >> > > > > > >> >> > > Atris-MacBook-Pro-15:ignite-azure atrisharma$ ls > > > >> >> > > > > > >> >> > > README.txt azure-storage-blob-12.0.0.jar > > > >> ignite-azure-2.11.0-SNAPSHOT.jar > > > >> >> > > > > > >> >> > > > > > >> >> > > Attached is the screenshot. > > > >> >> > > > > > >> >> > > I have fixed the comments on the PR, please see and let me > know. > > > >> >> > > > > > >> >> > > Atri > > > >> >> > > > > > >> >> > > On Thu, Apr 1, 2021 at 8:21 PM Ilya Kasnacheev > > > >> >> > > <[hidden email]> wrote: > > > >> >> > > > > > > >> >> > > > Hello! > > > >> >> > > > > > > >> >> > > > Were you successful with using it from the deliverable of > mvn > > > >> initialize > > > >> >> > > > -Prelease? > > > >> >> > > > > > > >> >> > > > Please refer for example to aws module dependencies > section: > > > >> >> > > > > > > >> >> > > > <dependency> > > > >> >> > > > <groupId>com.amazonaws</groupId> > > > >> >> > > > <artifactId>aws-java-sdk-core</artifactId> > > > >> >> > > > <version>${aws.sdk.version}</version> > > > >> >> > > > </dependency> > > > >> >> > > > > > > >> >> > > > <dependency> > > > >> >> > > > <groupId>com.amazonaws</groupId> > > > >> >> > > > <artifactId>aws-java-sdk-s3</artifactId> > > > >> >> > > > <version>${aws.sdk.version}</version> > > > >> >> > > > </dependency> > > > >> >> > > > > > > >> >> > > > <dependency> > > > >> >> > > > <groupId>com.amazonaws</groupId> > > > >> >> > > > <artifactId>aws-java-sdk-ec2</artifactId> > > > >> >> > > > <version>${aws.sdk.version}</version> > > > >> >> > > > </dependency> > > > >> >> > > > > > > >> >> > > > <dependency> > > > >> >> > > > <groupId>com.amazonaws</groupId> > > > >> >> > > > > <artifactId>aws-java-sdk-elasticloadbalancing</artifactId> > > > >> >> > > > <version>${aws.sdk.version}</version> > > > >> >> > > > </dependency> > > > >> >> > > > > > > >> >> > > > <dependency> > > > >> >> > > > <groupId>com.amazonaws</groupId> > > > >> >> > > > > <artifactId>aws-java-sdk-elasticloadbalancingv2</artifactId> > > > >> >> > > > <version>${aws.sdk.version}</version> > > > >> >> > > > </dependency> > > > >> >> > > > > > > >> >> > > > <dependency> > > > >> >> > > > <groupId>com.amazonaws</groupId> > > > >> >> > > > <artifactId>aws-java-sdk-kms</artifactId> > > > >> >> > > > <version>${aws.sdk.version}</version> > > > >> >> > > > </dependency> > > > >> >> > > > > > > >> >> > > > <!-- Do not remove > com.fasterxml.jackson.core:jackson-core it is > > > >> >> > > > required at runtime --> > > > >> >> > > > <dependency> > > > >> >> > > > <groupId>com.fasterxml.jackson.core</groupId> > > > >> >> > > > <artifactId>jackson-core</artifactId> > > > >> >> > > > <version>${jackson.version}</version> > > > >> >> > > > </dependency> > > > >> >> > > > > > > >> >> > > > <!-- Do not remove > > > >> com.fasterxml.jackson.core:jackson-annotations it > > > >> >> > > > is required at runtime --> > > > >> >> > > > <dependency> > > > >> >> > > > <groupId>com.fasterxml.jackson.core</groupId> > > > >> >> > > > <artifactId>jackson-annotations</artifactId> > > > >> >> > > > <version>${jackson.version}</version> > > > >> >> > > > </dependency> > > > >> >> > > > > > > >> >> > > > <!-- Do not remove > com.fasterxml.jackson.core:jackson-databind > > > >> it is > > > >> >> > > > required at runtime --> > > > >> >> > > > <dependency> > > > >> >> > > > <groupId>com.fasterxml.jackson.core</groupId> > > > >> >> > > > <artifactId>jackson-databind</artifactId> > > > >> >> > > > <version>${jackson.version}</version> > > > >> >> > > > </dependency> > > > >> >> > > > > > > >> >> > > > <dependency> > > > >> >> > > > <groupId>com.amazonaws</groupId> > > > >> >> > > > <artifactId>aws-encryption-sdk-java</artifactId> > > > >> >> > > > <version>${aws.encryption.sdk.version}</version> > > > >> >> > > > <exclusions> > > > >> >> > > > <exclusion> > > > >> >> > > > <groupId>org.bouncycastle</groupId> > > > >> >> > > > <artifactId>bcprov-ext-jdk15on</artifactId> > > > >> >> > > > </exclusion> > > > >> >> > > > </exclusions> > > > >> >> > > > </dependency> > > > >> >> > > > > > > >> >> > > > <dependency> > > > >> >> > > > <groupId>org.bouncycastle</groupId> > > > >> >> > > > <artifactId>bcprov-ext-jdk15on</artifactId> > > > >> >> > > > <version>${bouncycastle.version}</version> > > > >> >> > > > </dependency> > > > >> >> > > > > > > >> >> > > > <dependency> > > > >> >> > > > <groupId>joda-time</groupId> > > > >> >> > > > <artifactId>joda-time</artifactId> > > > >> >> > > > <version>2.8.1</version> > > > >> >> > > > </dependency> > > > >> >> > > > > > > >> >> > > > <dependency> > > > >> >> > > > <groupId>org.apache.httpcomponents</groupId> > > > >> >> > > > <artifactId>httpclient</artifactId> > > > >> >> > > > <version>${httpclient.version}</version> > > > >> >> > > > </dependency> > > > >> >> > > > > > > >> >> > > > <dependency> > > > >> >> > > > <groupId>org.apache.httpcomponents</groupId> > > > >> >> > > > <artifactId>httpcore</artifactId> > > > >> >> > > > <version>${httpcore.version}</version> > > > >> >> > > > </dependency> > > > >> >> > > > > > > >> >> > > > azure module should likewise list all of its actual > > > >> dependencies. Right > > > >> >> > > now > > > >> >> > > > the module directory is empty so I'm not even going to > test to > > > >> run it in > > > >> >> > > > stand-alone mode: > > > >> >> > > > ~/Downloads/apache-ignite-2.11.0-SNAPSHOT-bin% ls > > > >> >> > > libs/optional/ignite-azure > > > >> >> > > > azure-storage-blob-12.0.0.jar > ignite-azure-2.11.0-SNAPSHOT.jar > > > >> >> > > README.txt > > > >> >> > > > > > > >> >> > > > Please address this issue along with other things which I > have > > > >> also > > > >> >> > > > commented on the PR. > > > >> >> > > > > > > >> >> > > > Regards, > > > >> >> > > > -- > > > >> >> > > > Ilya Kasnacheev > > > >> >> > > > > > > >> >> > > > > > > >> >> > > > ср, 31 мар. 2021 г. в 08:17, Atri Sharma <[hidden email] > >: > > > >> >> > > > > > > >> >> > > > > Thank you for the review. > > > >> >> > > > > > > > >> >> > > > > I have updated the PR. Please see. > > > >> >> > > > > > > > >> >> > > > > On Mon, Mar 29, 2021 at 8:27 PM Ilya Kasnacheev > > > >> >> > > > > <[hidden email]> wrote: > > > >> >> > > > > > > > > >> >> > > > > > Hello! > > > >> >> > > > > > > > > >> >> > > > > > I have left some more comments after trying this > change on a > > > >> real > > > >> >> > > Azure > > > >> >> > > > > > cluster. > > > >> >> > > > > > > > > >> >> > > > > > Regards, > > > >> >> > > > > > -- > > > >> >> > > > > > Ilya Kasnacheev > > > >> >> > > > > > > > > >> >> > > > > > > > > >> >> > > > > > вт, 23 мар. 2021 г. в 18:32, Atri Sharma < > [hidden email]>: > > > >> >> > > > > > > > > >> >> > > > > > > Thank you! > > > >> >> > > > > > > > > > >> >> > > > > > > I have updated the PR. Please see and let me know. > > > >> >> > > > > > > > > > >> >> > > > > > > On Tue, Mar 23, 2021 at 4:27 PM Ilya Kasnacheev > > > >> >> > > > > > > <[hidden email]> wrote: > > > >> >> > > > > > > > > > > >> >> > > > > > > > Hello! > > > >> >> > > > > > > > > > > >> >> > > > > > > > I am going to check this change out when I have > time, > > > >> using my > > > >> >> > > Azure > > > >> >> > > > > > > > account. > > > >> >> > > > > > > > > > > >> >> > > > > > > > Regards, > > > >> >> > > > > > > > -- > > > >> >> > > > > > > > Ilya Kasnacheev > > > >> >> > > > > > > > > > > >> >> > > > > > > > > > > >> >> > > > > > > > вт, 23 мар. 2021 г. в 07:20, Atri Sharma < > > > >> [hidden email]>: > > > >> >> > > > > > > > > > > >> >> > > > > > > > > Gentle reminder on this -- please help in > reviewing > > > >> this. > > > >> >> > > > > > > > > > > > >> >> > > > > > > > > On Fri, Mar 19, 2021 at 10:23 AM Atri Sharma < > > > >> [hidden email]> > > > >> >> > > > > wrote: > > > >> >> > > > > > > > > > > > > >> >> > > > > > > > > > Thanks Denis. > > > >> >> > > > > > > > > > > > > >> >> > > > > > > > > > I have raised a PR for the same: > > > >> >> > > > > > > > > > > > > >> >> > > > > > > > > > https://github.com/apache/ignite/pull/8897 > > > >> >> > > > > > > > > > > > > >> >> > > > > > > > > > Regards, > > > >> >> > > > > > > > > > > > > >> >> > > > > > > > > > Atri > > > >> >> > > > > > > > > > > > > >> >> > > > > > > > > > On Wed, Mar 10, 2021 at 1:21 AM Denis Magda < > > > >> >> > > [hidden email]> > > > >> >> > > > > > > wrote: > > > >> >> > > > > > > > > > > > > > >> >> > > > > > > > > > > Atri, > > > >> >> > > > > > > > > > > > > > >> >> > > > > > > > > > > Let's discuss the subj together with the > > > >> community. Ignite > > > >> >> > > > > already > > > >> >> > > > > > > > > supports > > > >> >> > > > > > > > > > > AWS [1] and GCE [2] IP Finders out of the > box, but > > > >> the > > > >> >> > > Azure > > > >> >> > > > > one is > > > >> >> > > > > > > > > still > > > >> >> > > > > > > > > > > missing. I can confirm that the demand > exists, and > > > >> rather > > > >> >> > > > > > > frequently, > > > >> >> > > > > > > > > I see > > > >> >> > > > > > > > > > > developers asking for an Azure-native IP > finder > > > >> for Ignite. > > > >> >> > > > > > > > > > > > > > >> >> > > > > > > > > > > Atri, could you please research how to > implement > > > >> the IP > > > >> >> > > finder > > > >> >> > > > > and > > > >> >> > > > > > > > > suggest > > > >> >> > > > > > > > > > > a solution in this discussion thread? See > how it > > > >> was done > > > >> >> > > for > > > >> >> > > > > AWS > > > >> >> > > > > > > and > > > >> >> > > > > > > > > GCE, > > > >> >> > > > > > > > > > > we might go the same route or use a more > > > >> contemporary and > > > >> >> > > > > > > > > easy-to-configure > > > >> >> > > > > > > > > > > approach for Azure. > > > >> >> > > > > > > > > > > > > > >> >> > > > > > > > > > > [1] > > > >> >> > > > > > > > > > > > > > >> >> > > > > > > > > > > > >> >> > > > > > > > > > >> >> > > > > > > > >> >> > > > > > >> > https://ignite.apache.org/docs/latest/clustering/discovery-in-the-cloud#amazon-s3-ip-finder > > > >> >> > > > > > > > > > > [2] > > > >> >> > > > > > > > > > > > > > >> >> > > > > > > > > > > > >> >> > > > > > > > > > >> >> > > > > > > > >> >> > > > > > >> > https://ignite.apache.org/docs/latest/clustering/discovery-in-the-cloud#google-compute-discovery > > > >> >> > > > > > > > > > > > > > >> >> > > > > > > > > > > - > > > >> >> > > > > > > > > > > Denis > > > >> >> > > > > > > > > > > > > >> >> > > > > > > > > > > > > >> >> > > > > > > > > > > > > >> >> > > > > > > > > > -- > > > >> >> > > > > > > > > > Regards, > > > >> >> > > > > > > > > > > > > >> >> > > > > > > > > > Atri > > > >> >> > > > > > > > > > Apache Concerted > > > >> >> > > > > > > > > > > > >> >> > > > > > > > > > > > >> >> > > > > > > > > > > > >> >> > > > > > > > > -- > > > >> >> > > > > > > > > Regards, > > > >> >> > > > > > > > > > > > >> >> > > > > > > > > Atri > > > >> >> > > > > > > > > Apache Concerted > > > >> >> > > > > > > > > > > > >> >> > > > > > > > > > >> >> > > > > > > -- > > > >> >> > > > > > > Regards, > > > >> >> > > > > > > > > > >> >> > > > > > > Atri > > > >> >> > > > > > > Apache Concerted > > > >> >> > > > > > > > > > >> >> > > > > > > > >> >> > > > > -- > > > >> >> > > > > Regards, > > > >> >> > > > > > > > >> >> > > > > Atri > > > >> >> > > > > Apache Concerted > > > >> >> > > > > > > > >> >> > > > > > >> >> > > -- > > > >> >> > > Regards, > > > >> >> > > > > > >> >> > > Atri > > > >> >> > > Apache Concerted > > > >> >> > > > > > >> >> > > > >> >> -- > > > >> >> Regards, > > > >> >> > > > >> >> Atri > > > >> >> Apache Concerted > > > >> > > > >> -- > > > >> Regards, > > > >> > > > >> Atri > > > >> Apache Concerted > > > >> > > > > > > > > -- > > Regards, > > > > Atri > > Apache Concerted > > > > -- > Regards, > > Atri > Apache Concerted > |
Hello,
Sorry for the late reply. Thank you for taking a look. Indeed, there were some issues and they have now been fixed. I am able to start a two node cluster with Azure IPFinder enabled and shut it down successfully (using Ipv4). Please see the latest iteration and let me know your thoughts and comments. Regards, Atri On Fri, Apr 9, 2021 at 4:58 PM Ilya Kasnacheev <[hidden email]> wrote: > > Hello! > > I have responded to the ticket after testing on live Azure. > > Regards, > -- > Ilya Kasnacheev > > > пт, 9 апр. 2021 г. в 08:37, Atri Sharma <[hidden email]>: > > > Hi Ilya, > > > > Please let me know if I can help with any further iterations on the PR. > > > > Regards, > > > > Atri > > > > On Wed, Apr 7, 2021 at 5:04 PM Atri Sharma <[hidden email]> wrote: > > > > > > Hi Ilya, > > > > > > Thanks for taking a look. I was able to resolve dependencies (Thanks, > > > Sam!) and have updated the PR. > > > > > > Copying the jars from ignite-azure to libs works for me. > > > > > > Please see and let me know your thoughts. > > > > > > Regards, > > > > > > Atri > > > > > > On Mon, Apr 5, 2021 at 9:24 PM Ilya Kasnacheev > > > <[hidden email]> wrote: > > > > > > > > Hello again! > > > > > > > > I re-checked our cloud discovery options by moving ignite-aws, > > ignite-gce, > > > > ignite-cloud directories from lib/optional to lib/ and trying to run > > Ignite > > > > with simple config taken from examples. > > > > > > > > The results are the following: > > > > ignite-aws seems to work (complains about unknown key) > > > > ignite-gce seems to work too (complains about zero length key, this is > > > > before it tries any network access so maybe there are other issues > > down the > > > > path) > > > > ignite-cloud (jclouds) DOES NOT work. Filed > > > > https://issues.apache.org/jira/browse/IGNITE-14481 . I guess this > > discovery > > > > finder is not widely used. > > > > ignite-azure, presently, will not work too. Please use ignite-aws as an > > > > example since it sees the most usage. > > > > > > > > Regards, > > > > -- > > > > Ilya Kasnacheev > > > > > > > > > > > > пн, 5 апр. 2021 г. в 16:05, Ilya Kasnacheev <[hidden email] > > >: > > > > > > > > > Hello! > > > > > > > > > > I'm not sure that I can see any attachment to your e-mail. Can you > > please > > > > > re-send? > > > > > > > > > > We could have broken some of those, I guess, since we seem to not run > > > > > integration tests for them. > > > > > > > > > > Regards, > > > > > -- > > > > > Ilya Kasnacheev > > > > > > > > > > > > > > > пт, 2 апр. 2021 г. в 12:59, Atri Sharma <[hidden email]>: > > > > > > > > > >> Hello, > > > > >> > > > > >> Thank you for sharing. > > > > >> > > > > >> I was finally able to replicate the issue. However, I tried with > > other > > > > >> IPFinders and ran into the same problem (attached are the logs). > > > > >> > > > > >> Not sure what is causing this? > > > > >> > > > > >> Atri > > > > >> > > > > >> On Fri, Apr 2, 2021 at 2:29 PM Ilya Kasnacheev > > > > >> <[hidden email]> wrote: > > > > >> > > > > > >> > Hello! > > > > >> > > > > > >> > Please find attached the log file of errors. This is yesterday's > > (Apr > > > > >> 1) build. > > > > >> > > > > > >> > Regards, > > > > >> > -- > > > > >> > Ilya Kasnacheev > > > > >> > > > > > >> > > > > > >> > пт, 2 апр. 2021 г. в 11:52, Atri Sharma <[hidden email]>: > > > > >> >> > > > > >> >> I was able to, but then, it might be that something is cached > > locally. > > > > >> >> > > > > >> >> What errors did you run into? (I am assuming you are trying with > > a > > > > >> >> build from the latest iteration of the PR). > > > > >> >> > > > > >> >> Atri > > > > >> >> > > > > >> >> On Fri, Apr 2, 2021 at 2:19 PM Ilya Kasnacheev > > > > >> >> <[hidden email]> wrote: > > > > >> >> > > > > > >> >> > Hello! > > > > >> >> > > > > > >> >> > But are you successful in running a node with Azure IP finder > > > > >> enabled in > > > > >> >> > configuration, starting from that release directory? > > > > >> >> > > > > > >> >> > I amn't. > > > > >> >> > > > > > >> >> > Regards, > > > > >> >> > -- > > > > >> >> > Ilya Kasnacheev > > > > >> >> > > > > > >> >> > > > > > >> >> > пт, 2 апр. 2021 г. в 07:42, Atri Sharma <[hidden email]>: > > > > >> >> > > > > > >> >> > > Hello, > > > > >> >> > > > > > > >> >> > > Thank you for taking a look. > > > > >> >> > > > > > > >> >> > > I am not sure what the confusion is. On the current > > iteration of > > > > >> PR, I > > > > >> >> > > am able to build and release and looking in the location you > > > > >> >> > > mentioned, I see: > > > > >> >> > > > > > > >> >> > > Atris-MacBook-Pro-15:libs atrisharma$ cd optional/ > > > > >> >> > > > > > > >> >> > > Atris-MacBook-Pro-15:optional atrisharma$ cd ignite-azure/ > > > > >> >> > > > > > > >> >> > > Atris-MacBook-Pro-15:ignite-azure atrisharma$ ls > > > > >> >> > > > > > > >> >> > > README.txt azure-storage-blob-12.0.0.jar > > > > >> ignite-azure-2.11.0-SNAPSHOT.jar > > > > >> >> > > > > > > >> >> > > > > > > >> >> > > Attached is the screenshot. > > > > >> >> > > > > > > >> >> > > I have fixed the comments on the PR, please see and let me > > know. > > > > >> >> > > > > > > >> >> > > Atri > > > > >> >> > > > > > > >> >> > > On Thu, Apr 1, 2021 at 8:21 PM Ilya Kasnacheev > > > > >> >> > > <[hidden email]> wrote: > > > > >> >> > > > > > > > >> >> > > > Hello! > > > > >> >> > > > > > > > >> >> > > > Were you successful with using it from the deliverable of > > mvn > > > > >> initialize > > > > >> >> > > > -Prelease? > > > > >> >> > > > > > > > >> >> > > > Please refer for example to aws module dependencies > > section: > > > > >> >> > > > > > > > >> >> > > > <dependency> > > > > >> >> > > > <groupId>com.amazonaws</groupId> > > > > >> >> > > > <artifactId>aws-java-sdk-core</artifactId> > > > > >> >> > > > <version>${aws.sdk.version}</version> > > > > >> >> > > > </dependency> > > > > >> >> > > > > > > > >> >> > > > <dependency> > > > > >> >> > > > <groupId>com.amazonaws</groupId> > > > > >> >> > > > <artifactId>aws-java-sdk-s3</artifactId> > > > > >> >> > > > <version>${aws.sdk.version}</version> > > > > >> >> > > > </dependency> > > > > >> >> > > > > > > > >> >> > > > <dependency> > > > > >> >> > > > <groupId>com.amazonaws</groupId> > > > > >> >> > > > <artifactId>aws-java-sdk-ec2</artifactId> > > > > >> >> > > > <version>${aws.sdk.version}</version> > > > > >> >> > > > </dependency> > > > > >> >> > > > > > > > >> >> > > > <dependency> > > > > >> >> > > > <groupId>com.amazonaws</groupId> > > > > >> >> > > > > > <artifactId>aws-java-sdk-elasticloadbalancing</artifactId> > > > > >> >> > > > <version>${aws.sdk.version}</version> > > > > >> >> > > > </dependency> > > > > >> >> > > > > > > > >> >> > > > <dependency> > > > > >> >> > > > <groupId>com.amazonaws</groupId> > > > > >> >> > > > > > <artifactId>aws-java-sdk-elasticloadbalancingv2</artifactId> > > > > >> >> > > > <version>${aws.sdk.version}</version> > > > > >> >> > > > </dependency> > > > > >> >> > > > > > > > >> >> > > > <dependency> > > > > >> >> > > > <groupId>com.amazonaws</groupId> > > > > >> >> > > > <artifactId>aws-java-sdk-kms</artifactId> > > > > >> >> > > > <version>${aws.sdk.version}</version> > > > > >> >> > > > </dependency> > > > > >> >> > > > > > > > >> >> > > > <!-- Do not remove > > com.fasterxml.jackson.core:jackson-core it is > > > > >> >> > > > required at runtime --> > > > > >> >> > > > <dependency> > > > > >> >> > > > <groupId>com.fasterxml.jackson.core</groupId> > > > > >> >> > > > <artifactId>jackson-core</artifactId> > > > > >> >> > > > <version>${jackson.version}</version> > > > > >> >> > > > </dependency> > > > > >> >> > > > > > > > >> >> > > > <!-- Do not remove > > > > >> com.fasterxml.jackson.core:jackson-annotations it > > > > >> >> > > > is required at runtime --> > > > > >> >> > > > <dependency> > > > > >> >> > > > <groupId>com.fasterxml.jackson.core</groupId> > > > > >> >> > > > <artifactId>jackson-annotations</artifactId> > > > > >> >> > > > <version>${jackson.version}</version> > > > > >> >> > > > </dependency> > > > > >> >> > > > > > > > >> >> > > > <!-- Do not remove > > com.fasterxml.jackson.core:jackson-databind > > > > >> it is > > > > >> >> > > > required at runtime --> > > > > >> >> > > > <dependency> > > > > >> >> > > > <groupId>com.fasterxml.jackson.core</groupId> > > > > >> >> > > > <artifactId>jackson-databind</artifactId> > > > > >> >> > > > <version>${jackson.version}</version> > > > > >> >> > > > </dependency> > > > > >> >> > > > > > > > >> >> > > > <dependency> > > > > >> >> > > > <groupId>com.amazonaws</groupId> > > > > >> >> > > > <artifactId>aws-encryption-sdk-java</artifactId> > > > > >> >> > > > <version>${aws.encryption.sdk.version}</version> > > > > >> >> > > > <exclusions> > > > > >> >> > > > <exclusion> > > > > >> >> > > > <groupId>org.bouncycastle</groupId> > > > > >> >> > > > <artifactId>bcprov-ext-jdk15on</artifactId> > > > > >> >> > > > </exclusion> > > > > >> >> > > > </exclusions> > > > > >> >> > > > </dependency> > > > > >> >> > > > > > > > >> >> > > > <dependency> > > > > >> >> > > > <groupId>org.bouncycastle</groupId> > > > > >> >> > > > <artifactId>bcprov-ext-jdk15on</artifactId> > > > > >> >> > > > <version>${bouncycastle.version}</version> > > > > >> >> > > > </dependency> > > > > >> >> > > > > > > > >> >> > > > <dependency> > > > > >> >> > > > <groupId>joda-time</groupId> > > > > >> >> > > > <artifactId>joda-time</artifactId> > > > > >> >> > > > <version>2.8.1</version> > > > > >> >> > > > </dependency> > > > > >> >> > > > > > > > >> >> > > > <dependency> > > > > >> >> > > > <groupId>org.apache.httpcomponents</groupId> > > > > >> >> > > > <artifactId>httpclient</artifactId> > > > > >> >> > > > <version>${httpclient.version}</version> > > > > >> >> > > > </dependency> > > > > >> >> > > > > > > > >> >> > > > <dependency> > > > > >> >> > > > <groupId>org.apache.httpcomponents</groupId> > > > > >> >> > > > <artifactId>httpcore</artifactId> > > > > >> >> > > > <version>${httpcore.version}</version> > > > > >> >> > > > </dependency> > > > > >> >> > > > > > > > >> >> > > > azure module should likewise list all of its actual > > > > >> dependencies. Right > > > > >> >> > > now > > > > >> >> > > > the module directory is empty so I'm not even going to > > test to > > > > >> run it in > > > > >> >> > > > stand-alone mode: > > > > >> >> > > > ~/Downloads/apache-ignite-2.11.0-SNAPSHOT-bin% ls > > > > >> >> > > libs/optional/ignite-azure > > > > >> >> > > > azure-storage-blob-12.0.0.jar > > ignite-azure-2.11.0-SNAPSHOT.jar > > > > >> >> > > README.txt > > > > >> >> > > > > > > > >> >> > > > Please address this issue along with other things which I > > have > > > > >> also > > > > >> >> > > > commented on the PR. > > > > >> >> > > > > > > > >> >> > > > Regards, > > > > >> >> > > > -- > > > > >> >> > > > Ilya Kasnacheev > > > > >> >> > > > > > > > >> >> > > > > > > > >> >> > > > ср, 31 мар. 2021 г. в 08:17, Atri Sharma <[hidden email] > > >: > > > > >> >> > > > > > > > >> >> > > > > Thank you for the review. > > > > >> >> > > > > > > > > >> >> > > > > I have updated the PR. Please see. > > > > >> >> > > > > > > > > >> >> > > > > On Mon, Mar 29, 2021 at 8:27 PM Ilya Kasnacheev > > > > >> >> > > > > <[hidden email]> wrote: > > > > >> >> > > > > > > > > > >> >> > > > > > Hello! > > > > >> >> > > > > > > > > > >> >> > > > > > I have left some more comments after trying this > > change on a > > > > >> real > > > > >> >> > > Azure > > > > >> >> > > > > > cluster. > > > > >> >> > > > > > > > > > >> >> > > > > > Regards, > > > > >> >> > > > > > -- > > > > >> >> > > > > > Ilya Kasnacheev > > > > >> >> > > > > > > > > > >> >> > > > > > > > > > >> >> > > > > > вт, 23 мар. 2021 г. в 18:32, Atri Sharma < > > [hidden email]>: > > > > >> >> > > > > > > > > > >> >> > > > > > > Thank you! > > > > >> >> > > > > > > > > > > >> >> > > > > > > I have updated the PR. Please see and let me know. > > > > >> >> > > > > > > > > > > >> >> > > > > > > On Tue, Mar 23, 2021 at 4:27 PM Ilya Kasnacheev > > > > >> >> > > > > > > <[hidden email]> wrote: > > > > >> >> > > > > > > > > > > > >> >> > > > > > > > Hello! > > > > >> >> > > > > > > > > > > > >> >> > > > > > > > I am going to check this change out when I have > > time, > > > > >> using my > > > > >> >> > > Azure > > > > >> >> > > > > > > > account. > > > > >> >> > > > > > > > > > > > >> >> > > > > > > > Regards, > > > > >> >> > > > > > > > -- > > > > >> >> > > > > > > > Ilya Kasnacheev > > > > >> >> > > > > > > > > > > > >> >> > > > > > > > > > > > >> >> > > > > > > > вт, 23 мар. 2021 г. в 07:20, Atri Sharma < > > > > >> [hidden email]>: > > > > >> >> > > > > > > > > > > > >> >> > > > > > > > > Gentle reminder on this -- please help in > > reviewing > > > > >> this. > > > > >> >> > > > > > > > > > > > > >> >> > > > > > > > > On Fri, Mar 19, 2021 at 10:23 AM Atri Sharma < > > > > >> [hidden email]> > > > > >> >> > > > > wrote: > > > > >> >> > > > > > > > > > > > > > >> >> > > > > > > > > > Thanks Denis. > > > > >> >> > > > > > > > > > > > > > >> >> > > > > > > > > > I have raised a PR for the same: > > > > >> >> > > > > > > > > > > > > > >> >> > > > > > > > > > https://github.com/apache/ignite/pull/8897 > > > > >> >> > > > > > > > > > > > > > >> >> > > > > > > > > > Regards, > > > > >> >> > > > > > > > > > > > > > >> >> > > > > > > > > > Atri > > > > >> >> > > > > > > > > > > > > > >> >> > > > > > > > > > On Wed, Mar 10, 2021 at 1:21 AM Denis Magda < > > > > >> >> > > [hidden email]> > > > > >> >> > > > > > > wrote: > > > > >> >> > > > > > > > > > > > > > > >> >> > > > > > > > > > > Atri, > > > > >> >> > > > > > > > > > > > > > > >> >> > > > > > > > > > > Let's discuss the subj together with the > > > > >> community. Ignite > > > > >> >> > > > > already > > > > >> >> > > > > > > > > supports > > > > >> >> > > > > > > > > > > AWS [1] and GCE [2] IP Finders out of the > > box, but > > > > >> the > > > > >> >> > > Azure > > > > >> >> > > > > one is > > > > >> >> > > > > > > > > still > > > > >> >> > > > > > > > > > > missing. I can confirm that the demand > > exists, and > > > > >> rather > > > > >> >> > > > > > > frequently, > > > > >> >> > > > > > > > > I see > > > > >> >> > > > > > > > > > > developers asking for an Azure-native IP > > finder > > > > >> for Ignite. > > > > >> >> > > > > > > > > > > > > > > >> >> > > > > > > > > > > Atri, could you please research how to > > implement > > > > >> the IP > > > > >> >> > > finder > > > > >> >> > > > > and > > > > >> >> > > > > > > > > suggest > > > > >> >> > > > > > > > > > > a solution in this discussion thread? See > > how it > > > > >> was done > > > > >> >> > > for > > > > >> >> > > > > AWS > > > > >> >> > > > > > > and > > > > >> >> > > > > > > > > GCE, > > > > >> >> > > > > > > > > > > we might go the same route or use a more > > > > >> contemporary and > > > > >> >> > > > > > > > > easy-to-configure > > > > >> >> > > > > > > > > > > approach for Azure. > > > > >> >> > > > > > > > > > > > > > > >> >> > > > > > > > > > > [1] > > > > >> >> > > > > > > > > > > > > > > >> >> > > > > > > > > > > > > >> >> > > > > > > > > > > >> >> > > > > > > > > >> >> > > > > > > >> > > https://ignite.apache.org/docs/latest/clustering/discovery-in-the-cloud#amazon-s3-ip-finder > > > > >> >> > > > > > > > > > > [2] > > > > >> >> > > > > > > > > > > > > > > >> >> > > > > > > > > > > > > >> >> > > > > > > > > > > >> >> > > > > > > > > >> >> > > > > > > >> > > https://ignite.apache.org/docs/latest/clustering/discovery-in-the-cloud#google-compute-discovery > > > > >> >> > > > > > > > > > > > > > > >> >> > > > > > > > > > > - > > > > >> >> > > > > > > > > > > Denis > > > > >> >> > > > > > > > > > > > > > >> >> > > > > > > > > > > > > > >> >> > > > > > > > > > > > > > >> >> > > > > > > > > > -- > > > > >> >> > > > > > > > > > Regards, > > > > >> >> > > > > > > > > > > > > > >> >> > > > > > > > > > Atri > > > > >> >> > > > > > > > > > Apache Concerted > > > > >> >> > > > > > > > > > > > > >> >> > > > > > > > > > > > > >> >> > > > > > > > > > > > > >> >> > > > > > > > > -- > > > > >> >> > > > > > > > > Regards, > > > > >> >> > > > > > > > > > > > > >> >> > > > > > > > > Atri > > > > >> >> > > > > > > > > Apache Concerted > > > > >> >> > > > > > > > > > > > > >> >> > > > > > > > > > > >> >> > > > > > > -- > > > > >> >> > > > > > > Regards, > > > > >> >> > > > > > > > > > > >> >> > > > > > > Atri > > > > >> >> > > > > > > Apache Concerted > > > > >> >> > > > > > > > > > > >> >> > > > > > > > > >> >> > > > > -- > > > > >> >> > > > > Regards, > > > > >> >> > > > > > > > > >> >> > > > > Atri > > > > >> >> > > > > Apache Concerted > > > > >> >> > > > > > > > > >> >> > > > > > > >> >> > > -- > > > > >> >> > > Regards, > > > > >> >> > > > > > > >> >> > > Atri > > > > >> >> > > Apache Concerted > > > > >> >> > > > > > > >> >> > > > > >> >> -- > > > > >> >> Regards, > > > > >> >> > > > > >> >> Atri > > > > >> >> Apache Concerted > > > > >> > > > > >> -- > > > > >> Regards, > > > > >> > > > > >> Atri > > > > >> Apache Concerted > > > > >> > > > > > > > > > > > -- > > > Regards, > > > > > > Atri > > > Apache Concerted > > > > > > > > -- > > Regards, > > > > Atri > > Apache Concerted > > -- Regards, Atri Apache Concerted |
Free forum by Nabble | Edit this page |