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

Failed to compile the sdk generated by model 4.3.22 #151

Closed
imjoey opened this issue Apr 9, 2019 · 0 comments
Closed

Failed to compile the sdk generated by model 4.3.22 #151

imjoey opened this issue Apr 9, 2019 · 0 comments

Comments

@imjoey
Copy link
Member

imjoey commented Apr 9, 2019

The problem

After updating the model to 4.3.22, the generated sdk had compiling errors.

Environment

  • Desktop OS/version

macOS Mojave (version 10.14.4)

  • Java version to run maven

openjdk version "11.0.2" 2019-01-15

  • Go version under test

go1.12.1 darwin/amd64

  • The metamodel version speicifed in the root pom.xml

1.3.0

  • The model version sepecified in the root pom.xml

4.3.22

Description

When running mvn package command with the model 4.3.22, the compile phrase would exit unexpectedly with the following errors output.

[INFO] --- exec-maven-plugin:1.2.1:exec (compile) @ go-sdk ---
_/Users/joey/Work/ovirt/ovirt-engine-sdk-go/sdk/ovirtsdk
# _/Users/joey/Work/ovirt/ovirt-engine-sdk-go/sdk/ovirtsdk
ovirtsdk/services.go:40613:18: cannot use upgradeAction (type ClusterUpgradeAction) as type *ClusterUpgradeAction in assignment
ovirtsdk/services.go:40623:31: cannot use p.upgradeAction (type *ClusterUpgradeAction) as type ClusterUpgradeAction in argument to actionBuilder.UpgradeAction

Expected behavior: The compile phrase passed successfully.

Actual behavior: Failed to compile the sdk due to the errors above.

Reproduces how often: 100%

imjoey added a commit to imjoey/ovirt-engine-sdk-go that referenced this issue Apr 9, 2019
This pr will also fix the bug oVirt#151 that failed to compile the sdk
generated using model 4.3.22. The model 4.3.22 added a enum type model
as an param of ClusterUpgrade action request.

Signed-off-by: imjoey <[email protected]>
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

No branches or pull requests

1 participant