Skip to content

feat(ci): provide demo/preview environments#1406

Merged
aajimal merged 34 commits intomainfrom
preview-env-demo
Sep 13, 2024
Merged

feat(ci): provide demo/preview environments#1406
aajimal merged 34 commits intomainfrom
preview-env-demo

Conversation

@aajimal
Copy link
Copy Markdown
Contributor

@aajimal aajimal commented Aug 23, 2024 •
edited
Loading

Summary

Updates the celestia-local chart to allow helm-parameter overrides when creating PR preview environments.

Background

To show how we can leverage a preview environments for:
- testing work in progress PR
- providing a demo environment to external users
- Testing against a live cluster

Changes

  • Update celestia-local chart to:
    • remove hardcoded values
    • allow specifying ingress params
    • ensuring PVC storage is namespace scoped
  • Update sequencer and evm-stack components to support parameterized imagePullPolicy for ArgoCD
  • Include ArgoCD ApplicationSet for PR preview envs (to be applied by admins)

Testing

  • Rendering locally for manual inspection
  • Applied changes to argocd.dev.astria.org to test on live cluster
  • Verify components are being deployed successfully
  • Confirm smoke-tests pass when pointing to deployed components

@github-actions github-actions bot added the cd label Aug 23, 2024
@aajimal aajimal added docker-build used to trigger docker builds on PRs evm pertains to EVM demo sequencer pertaining to the astria-sequencer crate sequencer-faucet and removed evm pertains to EVM labels Aug 23, 2024
@aajimal aajimal added preview required if you intend to generate a preview environment demo sequencer pertaining to the astria-sequencer crate docker-build used to trigger docker builds on PRs evm pertains to EVM and removed demo preview required if you intend to generate a preview environment sequencer pertaining to the astria-sequencer crate evm pertains to EVM docker-build used to trigger docker builds on PRs cd labels Aug 28, 2024
@aajimal
Preview Environment Demo
798d376
@aajimal
Parameterize celestia-local helm chart ingress values
fae0740
@aajimal
Add preview manifests for testing
21b2b7f
@aajimal
point tests to preview env
9f394a1
@aajimal
space
306c87f
@aajimal
Ensure pvc belongs to correct namespace
a21d111
@aajimal
Fix volume/storage templates for evm-rollup
a40304f
@aajimal
Revert "Fix volume/storage templates for evm-rollup"
22f4e9a 
This reverts commit d1d014b.
@aajimal
Remove unnecessary PR tag from PV
456a001
@aajimal
Indents
ba1c83b
@aajimal
Restore smoketest urls
8b64507
@aajimal
Change appset labels to prevent conflict with originals currently run…
a58b062 
…ning
@aajimal
Copy Preview Env manifest parameter overrides to separate values file…
deac048 
…s for helm
@aajimal
Add ingress annotations
691634a
@aajimal
Copy template commits from PR-1458
2541e47
@aajimal
enable storage for evm-rollup values file
c0ac547
@aajimal
bump celestia chart version
53c65fb
@aajimal
Remove preview-env valueFiles until able to render goTemplate vars pr…
b03ea83 
…operly
@aajimal
Remove unused helper template references copied from elsewhere
ddff8e1
@aajimal
Rename argocd manifest files
b7851e9
@aajimal
Rename argocd manifest folder
3075bbb
@aajimal
Update manifests w/ ingress annotations, clean up vars, add documenta…
3d8b8c5 
…tion
@aajimal
Parammeterize imagePullPolicy for sequencer components
f1f9ae5
@aajimal
Parameterize imagePullPolicy for evm-stack components
30a9e79
@aajimal
Parameterize imagePullPolicy for remaining components
364c5f8
@aajimal
Bump chart versions
d10f366
@aajimal
Merge branch 'main' into preview-env-demo
cb27f40
@aajimal
Bump chart versions again
6c78032
@aajimal
Cleanup preview-env manifests after verifying smoke-tests
464308d
@aajimal
Add some useful helpers for helm/argo manifests
519bf24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@joroshiba joroshiba Awaiting requested review from joroshiba

2 more reviewers

@WafflesVonMaple WafflesVonMaple WafflesVonMaple approved these changes

@quasystaty1 quasystaty1 quasystaty1 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

cd docker-build used to trigger docker builds on PRs evm pertains to EVM preview required if you intend to generate a preview environment sequencer pertaining to the astria-sequencer crate

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@aajimal @SuperFluffy @WafflesVonMaple @quasystaty1