Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[helm-chart] sidecar container taraxa-indexer for rpc nodes #2344

Merged
merged 27 commits into from
Feb 28, 2023
Merged

[helm-chart] sidecar container taraxa-indexer for rpc nodes #2344

merged 27 commits into from
Feb 28, 2023

Conversation

rjonczy
Copy link
Contributor

@rjonczy rjonczy commented Feb 24, 2023 •
edited
Loading

Purpose

Expose data from taraxa-indexer

How does the solution address the problem

Added taraxa-indexer to RPC nodes and exposed via Ingress

Changes made

Added:

  • taraxa-indexer as a sidecar container to RPC nodes
  • new Ingress exposing Indexer's endpoints with sticky session based on cookie

@rjonczy
Copy link
Contributor Author

rjonczy commented Feb 24, 2023

/deploy

@taraxadeploy
Copy link
Collaborator

PR NET pr-2344 Successfully deployed!

explorerhttps://explorer-pr-2344.prnet.taraxa.io
RPChttps://rpc-pr-2344.prnet.taraxa.io

Boot Nodes

Boot Node 134.71.57.165
Boot Node 235.202.138.194
Boot Node 334.70.28.189

RPC Nodes

RPC Node 134.68.10.227
RPC Node 234.72.28.3
RPC Node 334.172.185.107

@rjonczy
Copy link
Contributor Author

rjonczy commented Feb 24, 2023

/cleanup

@rjonczy
Copy link
Contributor Author

rjonczy commented Feb 24, 2023

/deploy

@taraxadeploy
Copy link
Collaborator

PR NET pr-2344 Successfully deployed!

explorerhttps://explorer-pr-2344.prnet.taraxa.io
RPChttps://rpc-pr-2344.prnet.taraxa.io

Boot Nodes

Boot Node 134.71.57.165
Boot Node 235.202.138.194
Boot Node 334.70.28.189

RPC Nodes

RPC Node 134.68.10.227
RPC Node 234.72.28.3
RPC Node 334.172.185.107

@rjonczy
Copy link
Contributor Author

rjonczy commented Feb 27, 2023

/cleanup

@rjonczy
Copy link
Contributor Author

rjonczy commented Feb 27, 2023

/cleanup

@taraxadeploy
Copy link
Collaborator

prnet cleaned-up successfully

@rjonczy
Copy link
Contributor Author

rjonczy commented Feb 27, 2023

/deploy

@taraxadeploy
Copy link
Collaborator

PR NET pr-2344 Successfully deployed!

explorerhttps://explorer-pr-2344.prnet.taraxa.io
RPChttps://rpc-pr-2344.prnet.taraxa.io

Boot Nodes

Boot Node 134.70.28.189
Boot Node 234.172.185.107
Boot Node 335.202.138.194

RPC Nodes

RPC Node 134.71.57.165
RPC Node 234.72.28.3
RPC Node 334.68.10.227

@rjonczy
Copy link
Contributor Author

rjonczy commented Feb 27, 2023

/cleanup

@taraxadeploy
Copy link
Collaborator

prnet cleaned-up successfully

@rjonczy
Copy link
Contributor Author

rjonczy commented Feb 27, 2023

/deploy

@taraxadeploy
Copy link
Collaborator

PR NET pr-2344 Successfully deployed!

explorerhttps://explorer-pr-2344.prnet.taraxa.io
RPChttps://rpc-pr-2344.prnet.taraxa.io

Boot Nodes

Boot Node 134.173.203.104
Boot Node 234.72.180.147
Boot Node 334.70.28.189

RPC Nodes

RPC Node 134.172.185.107
RPC Node 234.71.57.165
RPC Node 335.202.138.194

@rjonczy
Copy link
Contributor Author

rjonczy commented Feb 27, 2023

/cleanp

@rjonczy
Copy link
Contributor Author

rjonczy commented Feb 27, 2023

/cleanup

@taraxadeploy
Copy link
Collaborator

prnet cleaned-up successfully

@rjonczy
Copy link
Contributor Author

rjonczy commented Feb 27, 2023

