-
Notifications
You must be signed in to change notification settings - Fork 12
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
SF SDK throws FormatException when building example. #131
Comments
Problem is related to regional settings in Windows, The The decimal value was found in
to this:
|
@mholo65 Good find @amanbha |
I tried changing the dot to a comma as @mholo65 showed and that did pass the But then it says that ServiceFabric failed to deploy. Looking in the Event Viewer it gives the FormatException with the comma. Only way for me to succeed was to change it to 1 cpu to get rid of the decimal problem. |
@JohanSellberg - thanks for the info. We will get this fixed in an update soon. FYI: The SfSbzYamlMerge.exe outputs a JSON file which get's posted to the local cluster endpoint. Can you share what's in the JSON file? The output folder is the one specified by the /o: parameter
|
I am not sure how this local deploy process works.. Here are the steps I took and the outputs: I first changed back to the comma and pressed F5 I then get the failed to deploy as previously mentioned.
But the output folder had not updated. the json file had 1.0 for cpus. as the last sucessfull. I then closed the project and reopened it (not sure but when opening project it tries to deploy). After it was done with a failed to deploy, the output folder was updated.
So output folder had with a . |
Thanks @JohanSellberg @anantshankar17, can you please help look in to this? |
Hello @JohanSellberg |
Sure here is my information from the Output when using , in the yml file:
As you see in the end it still fails even if it has changed it to a dot. And when I check the error message (could only find error message in the EventLog). I can see that the error is still FormatException. As previously posted the error is:
So when I use , the yml file works as expected. But ApplicationManifest gets Invalid FormatException. |
Can you manually execute the cmd and collect the cluster traces and share them ? C:\WINDOWS\sysnative\WindowsPowerShell\v1.0\powershell.exe -NoProfile -ExecutionPolicy Bypass -Command "import-module 'C:\Program Files\Microsoft SDKs\Service Fabric\Tools\Mesh\Scripts\PSModule\ServiceFabricHttpPSModule.dll'; Connect-SFCluster -Endpoint http://localhost:19080; .'C:\Program Files\Microsoft SDKs\Service Fabric\Tools\Mesh\Scripts\Deploy-ServiceFabricApplicationResource.ps1' -ApplicationResourcename SFMeshTestApp -ResourceDescriptionFile 'C:\Users\jose01\source\repos\SFMeshTestApp\SFMeshTestApp\obj\SFApp\merged-application-SFMeshTestApp.json'" Since this is the app json that you're deploying manually, you may also validate if this file doesn't container the comma which is which visible in the error string: " Exception: 0,5 är inte ett giltigt värde för Double". It is strange to see this error if the json is successfully getting generated without the comma separator. |
Well if you look close the second EventViewer Message it isn't the json, it is |
I tried the command but I still get an error in the Event Log... I also get a related warning at the same time.. Maybe that gives some more information.. I copies the XML for the warning:
|
I would still need the cluster traces to makeout what is not working correctly. Can you please share the traces ? For now I would suggest you to not use the comma, until we figure out what all is not working. In the meantime we will try to repro this locally to debug faster. The locale that you have is fi-FI, right? |
Not fully sure how to check locale but should be sv or sv-SE since its swedish. |
You would find the traces here: C:\SfDevCluster\Log\Traces |
Ok here is the trace file no0t sure if you need the lease_traces too, just say if you do. (both in 1 zip was too big) Also confirmed with powershell that my locale is sv-SE
|
I'm having the exact same problem. Did you find a solution? |
@nforss For now we suggest you avoid using the decimal values in the yamls. we will have fixes rolled out for these issues soon. |
These issues are all fixed in the latest SF Release 6.4.637 and SDK version 3.3.637. Kindly try them out and let us know. |
I've been following guides here, but when I create the project it fails at building and deployng the Docker image.
Output from
Service Fabric Tools
:The text was updated successfully, but these errors were encountered: