Thank you for proposing these updates Sean.
+1 on Immediate core removals:
The mentioned cores were very valuable contributors in the past, but they don't have any new activity in the last few years.
I agree that it will be good to do some cleanup now, and encourage new contributors to continue reviewing patches and join the core team in future cycles.
+1 on new watcher-core addition:
Joan's review activity in Watcher's repos[1] is consistent and has been very helpful, with many valuable comments, questions and code's change recommendations. He will be, for sure, a great addition to the core team and help us to land more patches that our current bandwidth allows us.
Also, just want to highlight his recent work in the zone migration strategy and everything else related to storage model updates (collector and notifications). He found many issues in current implementation, reported many bugs found, proposed a lot of fixes and improvements to the code as well to the user and configuration documentation[2][3].
These proposals will be one of the first topics that we will be covering in the next Watcher Team Meeting, on Thursday Sep 11th at 12:00 PM UTC in #openstack-watcher channel[4]. Feel free to join us there.
[1]
https://openstack.biterg.io/app/dashboards#/view/95487340-6762-11e9-a198-67126215b112?_g=(filters:!(),refreshInterval:(pause:!t,value:0),time:(from:now-90d,to:now))&_a=(description:'Gerrit%20Approvals%20panel%20by%20Bitergia',filters:!(('$state':(store:appState),meta:(alias:Bots,disabled:!f,index:gerrit,key:author_bot,negate:!t,params:(query:!t),type:phrase),query:(match:(author_bot:(query:!t,type:phrase)))),('$state':(store:appState),meta:(alias:!n,disabled:!f,index:gerrit,key:project,negate:!f,params:(query:watcher),type:phrase),query:(match_phrase:(project:watcher))),('$state':(store:appState),meta:(alias:!n,disabled:!f,index:gerrit,key:repository,negate:!t,params:(query:openstack%2Fwatcher-tempest-plugin),type:phrase),query:(match_phrase:(repository:openstack%2Fwatcher-tempest-plugin)))),fullScreenMode:!f,options:(darkTheme:!f,hidePanelTitles:!f,useMargins:!t),query:(language:lucene,query:''),timeRestore:!f,title:'Gerrit%20Approvals',viewMode:view)
[2]
https://review.opendev.org/q/owner:jgilaber@redhat.com[3]
https://bugs.launchpad.net/watcher/+bugs?field.searchtext=&orderby=-importance&field.status%3Alist=NEW&field.status%3Alist=CONFIRMED&field.status%3Alist=TRIAGED&field.status%3Alist=INPROGRESS&field.status%3Alist=DEFERRED&field.status%3Alist=FIXCOMMITTED&field.status%3Alist=INCOMPLETE_WITH_RESPONSE&field.status%3Alist=INCOMPLETE_WITHOUT_RESPONSE&assignee_option=any&field.assignee=&field.bug_reporter=jgilaber&field.bug_commenter=&field.subscriber=&field.structural_subscriber=&field.tag=&field.tags_combinator=ANY&field.has_cve.used=&field.omit_dupes.used=&field.omit_dupes=on&field.affects_me.used=&field.has_patch.used=&field.has_branches.used=&field.has_branches=on&field.has_no_branches.used=&field.has_no_branches=on&field.has_blueprints.used=&field.has_blueprints=on&field.has_no_blueprints.used=&field.has_no_blueprints=on&search=Search[4]
https://etherpad.opendev.org/p/openstack-watcher-irc-meeting#L28