Skip to content

Conversation

@DaedalusG
Copy link
Contributor

closes REL-807
https://linear.app/sourcegraph/issue/REL-807/generate-issue-for-helm-doc-update-regarding-version-targeting

Pull Request approval

You will need to get your PR approved by at least one member of the Sourcegraph team. For reviews of docs formatting, styles, and component usage, please tag the docs team via the #docs Slack channel.

@vercel
Copy link

vercel bot commented Mar 26, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
sourcegraph-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Mar 27, 2025 7:27pm

Copy link
Contributor

@loujar loujar left a comment

Choose a reason for hiding this comment

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

A few notes, but otherwise looks great. thanks!

```bash
kubectl exec -it pgsql -- psql -U sg -c "SELECT version();"
```
- The output should show PostgreSQL 16.x
Copy link
Contributor

Choose a reason for hiding this comment

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

Just curious, aren't we locked to a specific postgres 16 version in these Sourcegraph versions? If so, could we make this note a bit more precise?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ahh good point yea, I think its 16.6, but actually and 16 version would do

@DaedalusG DaedalusG merged commit 3410b6b into main Mar 27, 2025
4 of 5 checks passed
@DaedalusG DaedalusG deleted the wg/rel/generalize-helm-mvu-warning branch March 27, 2025 19:26
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.

3 participants