Skip to content

Conversation

@MasterPtato
Copy link
Contributor

@MasterPtato MasterPtato commented Feb 20, 2024

Fixes RVT-3384

feat(Infra): Reduce minimal infrastructure required to get Rivet running
fix(Infra): Disabled K8s Dashboard, Prometheus and friends (Vector, Loki, Promtail), Clickhouse, NSFW Check API, and Image Resizing (via Imagor) by default

@graphite-app
Copy link
Contributor

graphite-app bot commented Feb 20, 2024

Your org requires the Graphite merge queue for merging into main

You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link.

You can enable merging using labels in your Graphite merge queue settings.

Copy link
Contributor Author

MasterPtato commented Feb 20, 2024

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @MasterPtato and the rest of your teammates on Graphite Graphite

@linear
Copy link

linear bot commented Feb 20, 2024

@MasterPtato MasterPtato force-pushed the 02-20-Reduce_required_infra branch 3 times, most recently from e103c9a to 7eced16 Compare February 21, 2024 23:52
@MasterPtato MasterPtato marked this pull request as ready for review February 21, 2024 23:52
@MasterPtato MasterPtato force-pushed the 02-20-Reduce_required_infra branch from 7eced16 to d4d2728 Compare February 22, 2024 00:42
Copy link
Member

@NathanFlurry NathanFlurry left a comment

Choose a reason for hiding this comment

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

changelog required.

did you have a sec to validate that the default bolt init config is still valid?

can we move the docs/getting_started/START_TIMES.md to something like docs/benchmarks/START_TIMES.md or docs/research/START_TIMES.md? geeting started is meant for teaching the fundamentals about the repo.

@MasterPtato MasterPtato force-pushed the 02-20-Reduce_required_infra branch from d4d2728 to 50c339e Compare February 23, 2024 01:29
Copy link
Contributor Author

@NathanFlurry yes, the config produced by bolt init is still valid

@MasterPtato MasterPtato force-pushed the 02-20-Reduce_required_infra branch 2 times, most recently from 0cd64b5 to 529d636 Compare February 23, 2024 01:31
Copy link
Contributor

@AngelOnFira AngelOnFira left a comment

Choose a reason for hiding this comment

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

LGTM, just requesting changes to make sure I get the changelog setup in first

@MasterPtato MasterPtato force-pushed the 02-20-Reduce_required_infra branch from 529d636 to e44d6e3 Compare March 6, 2024 01:02
@MasterPtato MasterPtato force-pushed the 02-20-Reduce_required_infra branch from e44d6e3 to 46f1dd1 Compare March 15, 2024 20:09
@MasterPtato MasterPtato force-pushed the 02-20-Reduce_required_infra branch 2 times, most recently from 3546ed0 to d68d8db Compare April 4, 2024 17:39
@MasterPtato MasterPtato force-pushed the 02-20-Reduce_required_infra branch from d68d8db to 8bb10d2 Compare April 18, 2024 18:17
@graphite-app
Copy link
Contributor

graphite-app bot commented Apr 18, 2024

Merge activity

@MasterPtato MasterPtato force-pushed the 02-20-Reduce_required_infra branch 3 times, most recently from 8b1c63b to 692671a Compare April 23, 2024 19:43
Fixes RVT-3384

feat(Infra): Reduce minimal infrastructure required to get Rivet running
fix(Infra): Disabled K8s Dashboard, Prometheus and friends (Vector, Loki, Promtail), Clickhouse, NSFW Check API, and Image Resizing (via Imagor) by default
@MasterPtato MasterPtato force-pushed the 02-20-Reduce_required_infra branch from 692671a to baba883 Compare April 23, 2024 20:13
@graphite-app graphite-app bot merged commit baba883 into main Apr 23, 2024
@graphite-app graphite-app bot deleted the 02-20-Reduce_required_infra branch April 23, 2024 20:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants