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

added fix for failing preconfigure_nic & bumped the version to 2.0.3 #24

Conversation

jrzeszutek
Copy link

No description provided.

@EarthmanT
Copy link
Collaborator

Hi @jrzeszutek Thanks for the PR. Why does this prevent the issue? It seems the root cause is packaging the pyvcloud from their broken releases in PyPi.

@jrzeszutek
Copy link
Author

Hi @jrzeszutek Thanks for the PR. Why does this prevent the issue? It seems the root cause is packaging the pyvcloud from their broken releases in PyPi.

Hi @EarthmanT I think this is not related to the previous issue, this is something else. At the debug session, the file which you mention was replaced already, so the one you fixed was in use at that moment. The deployment was still failing, but not in every execution, with this error: https://github.com/cloudify-cosmo/cloudify-common/blob/3bb3c08c2bdce5514ea55122d8b8ca0b96c16544/cloudify/manager.py#L512 So I suppose it was failing only when there was a retry operation. It tried to set a message in the runtime properties of the target node (server?) using the port relationship, which is not supported I guess (I'm not sure, but the error message says so). After removing this part, it stopped failing.

@EarthmanT EarthmanT merged commit 1d3405f into cloudify-cosmo:RD-3371-Tosca-Compat Dec 15, 2021
@EarthmanT
Copy link
Collaborator

@jrzeszutek OK thanks. Merged. Will you build a new wagon - making sure to get the fork of the pyvcloud?

@jrzeszutek
Copy link
Author

@EarthmanT This is another issue, but I don't have it fixed. Looks like the wagon builder we have in dockerhub is ignoring the constraints.txt file
wagon_builder_constraints_issue.txt

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 this pull request may close these issues.

None yet

2 participants