-
Notifications
You must be signed in to change notification settings - Fork 24
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
Fixes #16024: Make a plugin from the current OpenSCAP technique #200
Fixes #16024: Make a plugin from the current OpenSCAP technique #200
Conversation
f67952f
to
5d00fbd
Compare
Commit modified |
ansible-report/README.asciidoc
Outdated
@@ -0,0 +1,89 @@ | |||
# Rudder plugin: ansible_report |
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.
File extension needs to be .adoc
PR updated with a new commit |
ansible-report/packaging/prerm
Outdated
git rm -r $folder/$NAME | ||
done | ||
git commit -m "Ansible-report plugin uninstallation" | ||
# need to be run 2 times to be updated, first one will fail |
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.
Do you have any idea why?
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.
Not really the error given is the following:
- rudder server reload-techniques
error: Could not reload techniques from files
Error: An error occured when updating the Technique library from file system
cause:Error when trying to delete category with ID 'ActiveTechniqueCategoryId(ansible_report)'
cause:Can not delete 'techniqueCategoryId=ansible_report,techniqueCategoryId=Active Techniques,ou=Rudder,cn=rudder-configuration': subordinate objects must be deleted first
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.
Ok, could you open a ticket and add the link in a comment with a TODO to remove it?
8e99750
to
d710fee
Compare
set -x | ||
|
||
# Import Ansible Technique | ||
FOLDERS="ncf/50_techniques/plugin_openscap_report techniques/plugin_openscap_report techniques/ncf_techniques" |
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.
why committing the whole techniques/ncf_techniques
folder?
OK, merging this PR |
https://issues.rudder.io/issues/16048