Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Integrate with tagd #5420

Closed
nlowe opened this issue Jul 27, 2018 · 20 comments
Closed

Integrate with tagd #5420

nlowe opened this issue Jul 27, 2018 · 20 comments
Assignees
Labels
external_integration Track the work of integrations with external projects/products kind/requirement New feature or idea on top of harbor

Comments

@nlowe
Copy link
Contributor

nlowe commented Jul 27, 2018

Follow up from the Community Call on 2018-07-20.

We have open-sourced our tag cleanup utility (briefly talked about in #1168 and #5085), which is now available at https://github.com/HylandSoftware/Harbor.Tagd. We do not yet have approval from our legal department to publish images to the docker hub so for now it will need to be built manually, but I intend on correcting that as soon as I have approval (Follow HylandSoftware/Harbor.Tagd#1 for tracking).

Based on the community call we had last week, we should investigate if this project can be integrated with Harbor as an "external supporting system". I'd be glad to contribute on this front in any way I can in my free time.

/cc @steven-zou

@steven-zou steven-zou added the external_integration Track the work of integrations with external projects/products label Jul 31, 2018
@steven-zou
Copy link
Contributor

@nlowe
So glad to see the project is published! Let's work together to find a proper way to do the integration.

We can leverage this issue thread to discuss the possible way of the integration. Append comments directly at here. Is that ok?

@nlowe
Copy link
Contributor Author

nlowe commented Jul 31, 2018

Sounds good to me. I'm still waiting on an update from our legal department regarding pushing images to the Docker hub.

An initial "integration" could be something as simple as contributing to the harbor documentation explaining how to use tagd.

@steven-zou steven-zou added the kind/requirement New feature or idea on top of harbor label Aug 3, 2018
@steven-zou
Copy link
Contributor

@nlowe

What I'm thinking is totally same as your's, create a useful guideline to tell the user how to use it and link the Tagd project in the readme file of Harbor.

Then, the next step, we can start to consider if rest APIs can be provided for the integration work. Also, there might be UI needed to let users manage and edit the clearing policy. That would be a deep feature integration.

@nlowe
Copy link
Contributor Author

nlowe commented Aug 8, 2018

Would you like to see guidelines now, with instructions on how to build from source? Or should I hold off on documentation until we get approval to push tagd to the docker hub?

@nlowe
Copy link
Contributor Author

nlowe commented Aug 8, 2018

Scratch that, I should be able to get something written up by the end of the week, next week at the latest. We just got approval to push to the docker hub a few minutes ago, we're getting everything setup over the next few days.

@nlowe
Copy link
Contributor Author

nlowe commented Aug 9, 2018

https://hub.docker.com/r/hylandsoftware/tagd/ is now live! I'll work on getting something written up and a PR sent in for documentation.

@steven-zou
Copy link
Contributor

@nlowe

I saw the docker image in the hub. It's really awesome! Once your guide document is ready, I think the user can start to have a try. And then we can start the discussion about how to integrate it into the harbor platform.

@steven-zou
Copy link
Contributor

@nlowe

So far, we treat tagd as an external tool for harbor users, I don't think we need to put the tagd document into harbor repo. You can keep it in the tagd repo. We can add some contents in the readme of harbor project to navigate the user to the tagd repo. What do you think?

@nlowe
Copy link
Contributor Author

nlowe commented Aug 16, 2018

I see, that makes sense to me!

@steven-zou
Copy link
Contributor

@nlowe

Your tool is onboarding in Harbor community now, please check https://github.com/goharbor/harbor#additional-tools !

Do you have any idea about what's next step of this project?

@nlowe
Copy link
Contributor Author

nlowe commented Aug 24, 2018

@steven-zou thanks! I imagine the next step should be looking into what it would take to make tagd a native component of harbor. Is there an intermediate step you would like to see first?

@steven-zou
Copy link
Contributor

@nlowe
Actually, that's totally what I'm thinking. How to make it as a harbor native component. It's really a valuable feature and required a lot by the community. We can head to the step of reimplement it with golang as a native component.

What do you think?

@nlowe
Copy link
Contributor Author

nlowe commented Aug 27, 2018

@steven-zou Makes sense to me! Do you want to see the conversion as an out-of-tree project, and then figure out the best way to integrate it after that? Or shall I fork harbor and do the conversion directly in-tree?

I'm leaning towards doing this out-of-tree initially. I think it may be cleaner to work out the kinks initially in this manner.

@steven-zou
Copy link
Contributor

@nlowe

To let the community better understanding the conversion solution, I suggest you raise a new proposal to describe the native solution and then review and discuss (hope we can get some valuable feedbacks as many community users has mention this function) it in the community meeting.

If the proposal is ready, for the development work, you can fork a repo and send PR back to harbor repo once the code is done.

Is that ok for u?

@nlowe
Copy link
Contributor Author

nlowe commented Sep 4, 2018

@steven-zou That works for me. I'll work on getting something drafted by the end of this week or next. Do you want me to submit the proposal as a new Github issue or a post on the Google Groups forum?

@steven-zou
Copy link
Contributor

@nlowe

Please raise a new one with the proposal type. Thanks!
ps: you can also post a message containing the proposal link to the harbor-dev channel for offline discussion. I'll also find a proper time slot for you in the incoming community conf call to introduce it and get feedback from the community.

@steven-zou
Copy link
Contributor

As @nlowe did the work, assign the owner back to @nlowe.

@nlowe
Copy link
Contributor Author

nlowe commented Sep 5, 2018

I'm developing the proposal here: nlowe/harbor-tag-retention-proposal. I'll open a new issue and drop discussion links in the relevant places when it's ready for review.

@steven-zou
Copy link
Contributor

steven-zou commented Sep 6, 2018

@nlowe

I quickly go through the proposal doc, it's really awesome. The issue you drafted can be used as a proposal example/template for other community contributors to refer.

@nlowe
Copy link
Contributor Author

nlowe commented Sep 12, 2018

I'm going to close this as the proposal has now been submitted (sending notifications to the relevant channels shortly). If the proposal is accepted I'll make new issues for implementation details.

@nlowe nlowe closed this as completed Sep 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
external_integration Track the work of integrations with external projects/products kind/requirement New feature or idea on top of harbor
Projects
None yet
Development

No branches or pull requests

2 participants