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 diagrams generated with Cloudnet TOSCA toolbox #50

Closed
wants to merge 3 commits into from

Conversation

philippemerle
Copy link
Contributor

Related to #48

@anzoman anzoman self-requested a review November 17, 2021 09:16
@anzoman anzoman added enhancement New feature or request TOSCA This has to do something with TOSCA standard labels Nov 17, 2021
Copy link
Contributor

@anzoman anzoman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@philippemerle thanks for pushing #50 forward and generating these diagrams.

I left a comment about this here: #48 (comment).

Comment on lines +1 to +3
<map id="tosca-intrinsic-functions-service-uml2-class-diagram_map" name="tosca-intrinsic-functions-service-uml2-class-diagram_map">
<area shape="rect" id="id1" href="'a'," title="&#39;a&#39;," alt="" coords="565,179,717,192"/>
</map>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm, why is this file different than the others?

Comment on lines +10 to +15
# Check that the CLOUDNET_BINDIR environment variable is set correctly.
if [ ! -f ${CLOUDNET_BINDIR}/cloudnet_rc.sh ]
then
echo "The CLOUDNET_BINDIR environment variable is not set correctly!"
exit 1
fi
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This script can be executed if the user has already installed Cloudnet TOSCA toolbox, right? It would be nice to have also the installation part here in this script.

@@ -0,0 +1,26 @@
######################################################################
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would rename this folder to .cloudnet-tosca-toolbox for being clearer. It's important that this directory name starts with the dot (.) to keep it hidden. This is needed because xOpera examples are already separated into folders and we cannot have a folder for generating diagrams on the same level.

rm diagrams/*-workflow-diagram.plantuml

# Generate UML2 diagrams.
generate_uml2_diagrams diagrams/*.plantuml
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd also rather see that each diagram would be separated the folder that contains the corresponding example.

@anzoman
Copy link
Contributor

anzoman commented Jan 24, 2024

Closing due to inactivity and can be reopened in the future.

@anzoman anzoman closed this Jan 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request TOSCA This has to do something with TOSCA standard
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants