Jason Frey
Jason Frey
cc @agrare Thoughts here? Sounds similar to #924 ?
cc @kbrock
Removed this from the Jansa milestone... @gtanzillo please link to an existing roadmap item or add it directly to the roadmap.
What about new files though?
Yeah I was thinking that maybe we get pronto over the finish line, and then like you said just inherit a lot from that. I wonder if our "custom" commit...
Really like this idea overall...would be cool to be able to run the guts of the bot locally. I have to wrap my head around it a bit more. I'm...
strange...how is this working in production? We definitely don't have rules for every repo, and according to https://github.com/ManageIQ/miq_bot/blob/8bbabd9c3dc8c7dd6755d3b7128b3ad3426713d4/app/workers/commit_monitor_handlers/commit_range/path_based_labeler.rb#L31 a missing repo will return nil.
Right, so I can't understand how's it's working in production presently. It should be blowing up constantly.
yeah, it turns out it's a coincidence that it's working in production...the settings are: ```yaml path_based_labeler: included_repos: - ManageIQ/manageiq - ManageIQ/manageiq-ui-classic rules: ManageIQ/manageiq: - ... ManageIQ/manageiq-ui-classic: - ... ``` So,...
Note that we believed this was because the failing environment was FIPS enabled, but we've tried on other FIPS enabled envs and we can't duplicate it.