/deploy

@taraxadeploy
Copy link
Collaborator

PR NET pr-2344 Successfully deployed!

explorerhttps://explorer-pr-2344.prnet.taraxa.io
RPChttps://rpc-pr-2344.prnet.taraxa.io

Boot Nodes

Boot Node 134.68.10.227
Boot Node 234.172.185.107
Boot Node 334.72.180.147

RPC Nodes

RPC Node 135.202.138.194
RPC Node 234.173.203.104
RPC Node 334.72.28.3

@rjonczy
Copy link
Contributor Author

rjonczy commented Feb 27, 2023

/redeploy

@taraxadeploy
Copy link
Collaborator

PR NET pr-2344 Successfully Re deployed!

explorerhttps://explorer-pr-2344.prnet.taraxa.io
RPChttps://rpc-pr-2344.prnet.taraxa.io

Boot Nodes

Boot Node 1
Boot Node 2
Boot Node 3

RPC Nodes

RPC Node 135.202.138.194
RPC Node 234.173.203.104
RPC Node 334.72.28.3

@rjonczy
Copy link
Contributor Author

rjonczy commented Feb 27, 2023

/redeploy

@taraxadeploy
Copy link
Collaborator

Failed to ReDeploy prnet

@rjonczy
Copy link
Contributor Author

rjonczy commented Feb 27, 2023

/deploy

@taraxadeploy
Copy link
Collaborator

PR NET pr-2344 Successfully deployed!

explorerhttps://explorer-pr-2344.prnet.taraxa.io
RPChttps://rpc-pr-2344.prnet.taraxa.io

Boot Nodes

Boot Node 134.72.180.147
Boot Node 234.70.28.189
Boot Node 334.71.57.165

RPC Nodes

RPC Node 134.172.185.107
RPC Node 234.68.10.227
RPC Node 334.173.203.104

rattrap
rattrap requested changes Feb 28, 2023
View reviewed changes
.gitignore Outdated
Comment on lines 43 to 46

# Helm stuff
requirements.lock
Chart.lock
Copy link
Contributor

Choose a reason for hiding this comment

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

We should move these into the chart dir

Copy link
Contributor Author

Choose a reason for hiding this comment

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

moved.

.circleci/config.yml Outdated Show resolved Hide resolved
.circleci/config.yml Outdated Show resolved Hide resolved
charts/taraxa-node/templates/bootnode.yaml Outdated Show resolved Hide resolved
charts/taraxa-node/templates/bootnode.yaml Outdated
image: "{{ .Values.image.repository }}:{{ .Values.image.tag }}"
{{- end }}

{{- if .Values.bootnode.image.imagePullPolicy }}
Copy link
Contributor

Choose a reason for hiding this comment

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

Wrong key. Should be .Values.bootnode.image.pullPolicy

charts/taraxa-node/templates/consensus-node.yaml Outdated Show resolved Hide resolved
charts/taraxa-node/templates/consensus-node.yaml Outdated
image: "{{ .Values.image.repository }}:{{ .Values.image.tag }}"
{{- end }}
{{- if .Values.consensusnode.image.imagePullPolicy }}
Copy link
Contributor

Choose a reason for hiding this comment

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

Wrong key. Should be .Values.consensusnode.image.pullPolicy

charts/taraxa-node/templates/taraxa-node.yaml Outdated Show resolved Hide resolved
charts/taraxa-node/templates/taraxa-node.yaml Outdated Show resolved Hide resolved
@rjonczy rjonczy requested a review from rattrap February 28, 2023 16:31
@rjonczy rjonczy merged commit 9edea9f into develop Feb 28, 2023
@rjonczy rjonczy deleted the chart/v0.3.3 branch February 28, 2023 16:42
@taraxadeploy
Copy link
Collaborator

prnet cleaned-up successfully

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MatusKysel MatusKysel MatusKysel approved these changes

@rattrap rattrap rattrap approved these changes

@kstdl kstdl Awaiting requested review from kstdl

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@rjonczy @taraxadeploy @rattrap @MatusKysel