-
Notifications
You must be signed in to change notification settings - Fork 21
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 logging properties to all workflows #411
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@ElaiShalevRH You missed greeting
and request-vm-cnv
and create-ocp-project
workflows
@masayag Maybe we should create a subfolder 'workflows' and put all workflows in it so we don't get lost with all non-related folder that come in between the workflows folders when listed in alphabetical order
@gabriel-farache that's a good idea. this entitle a jira issue on its own, since it isn't only moving the folders, but also adapting all of the scripts and github actions. |
aee5f88
to
079302f
Compare
move2kube/m2k-func/pom.xml
Outdated
@@ -130,13 +131,21 @@ | |||
<version>2.10.0</version> | |||
<scope>test</scope> | |||
</dependency> | |||
|
|||
<dependency> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we might need to add the same to the software template.
5273cec
to
079302f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thx
Related to this ticket: https://issues.redhat.com/browse/FLPATH-1659