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

Deleted comment from GH makes sync to fail #156

Open
lgecse opened this issue Aug 7, 2023 · 3 comments
Open

Deleted comment from GH makes sync to fail #156

lgecse opened this issue Aug 7, 2023 · 3 comments
Assignees
Labels
bug Something isn't working
Milestone

Comments

@lgecse
Copy link
Contributor

lgecse commented Aug 7, 2023

Steps to reproduce:

  • Make a GH issue, have some comments on it
  • Run sync tool, should be successful
  • Go to GH, remove a comment from the synced issue
  • Run sync tool again, will fail
DEBU Updating Jira OSE-15 with GitHub #1           file="issue/issue.go:187"
DEBU Comparing GitHub issue #1 and Jira issue OSE-15  file="issue/issue.go:124"
DEBU Issues have any differences: false            file="issue/issue.go:172"
DEBU Jira issue OSE-15 is already up to date!      file="issue/issue.go:245"
DEBU Jira issue OSE-15 has 5 comments              file="comment/comment.go:77"
DEBU Retrieving GitHub user (rappizs)              file="github/github.go:115"
ERRO Update Jira comment did not return comment! Got: <nil>  file="jira/jira.go:457"
ERRO Error updating issue OSE-15. Error: comparing comments for issue OSE-15: updating Jira comment: update Jira comment fail

Edit: updated description, and title. GH comment removal makes the tool fail.

@lgecse lgecse modified the milestones: v.next, v0.1.0 Aug 7, 2023
@rappizs
Copy link
Contributor

rappizs commented Aug 7, 2023

We have two options for the solution, if a comment has been deleted on GitHub:

  • we can remove it from Jira as well
  • we can flag the comment in Jira somehow to indicate that it has been deleted on GitHub, but we don't actually remove the comment from Jira

Which solution would we prefer @justaugustus?

@lgecse lgecse changed the title Deleted comment from Jira makes sync to fail Deleted comment from GH makes sync to fail Aug 7, 2023
@lgecse
Copy link
Contributor Author

lgecse commented Aug 11, 2023

@rappizs Please go ahead with

we can flag the comment in Jira somehow to indicate that it has been deleted on GitHub, but we don't actually remove the comment from Jira

@lgecse lgecse added the bug Something isn't working label Aug 11, 2023
@rappizs
Copy link
Contributor

rappizs commented Aug 11, 2023

@lgecse I couldn't reproduce this issue. After deleting a comment on GH everything works fine, except the comment is still in Jira, but I can't see any errors.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants