Channels
j
Jared Rhizor (Airbyte)
01/14/2021, 10:06 PM
What do you think of these badges for connector build status?
u
user
01/14/2021, 10:06 PMIs it clear what the colors and numbers mean @s and @charles?
u
user
01/14/2021, 10:07 PMoooh. pretty.
u
user
01/14/2021, 10:07 PMi actually don't think i know what yellow means.
u
user
01/14/2021, 10:07 PMwell
u
user
01/14/2021, 10:07 PMAlso what should we say if it’s not certified?
u
user
01/14/2021, 10:07 PMjust no message?
u
user
01/14/2021, 10:07 PMi assume green means last build passed. red means last build failed. i don't know what yellow means.
u
user
01/14/2021, 10:07 PMI was thinking yellow would be the most recent one passed but it has at least one failure in the last 10 builds
u
user
01/14/2021, 10:09 PMi think i would just do green and red then. i think that's too nuanced.
u
user
01/14/2021, 10:09 PMthat or you need to explain it whereever these get posted.
u
user
01/14/2021, 10:10 PMbut it's not self evident (at least for me)
u
user
01/14/2021, 10:30 PMI think I’ll document it. It isn’t self evident but it’ll make a dashboard more scannable
u
user
01/14/2021, 10:30 PM👍
u
user
01/14/2021, 10:44 PMthe different categories are: Alpha, beta, Airbyte certified
u
user
01/14/2021, 10:45 PMor actually, let’s just make an alpha, <blank>, and AB certified
u
user
01/14/2021, 10:47 PM@s should we add that as a field to SourceDefinitions?
u
user
01/14/2021, 10:48 PMI guess what is the ground truth of alpha/blank/certified?
u
user
01/14/2021, 10:50 PMbecause this standard is centrally enforced managed, I’m having trouble thinking about how it would be displayed in the app itself
u
user
01/14/2021, 10:51 PMI think the ground truth should be blank
u
user
01/14/2021, 10:52 PMalpha/certified are modifiers we publish to give context imo
u
user
01/14/2021, 10:53 PMsorry by ground truth
u
user
01/14/2021, 10:53 PMI meant default value not ground truth
u
user
01/14/2021, 10:53 PMI mean how/where the status of connectors is managed
u
user
01/14/2021, 10:53 PMright
u
user
01/14/2021, 10:53 PMLike should we add it to SourceDefinitoin and read it from there
u
user
01/14/2021, 10:53 PMand `master`’s config files are used to determine certification
u
user
01/14/2021, 10:54 PMThat sounds good actually.
u
user
01/14/2021, 10:54 PMCertainly better than having an endpoint to manage this stuff
u
user
01/14/2021, 10:54 PMactually, there’s one problem with it
u
user
01/14/2021, 10:55 PMif you update from an alpha to a certified version, it won’t update your instance
u
user
01/14/2021, 10:55 PMhmm
u
user
01/14/2021, 10:55 PMit’s probably tied to a specific version though
u
user
01/14/2021, 10:56 PMoh but if you update it’s only through docker?
u
user
01/14/2021, 10:56 PMnot the config files on master
u
user
01/14/2021, 10:56 PMI don’t think it makes sense to put it in the spec
u
user
01/14/2021, 10:56 PMsince we’re the only people that can make it airbyte certified
u
user
01/14/2021, 10:56 PMI guess I’ll just add a separate file where we’ll track this.
u
user
01/14/2021, 10:57 PMOr we could remove the certification from badging and purely handle it in documentation
u
user
01/14/2021, 10:57 PMAnd manually label on the dashboard doc page
u
user
01/14/2021, 10:57 PMThat might be the easiest at first
u
user
01/14/2021, 10:58 PMLeast lockin too
u
user
01/14/2021, 11:00 PMwhy not in the spec?
u
user
01/14/2021, 11:04 PM^because we don’t control specs of connectors out in the wild
u
user
01/14/2021, 11:05 PMyeah
u
user
01/14/2021, 11:05 PMif someone uses a 3rd party docker image it should never say it’s airbyte certified within our UI
u
user
01/14/2021, 11:07 PMhuh. fair.
u
user
01/14/2021, 11:07 PMso i guess we are saying we would want to be able to certify connectors we don't control?
u
user
01/14/2021, 11:07 PMmm no
u
user
01/14/2021, 11:07 PMI think we’re just saying that we want to control certification.
u
user
01/14/2021, 11:08 PMI suppose we could certify external ones if we wanted but that definitely isn’t a priority
u
user
01/14/2021, 11:08 PMit feels like we need a new “table” in our configs then
u
user
01/14/2021, 11:08 PMlinking version to certification status
u
user
01/14/2021, 11:11 PMYeah
u
user
01/14/2021, 11:11 PMWe can also use a json file in master for the UI too. It could pull that as an “api” to get the current list
u
user
01/14/2021, 11:11 PMat least to tide us over
u
user
01/14/2021, 11:11 PMI’d like to limit table creation as much as possible so we don’t have to do db migrations
u
user
01/14/2021, 11:12 PMI imagine our tiers/certification will change over time
u
user
01/14/2021, 11:12 PMI think for now I’ll just remove the certification from badging and purely handle it in documentation
u
user
01/14/2021, 11:12 PMtu
u
user
01/14/2021, 11:12 PMAnd we can add that on later when it gets solidified more
u
user
01/16/2021, 12:40 AMto follow up
4 Views