forked from openmobilityfoundation/mds-core
-
Notifications
You must be signed in to change notification settings - Fork 9
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
add ability to deploy an observable nats without stan stack #185
Merged
Merged
Changes from all commits
Commits
Show all changes
89 commits
Select commit
Hold shift + click to select a range
7855f5f
spike: surveyor
jwtodd c50b60c
mdsctl: spike kill-switch; eg: MDS_NO_BREW=true ./bin/mdsctl
jwtodd 9660dc1
Merge branch 'develop' into feature/jwtodd-surveyor
jwtodd 928826f
+surveyor, no_brew kill-switch
jwtodd 4c4bb78
Merge branch 'develop' into feature/jwtodd-surveyor
jwtodd d544962
pin stan(s) to mds ns, wire stan:cluster-id=tenantId(default:mds)
jwtodd dde4398
stan
jwtodd 0733b51
Merge branch 'develop' into feature/jwtodd-surveyor
jwtodd bd120a7
bootstrap nsc; next: deploy credentialed (nats,stan), surveyor
jwtodd bf472f2
nats: u/g operator 0.7, nats 2.1.2
jwtodd d7630ef
natsbox: fix yml typo
jwtodd f4e027f
nats-server/nats-account-server authenticated; next: nets-surveyor
jwtodd da66bd4
mdsctl: simplier nsc-account-id util
jwtodd 216ddca
mdsctl/install:nats : progressive (via kubectl-wait) deploy, authenti…
jwtodd 599ecee
mdsctl: istio-wait
jwtodd 6f96eba
mdsctl/helm: broaden support for `-p local` (ie drive down cpu/mem re…
jwtodd dcfd016
mdsctl: invokable from zsh
jwtodd 6cdabc0
mdsctl/installNats: use nats-operator helm charts
jwtodd fe9e92d
mdsctl: use nats-operator helm charts
jwtodd ce82076
stan
jwtodd b977f95
Merge branch 'develop' into feature/jwtodd-surveyor
jwtodd 66cb56e
mdsctl: authenticated/connected stan; next: surveyor,mds
jwtodd 9987474
mdsctl: u/d stan:0.17.0, stan:authenticted, surveyor up with exceptio…
jwtodd 5266dbb
mdsctl: authenticated surveyor; next: verify with natsbox, credential…
jwtodd b3ffed6
mdsctl: add s'more helm plugins (eg: diff, secrets, etc)
jwtodd 6772315
nats: helm cleanup, make (max_age, max_bytes) configurable
jwtodd 6d5d5bc
helm: cleanup; nats-surveyor: make cluster-size configurable
jwtodd 5dd11fd
helm: mds+stan-creds
jwtodd cc87dcb
surveyor-observations: externalize to helm/values
jwtodd 118cf63
mdsctl: nsc credentials override
jwtodd 18d7e6b
mdscl: nsc cleanup
jwtodd 37e76fc
Merge branch 'develop' into feature/jwtodd-surveyor
jwtodd c50814d
mdsctl: remove stan from bootstrap; helm/nats-account-server: change …
jwtodd 0574c18
mdsctl: cd to nsc store-home
jwtodd 670bed3
fix: nats-account-server deployment template, stan-namespace typo
jwtodd 849eae1
fix: surveyor sys.creds, stan namespace, mds to nats egress
jwtodd 72af09c
mdctl: s/pause-kubectl=45/pause-kubectl=60/
jwtodd 61c2409
mdsctl: provide config overlay for profile(s)
jwtodd a241276
mdsctl: cleanup nsc/credentials conf
jwtodd cad71ec
mdsctl: support credential store config modality
jwtodd 0801b8c
mdsctl: nsc install cleanup
jwtodd a80ffbf
patch
jwtodd cf80c61
container-images: patch for removed -c:foo=bar syntax
jwtodd 178f5cb
diags: first cut at an agency->nats->processors sequence diag
jwtodd e21ca2e
mdsctl: natsbox configured to leverage surveyor creds, update readme …
jwtodd 7a9eeb6
mdsctl: zip 'n ship nas operator data
jwtodd 062a294
mdsctl: nas u/d
jwtodd 6906abe
nats: rbac=false, split out 1x nats-init from nats-operator
jwtodd c152611
Merge branch 'develop' into feature/jwtodd-surveyor
jwtodd 3a08d69
nats: deploys to aws:individual-developers
jwtodd 3d219b3
nats: support no-nats deployments
jwtodd dc25569
surveyor-prometheus now starts up
jwtodd 9cebdc6
mdsctl: better support of simulator operatsions, add values to nats-s…
jwtodd 8602a9b
simple prometheus nats pub/sub event counter; next: figure out how to…
jwtodd d0a2d4c
bit more prometheus
jwtodd e950f15
nats: isolated without surveyor (credentials)
jwtodd 2b0312e
mdsctl: override default nats namespace
jwtodd 1b2cc47
remote
jwtodd 9b21c23
mdsctl: conditionally leverage cloud-ops infra
jwtodd 06bc61d
mdsctl: install mds w/ cloud-ops env cleanup
jwtodd 23317ca
mdsctl: glob helm unittest
jwtodd ad88dfa
Merge branch 'develop' into feature/jwtodd-surveyor
jwtodd be77ca0
mds/values: comment out ( geography-author, jurisdiction) until image…
jwtodd 45c5be5
fix build image: jurisdiction,geography-author
jwtodd cca0a96
nats-streaming: change pvc from 20G->11G
jwtodd 19018c2
nit
jwtodd 6f43d78
mdsctl: put pg/pwd back in for local deploy
jwtodd 90532bf
working through deploying nats/stan to non-default ns
jwtodd e49bfaf
Merge branch 'develop' into feature/jwtodd-surveyor
jwtodd 9fbce17
drop nats-operator
jwtodd f5f63d7
drop helm/nats-streaming-server chart
jwtodd a3ece00
s/stan-creds/nats-creds/g
jwtodd 367f739
Merge branch 'develop' into feature/jwtodd-surveyor
jwtodd 712d4e4
Merge branch 'develop' into feature/jwtodd-surveyor
jwtodd bdcee3d
more stann cleanup
jwtodd 56e5156
Merge branch 'develop' into feature/jwtodd-surveyor
jwtodd 456838d
fix: omit operations pre-check
jwtodd fcfabcb
cleanup
jwtodd 9da24c5
Merge branch 'develop' into feature/jwtodd-surveyor
jwtodd 963f209
fix mds-telemetry buildImage
jwtodd 3c830e3
patch deployment.yaml with conditional nats block
jwtodd 35fd5ba
Merge branch 'develop' into feature/jwtodd-surveyor
jwtodd 6d64df9
patch: (alex) re-imagined service-deployment-status check
jwtodd 887d74c
better helm-repo-up
jwtodd a63840c
u/d package.json, yarn.lock
jwtodd e54cb5a
Merge branch 'develop' into feature/jwtodd-surveyor
jwtodd 759499c
kick build
jwtodd bb73947
Merge branch 'develop' into feature/jwtodd-surveyor
avatarneil 1655251
Remove lingering mds-config artifacts
avatarneil File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Large diffs are not rendered by default.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,5 @@ | ||
apiVersion: v1 | ||
appVersion: '0.4.0' | ||
description: NATS Account Server | ||
name: nats-account-server | ||
version: 0.8.4 |
73 changes: 73 additions & 0 deletions
73
helm/nats-account-server/templates/nats-account-server.yaml
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,73 @@ | ||
--- | ||
apiVersion: apps/v1 | ||
kind: Deployment | ||
metadata: | ||
name: nats-account-server | ||
namespace: {{ $.Release.Namespace | default "default" }} | ||
labels: | ||
app: nats-account-server | ||
annotations: | ||
sidecar.istio.io/inject: 'false' | ||
spec: | ||
replicas: 1 | ||
selector: | ||
matchLabels: | ||
app: nats-account-server | ||
template: | ||
metadata: | ||
labels: | ||
app: nats-account-server | ||
spec: | ||
initContainers: | ||
- name: nats-operator-data | ||
image: busybox:1.28 | ||
command: | ||
- 'unzip' | ||
- '/tmp/nats-operator-data.zip' | ||
- '-d' | ||
- '/nsc-operator' | ||
volumeMounts: | ||
- mountPath: /nsc-operator | ||
name: nsc-operator | ||
- mountPath: /tmp | ||
name: nsc-operator-data | ||
containers: | ||
- name: nats-account-server | ||
image: synadia/nats-account-server:0.8.4 | ||
imagePullPolicy: Always | ||
resources: | ||
limits: | ||
cpu: {{ $.Values.resourcesLimitsCpu | default "500m" }} | ||
memory: {{ $.Values.resourcesLimitsMemory | default "512Mi" }} | ||
requests: | ||
cpu: {{ $.Values.resourcesRequestsCpu | default "50m" }} | ||
memory: {{ $.Values.resourcesRequestsMemory | default "128Mi" }} | ||
volumeMounts: | ||
- name: nsc-operator | ||
mountPath: /nsc-operator | ||
command: | ||
- 'nats-account-server' | ||
- '-DV' | ||
- '-hp' | ||
- '0.0.0.0:9090' | ||
- '-nsc' | ||
- '/nsc-operator/KO' | ||
volumes: | ||
- name: nsc-operator | ||
emptyDir: {} | ||
- name: nsc-operator-data | ||
configMap: | ||
name: nats-operator-data | ||
--- | ||
apiVersion: v1 | ||
kind: Service | ||
metadata: | ||
name: nats-account-server | ||
namespace: {{ $.Release.Namespace | default "default" }} | ||
spec: | ||
selector: | ||
app: nats-account-server | ||
ports: | ||
- name: http-nats-account-server | ||
port: 80 | ||
targetPort: 9090 |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
I was wondering about this - why does the nats namespace also get prefixed to the service name? seems redundant.
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.
yeah we could fix/reduce the redundancy.
this might appear in other templates as well so we should be consistent regardless.
likely just picked the pattern up and continued using it.
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.
I think this should actually just pick up the nats endpoint name from
.Values.natsUrl
or something like that. We shouldn't be trying to construct what the endpoint is from a different software install. What if we're using an external (hosted) nats server for example? I'd recommend to replace.Values.natsNamespace
with.Values.natsUrl
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.
i'd like to push this to a subsequent incremental patch given as implemented it follows the current conventions for cluster-hosted endpoint resolution