Skip to content

Makefile: suppress msggen and grpc gen if SUPPRESS_GENERATION is set#9244

Open
daywalker90 wants to merge 1 commit into
ElementsProject:masterfrom
daywalker90:check-gen-updated-fix
Open

Makefile: suppress msggen and grpc gen if SUPPRESS_GENERATION is set#9244
daywalker90 wants to merge 1 commit into
ElementsProject:masterfrom
daywalker90:check-gen-updated-fix

Conversation

@daywalker90

Copy link
Copy Markdown
Collaborator

there is a race condition between sed rewriting node_pb2.py and node_pb2_grpc.py and the git diff's from check-source causing the prebuild CI step to fail with:

--- a/contrib/pyln-grpc-proto/pyln/grpc/node_pb2_grpc.py
+++ b/contrib/pyln-grpc-proto/pyln/grpc/node_pb2_grpc.py
@@ -3,7 +3,7 @@
 import grpc
 import warnings

-from pyln.grpc import node_pb2 as node__pb2
+import node_pb2 as node__pb2

Changelog-None

@daywalker90 daywalker90 linked an issue Jul 3, 2026 that may be closed by this pull request
@daywalker90 daywalker90 self-assigned this Jul 3, 2026
@daywalker90 daywalker90 requested a review from ddustin July 3, 2026 09:14
@daywalker90 daywalker90 added the Status::Ready for Review The work has been completed and is now awaiting evaluation or approval. label Jul 3, 2026
there is a race condition between sed rewriting node_pb2.py and node_pb2_grpc.py
and the `git diff`'s from check-source causing the prebuild CI step to fail with:

```
--- a/contrib/pyln-grpc-proto/pyln/grpc/node_pb2_grpc.py
+++ b/contrib/pyln-grpc-proto/pyln/grpc/node_pb2_grpc.py
@@ -3,7 +3,7 @@
 import grpc
 import warnings

-from pyln.grpc import node_pb2 as node__pb2
+import node_pb2 as node__pb2
```

Changelog-None
@daywalker90 daywalker90 force-pushed the check-gen-updated-fix branch from ba7da33 to ac8e5dd Compare July 3, 2026 09:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Status::Ready for Review The work has been completed and is now awaiting evaluation or approval.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CI flake in pre-build checks

1 participant