Skip to content

chore(workflows): Buld postgres container#129

Merged
devsjc merged 2 commits intomainfrom
devsjc/build-pgdb
Mar 2, 2026
Merged

chore(workflows): Buld postgres container#129
devsjc merged 2 commits intomainfrom
devsjc/build-pgdb

Conversation

@devsjc
Copy link
Contributor

@devsjc devsjc commented Feb 27, 2026

Contribution Checklist

  • Have you followed the Open Climate Fix Contribution Guidelines?
  • Have you referenced the Issue this PR addresses, where applicable?
  • Have you checked to ensure there aren't other open Pull Requests for the same change?
  • Have you added a summary of the changes?
  • Have you written new tests for your changes, where applicable?
  • Have you successfully run make lint with your changes locally?
  • Have you successfully run make test with your changes locally?

Warning

PRs may be closed if all the above boxes are not checked.

Changes in this Pull Request

Adds a build step to make a postgres container.

@peterdudfield
Copy link
Contributor

Here's the current error
Screenshot 2026-02-27 at 16 51 11

@peterdudfield
Copy link
Contributor

I think this is due to chagnes here

@github-actions
Copy link

github-actions bot commented Feb 27, 2026

Benchmark Results

Benchmark results
?   	github.com/openclimatefix/data-platform/cmd	[no test files]
?   	github.com/openclimatefix/data-platform/internal/gen/ocf/dp	[no test files]
?   	github.com/openclimatefix/data-platform/internal/interceptors	[no test files]
PASS
ok  	github.com/openclimatefix/data-platform/internal/server/dummy	0.004s
{"level":"debug","time":"2026-03-02T11:20:31Z","message":"Completed migrations"}
goos: linux
goarch: amd64
pkg: github.com/openclimatefix/data-platform/internal/server/postgres
cpu: Intel(R) Xeon(R) Platinum 8370C CPU @ 2.80GHz
BenchmarkPostgresClient/2400000/GetForecastAsTimeseries-4         	      63	  18189336 ns/op
BenchmarkPostgresClient/2400000/GetForecastAtTimestamp-4          	      60	  19398117 ns/op
BenchmarkPostgresClient/2400000/GetObservationsAsTimeseries-4     	    1126	    992314 ns/op
BenchmarkPostgresClient/2400000/CreateForecast-4                  	      61	  18580926 ns/op
PASS
ok  	github.com/openclimatefix/data-platform/internal/server/postgres	1128.129s
?   	github.com/openclimatefix/data-platform/internal/server/postgres/gen	[no test files]

@peterdudfield
Copy link
Contributor

In docker compose I set command: - "postgres" - "-c" - "shared_preload_libraries=pg_stat_statements,pg_cron"
and this seemed to work

@peterdudfield
Copy link
Contributor

In docker compose I set command: - "postgres" - "-c" - "shared_preload_libraries=pg_stat_statements,pg_cron" and this seemed to work

Add CMD [ "postgres","-c","shared_preload_libraries=pg_stat_statements,pg_cron"] to here seems to work

@peterdudfield
Copy link
Contributor

In docker compose I set command: - "postgres" - "-c" - "shared_preload_libraries=pg_stat_statements,pg_cron" and this seemed to work

Add CMD [ "postgres","-c","shared_preload_libraries=pg_stat_statements,pg_cron"] to here seems to work

With these changes, im still getting the error when adding metadata to uk-pvnet of

E           grpclib.exceptions.GRPCError: (<Status.INTERNAL: 13>, 'grpc: error unmarshalling request: proto: cannot parse invalid wire-format data', None)

@devsjc devsjc merged commit 80d231a into main Mar 2, 2026
4 of 5 checks passed
@devsjc devsjc deleted the devsjc/build-pgdb branch March 2, 2026 11:45
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.

2 participants