-
Notifications
You must be signed in to change notification settings - Fork 11
updating to newest release versions #179
Changes from 17 commits
7adfbcd
0ef9c44
5ae35eb
c04286b
23a7fae
c3d9f24
d25c91d
440cd41
4e3d0f7
4125d8e
6efb086
d2ce22f
e032134
d769ec5
b2dea44
a67a926
1db161e
a8abd2a
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -59,4 +59,6 @@ exec celestia bridge start \ | |
--gateway.port "$bridge_host_port" \ | ||
--rpc.addr 0.0.0.0 \ | ||
--rpc.port "$bridge_rpc_port" \ | ||
--keyring.accname "$validator_key_name" | ||
--keyring.accname "$validator_key_name" \ | ||
--log.level "debug" \ | ||
--log.level.module "share/discovery:error" | ||
Comment on lines
+63
to
+64
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Cleaning up logs, and removing share discovery warnings about no networking, cleaner logs in case there is a celestia local issue. |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -85,6 +85,13 @@ spec: | |
port: bridge-rest | ||
failureThreshold: 30 | ||
periodSeconds: 10 | ||
resources: | ||
requests: | ||
cpu: 1 | ||
memory: 4Gi | ||
limits: | ||
cpu: 2 | ||
memory: 8Gi | ||
Comment on lines
+88
to
+94
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Contraining bridge node on dev cluster since it can be an expensive service to run. |
||
- name: token-server | ||
image: {{ .Values.tokenServerImage }} | ||
command: [ "/bin/httpd", "-v", "-f", "-p", "5353", "-h", "/home/celestia/token-server/" ] | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,53 @@ | ||
apiVersion: networking.k8s.io/v1 | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Added ingress which makes it possible to communicate directly with deployed Celestia app and bridge node from local machine. |
||
kind: Ingress | ||
metadata: | ||
name: celestia-local-network-ingress | ||
namespace: {{ $.Values.global.namespace }} | ||
labels: | ||
app: celestia-local-network | ||
annotations: | ||
nginx.ingress.kubernetes.io/proxy-read-timeout: "3600" | ||
nginx.ingress.kubernetes.io/proxy-send-timeout: "3600" | ||
spec: | ||
ingressClassName: nginx | ||
rules: | ||
- host: rpc.node.celestia.localdev.me | ||
http: | ||
paths: | ||
- path: / | ||
pathType: Prefix | ||
backend: | ||
service: | ||
name: celestia-service | ||
port: | ||
name: bridge-jsonrpc | ||
- host: rest.node.celestia.localdev.me | ||
http: | ||
paths: | ||
- path: / | ||
pathType: Prefix | ||
backend: | ||
service: | ||
name: celestia-service | ||
port: | ||
name: bridge-rest | ||
- host: token.node.celestia.localdev.me | ||
http: | ||
paths: | ||
- path: / | ||
pathType: Prefix | ||
backend: | ||
service: | ||
name: celestia-service | ||
port: | ||
name: token-svc | ||
- host: rpc.app.celestia.localdev.me | ||
http: | ||
paths: | ||
- path: / | ||
pathType: Prefix | ||
backend: | ||
service: | ||
name: celestia-app-service | ||
port: | ||
name: app-rpc |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -2,18 +2,18 @@ | |
# This is a YAML-formatted file. | ||
# Declare variables to be passed into your templates. | ||
global: | ||
namespace: "astria-dev-cluster" | ||
namespaceOverride: '' | ||
replicaCount: 1 | ||
|
||
config: | ||
name: celestia-node | ||
network: mocha-4 | ||
chainId: mocha-4 | ||
coreIp: "full.consensus.mocha-4.celestia-mocha.com" | ||
configInfo: false | ||
type: light | ||
coreGrpcPort: 9090 | ||
tokenAuthLevel: 'read' | ||
customInfo: '' | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. more technically correct, still falsey. |
||
|
||
images: | ||
pullPolicy: IfNotPresent | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,6 +1,6 @@ | ||
dependencies: | ||
- name: celestia-node | ||
repository: file://../celestia-node | ||
version: 0.1.11 | ||
digest: sha256:d8bafe8e16323154332ee709b38754df209a51d4577b1d9c80a29898d7733421 | ||
generated: "2024-01-22T12:37:57.716528-08:00" | ||
version: 0.2.0 | ||
digest: sha256:d259cf4f2d0f17d0c70a2e88f9735ab011831410284dade0f5846677fc836ba4 | ||
generated: "2024-02-15T15:04:33.506122-08:00" |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -16,11 +16,17 @@ | |
"ethash": {}, | ||
{{- if .Values.config.rollup.genesis.extraDataOverride }} | ||
"astriaExtraDataOverride": "{{ .Values.config.rollup.genesis.extraDataOverride }}", | ||
{{ end -}} | ||
{{- end }} | ||
"astriaOverrideGenesisExtraData": {{ .Values.config.rollup.genesis.overrideGenesisExtraData }}, | ||
"astriaSequencerInitialHeight": {{ .Values.config.sequencer.initialBlockHeight }}, | ||
{{- if not .Values.global.dev }} | ||
"astriaDataAvailabilityInitialHeight": {{ .Values.config.celestia.initialBlockHeight }}, | ||
"astriaDataAvailabilityHeightVariance": {{ .Values.config.celestia.heightVariance }} | ||
{{- else }} | ||
"astriaRollupName": "{{ .Values.config.rollup.name }}", | ||
"astriaCelestiaInitialHeight": {{ .Values.config.celestia.initialBlockHeight }}, | ||
"astriaCelestiaHeightVariance": {{ .Values.config.celestia.heightVariance }} | ||
{{- end }} | ||
Comment on lines
+22
to
+29
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Note pattern used throughout for configuration which might change between releases. |
||
}, | ||
"difficulty": "10000000", | ||
"gasLimit": "8000000", | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We don't need celestia local to have any peers, it shouldnt have any it's a one off instance.