Skip to content
This repository has been archived by the owner on Jan 19, 2022. It is now read-only.

Add ability to generate ingress resources. #14

Merged
merged 1 commit into from
Apr 17, 2020

Conversation

libby
Copy link
Contributor

@libby libby commented Apr 17, 2020

Add the ability to generate ingress resources, either
OneToOne where a k8s ingress resource is created for each
node (1:1), or OneToMany (1:N) where one ingress resource is created
for all quorum nodes deployed in the cluster, and all nodes share a
common host name.

Currently only support one host FQDN per ingress.

Only exposing the geth rpc endpoint.

Not supporting tls at this time, however tls can be added with
self signed certificate (which would requires modification of the
geth client) or using a valid root CA.

Add the ability to generate ingress resources, either
OneToOne where a k8s ingress resource is created for each
node (1:1), or OneToMany (1:N) where one ingress resource is created
for all quorum  nodes deployed in the cluster, and all nodes share a
common host name.

Currently only support one host FQDN per ingress.

Only exposing the geth rpc endpoint.

Not supporting tls at this time, however tls can be added with
self signed certificate (which would requires modification of the
geth client) or using a valid root CA.
@libby libby requested a review from jbhurat April 17, 2020 18:44
@libby
Copy link
Contributor Author

libby commented Apr 17, 2020

To test run:

$> docker run --rm -it -v $(pwd):/qubernetes -ti quorumengineering/qubernetes ./quorum-init examples/config/qubernetes-ingress.yaml

The examples/config/qubernetes-ingress.yaml file can be modified to test the different strategies:

 Ingress:
    # OneToMany | OneToOne
    Strategy: OneToMany
    Host: "quorum.testnet.com
 Ingress:
    # OneToMany | OneToOne
    Strategy: OneToOne
    Host: "quorum.testnet.com

Copy link
Contributor

@jbhurat jbhurat left a comment

Choose a reason for hiding this comment

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

LGTM

@libby libby merged commit cef513a into Consensys:master Apr 17, 2020
@libby libby deleted the add-ingress branch August 11, 2020 19:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants