Github user dlyle65535 commented on the pull request:
https://github.com/apache/incubator-metron/pull/96#issuecomment-215984656
No specific ideas on the -f issue. I can work on it a bit Sunday or Monday.
Of course, everything should work when it's over including a pre-built
snapshot. I'd just prefer we use the existing Vagrant capability for VM
Provider -> Provisioner handoff.
@cestella - I think we're good on a generic solution, just use the regular
ansible-playbook command with --tags and/or --skip-tags. I don't really see any
reason to abstract that. Not abstracting it has a lot of advantages: it's well
documented by Ansible, anyone with a background in Ansible already knows about
it, we don't have to maintain it- stuff like that. In my opinion, our Vagrant
provisioning is the oddball here as we currently have to modify the Vagrantfile
to leverage tags. That's why I'm recommending that we do something specific in
the Vagrantfile to expose those arguments.
All that said, I'm happy to include the script as well. Though I don't
agree that run_enrichment_role.sh is more intuitive than doing it using Vagrant
and/or ansible-playbook. Mostly for the reasons stated above. Though, I
probably should have suggested the arguments for Vagrant be --tags and
--skip-tags to make it consistent.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
