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

add reinstall and unistall actions #47

Merged
merged 1 commit into from
Dec 24, 2021
Merged

add reinstall and unistall actions #47

merged 1 commit into from
Dec 24, 2021

Conversation

lfbdev
Copy link
Contributor

@lfbdev lfbdev commented Dec 21, 2021

Summary

feat:
1.Implement uninstall/reinstall interface for argocdapp plugin
2.Implement uninstall/reinstall interface for githubactions plugin

for a argocdapp:
Reinstall: kubectl delete + kubectl apply again
Uninstall: kubectl delete

for a githubaction:
Reinstall: DeleteFile + CreateFile from github repo
Uninstall: DeleteFile from github repo

Key Points

  • This is a breaking change.
  • Documentation (new or existing) is updated.

Related Issues

#7 #9

@lfbdev lfbdev added the enhancement New feature or request label Dec 21, 2021
@lfbdev lfbdev added this to the v0.1.0 milestone Dec 21, 2021
@daniel-hutao
Copy link
Member

I suggest you to briefly describe this pr, quoting the issue you implemented #9 instead of my pr #46

@IronCore864
Copy link
Member

IronCore864 commented Dec 22, 2021

@daniel-hutao I think it's OK if it's still a draft PR.

@lfbdev, don't forget to enrich the description of the PR once you are ready to make it as "ready for review".

@IronCore864
Copy link
Member

As a rule of thumb, we don't need to review draft PRs because they are highly likely to change.

@lfbdev lfbdev marked this pull request as ready for review December 22, 2021 13:34
@lfbdev
Copy link
Contributor Author

lfbdev commented Dec 22, 2021

@daniel-hutao I think it's OK if it's still a draft PR.

@lfbdev, don't forget to enrich the description of the PR once you are ready to make it as "ready for review".

OK, I have edited

Copy link
Member

@daniel-hutao daniel-hutao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's all from me.

cmd/argocdapp/main.go Outdated Show resolved Hide resolved
cmd/githubactions/main.go Outdated Show resolved Hide resolved
internal/pkg/argocdapp/reinstall.go Outdated Show resolved Hide resolved
internal/pkg/argocdapp/reinstall.go Outdated Show resolved Hide resolved
internal/pkg/argocdapp/reinstall.go Outdated Show resolved Hide resolved
internal/pkg/githubactions/reinstall.go Outdated Show resolved Hide resolved
internal/pkg/githubactions/uninstall.go Outdated Show resolved Hide resolved
internal/pkg/githubactions/uninstall.go Outdated Show resolved Hide resolved
internal/pkg/argocdapp/argocdapp.go Outdated Show resolved Hide resolved
internal/pkg/argocdapp/argocdapp.go Outdated Show resolved Hide resolved
@daniel-hutao
Copy link
Member

Some tips may be hidden by default. Pay attention for this.

@IronCore864
Copy link
Member

If all tests have been done, we can go ahead and merge this one now. @lfbdev @daniel-hutao

Copy link
Member

@daniel-hutao daniel-hutao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

some tips

internal/pkg/githubactions/githubactions.go Outdated Show resolved Hide resolved
deal with exception and spelling error
fix the review issues
@daniel-hutao
Copy link
Member

/lgtm
/approve

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
No open projects
v0.1.0
Awaiting triage
Development

Successfully merging this pull request may close these issues.

None yet

3 participants