Hi Team I am planning to work on this <issue> Ple...
# dev
v
Hi Team I am planning to work on this issue Please tag me to this. Github vaibhavk1992 And want to understand more around the issue. Please help me with more details.
๐Ÿ‘€ 1
jumping lakefs 3
๐Ÿ™Œ 1
i
Hey @Vaibhav Kumar ๐Ÿ‘‹ Weโ€™re always happy to accept contributions! Letโ€™s continue the discussion on the issue itself
v
done
๐Ÿ™ 1
@Itai Admi Have posted some question on the github. Kindly answer
๐Ÿ‘ 1
Do I have to create a new provider package in this issue(A new package all together) or I have to include it in lakfe-provider-package code?
i
I'm pretty sure it belongs to the lakeFS provider package. Does the Airflow docs suggest otherwise? I can check it again tomorrow
v
No ,it doesnt looks in the airflow side But just confused whether i need to create a new provider package or need to creates files within lakefs provider
@Itai Admi I am able to get a connection form with lakeFS type on airflow. Was this the only thing to be done in this issue?
image.png
i
That's amazing, great job! Please open a pr and add me as a reviewer
v
Will do it. Meanwhile please assign this issue on my github id.
๐Ÿ‘ 1
what could be the next issue to pick? This time I am looking for something in Lakefs repo. I have listed out some of them but not sure how much effort will be needed for each of them https://github.com/treeverse/lakeFS/issues/4908 https://github.com/treeverse/lakeFS/issues/2801 https://github.com/treeverse/lakeFS/issues/4535
i
I would suggest you filter by the
good-first-issue
label
v
yeah, Actually 2 and 3 are
good-first-issue
but I was curious as well if I can pick the first one related to terraform
Any update around the PR review?
i
I see that @Ariel Shaqed (Scolnicov) started reviewing, thank you!
Regarding the next items, the terraform provider issue is linked to our recent efforts around authorization & authentication. Since these 2 are key elements for any audit review, the team thought it would be best to not label it as
good-first-issue
. For the third item, I requested some product input, Iโ€™m unsure that we want to follow the suggested path described in the issue. The second issue would benefit the lakeFS project a lot. Our error messages in the HadoopFS can be improved a lot. Are you familiar with Hadoop/Spark or curious about learning more about it? If so, I think thatโ€™s the best option out of the 3 issues.
v
Yes, I am a data engineer by profession. I do understand Hadoop/Spark/Hive/Cloud. ๐Ÿ™‚
๐Ÿฆœ 1
jiggling lakefs 1
Hi @Ariel Shaqed (Scolnicov) /johnnyaug On the last PR raised I was trying to follow how can we test connection created. I was following this link. I am not sure how shall I use test_connection in my hook. Can you please share some example or direction how shall I use it my hook?
@Itai Admi @Ariel Shaqed (Scolnicov) I have re worked on the PR and tested all the things this time. it is working fine. Please verify it once.