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

importing .proto file sometimes fails when running the tests with the docker image #250

Open
scheroiu opened this issue Aug 13, 2024 · 4 comments
Labels
bug Something isn't working status:needs-info To investigate this, more information is needed

Comments

@scheroiu
Copy link

scheroiu commented Aug 13, 2024

Describe the bug
We are running some Kreya grpc tests using Azure pipelines. Sometimes the pipeline fails with "Importing proto files failed."
We are using the 1.15 docker image of kreyac
We downgraded to the 1.14 docker image of kreyac and it works without issue

To Reproduce
Steps to reproduce the behavior:

  1. Have the kreya tests and all dependencies (proto file) committed in a Azure git repo
  2. Have a Azure Pipeline that: deploys the docker image in the k8s aks cluster, clones the repo in the deployed images, runs the tests
  3. The tests execute => "Importing proto files failed."

Expected behavior
The tests need to run consistently each and every time

Environment (if possible, copy the information from the error dialog or the About menu):

  • Azure aks environemnt the kreyac docker image
  • Kreya Version 1.15

Additional context
Using version 1.14 the tests execute without issue.
The tests are executed using the following commande: kreyac operation invoke --scripting-timeout 60 -p /home/tests/Kreya/ ./com/**

The proto file is part of the project folder and defined in the project file as such:
"importStreams": [ { "id": "0b63a0d5-5a47-425e-a5e1-05c3d5cfa78d", "name": "ADP native", "importerName": "ProtoFile", "createMissingOperations": true, "options": { "files": [ "adp_service.proto" ], "importPaths": [] } } ],

@scheroiu scheroiu added the bug Something isn't working label Aug 13, 2024
@CommonGuy
Copy link
Contributor

Could you run the command with the --verbose option and see if there is any additional information?

@CommonGuy CommonGuy added the status:needs-info To investigate this, more information is needed label Aug 13, 2024
@scheroiu
Copy link
Author

grpc_adp_2024-08-19_07-46_580801.log

I attached a verbose log for a run from yesterday. I see nothing wrong.
Note: I replaced with '***' some of the fields is the log: host; pathAndQuery; hostNameOrAddress; authValue

I will try and add later today also a log with the working 1.14 version

@CommonGuy
Copy link
Contributor

If you use Kreya 1.15 (the GUI version, not the CLI) locally and run the importer (via the "refresh icon" above the operation list), does that also result in an error?

@scheroiu
Copy link
Author

scheroiu commented Sep 5, 2024

I get no error. Just the confirmation "API definitions imported".
When running the tests in the GUI version I get no error
when running the tests in the downloaded Kreya CLI tools i get no error.

Just running the tests using the docker image has issues on 1.15.

Maybe there is some sort of dependency when using Microsoft aks clusters?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working status:needs-info To investigate this, more information is needed
Projects
None yet
Development

No branches or pull requests

2 participants