Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Hapi module build doesn't recognize services tags #13023

Closed
Tracked by #11230
mhess-swl opened this issue Apr 26, 2024 · 3 comments · Fixed by #13051
Closed
Tracked by #11230

Hapi module build doesn't recognize services tags #13023

mhess-swl opened this issue Apr 26, 2024 · 3 comments · Fixed by #13051
Assignees

Comments

@mhess-swl
Copy link
Member

mhess-swl commented Apr 26, 2024

Description

When building the hapi module (with gradle), I expect the branchOrTag variable here to recognize, build, and incorporate tags created in the hedera-protobufs repo. However, building the hapi module currently either 1) fails or 2) does not include the changes with said tag (if no compiler change is needed)

Steps to reproduce

  • Make a change in the hedera-protobufs repo, e.g. add a new grpc endpoint getXInfo
  • Open a PR with getXInfo to the hedera-protobufs repo, then merge the PR
  • Create a git tag that includes getXInfo
  • Modify the branchOrTag variable in the hedera-services repo to the tag from the previous step, e.g. branchOrTag = "protobuf-tag_v0.91.0"
  • In HapiUtils#functionOf(Query) (services repo), add a new case to the switch statement, e.g. case GET_X_INFO -> HederaFunctionality.GET_X_INFO;
  • Attempt to build the hapi module (or the services project) with ./gradlew assemble. The gradle build will fail with a compilation error because it does not download and incorporate the new protobuf tag, protobuf-tag_v0.91.0

Additional context

According to @jjohannes, the issue is in this line: tags can't have the origin/ prefix. Removing origin/ from this code seems to fix the problem, but I'm not sure what other parts of the build might break as a result.

Hedera network

other

Version

I believe this happens with the current develop branch (commit 8ada1851a)

Operating system

macOS

@jjohannes jjohannes self-assigned this Apr 27, 2024
@jjohannes
Copy link
Collaborator

Thanks for creating this issue. I can look into this.

@jjohannes
Copy link
Collaborator

PR that solves this by letting you explicitly define a "tag" OR a "branch: #13051

@mhess-swl
Copy link
Member Author

Thanks for fixing this @jjohannes! 🙏

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 a pull request may close this issue.

2 participants