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

Jump to bottom

x/build/maintner: GitHub issue becoming 404 (e.g., due to being deleted) is not reflected in model #30184

Open
dmitshur opened this issue Feb 12, 2019 · 2 comments
Open

x/build/maintner: GitHub issue becoming 404 (e.g., due to being deleted) is not reflected in model #30184

dmitshur opened this issue Feb 12, 2019 · 2 comments
Labels
Builders x/build issues (builders, bots, dashboards) NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one.
Milestone
Unreleased

Comments

@dmitshur
Copy link
Contributor

dmitshur commented Feb 12, 2019 •
edited

GitHub issue #23772 is currently 404.

If it's possible to do so in the scope of the work that maintner already does (i.e., it doesn't require going out of its way), then it should detect that and set its NotExist field gets to true.

c, err := godata.Get(context.Background())
if err != nil {
	panic(err)
}
i := c.GitHub().Repo("golang", "go").Issue(23772)

fmt.Println(i.NotExist)

// Output: false (but should be true)

/cc @bradfitz @jmdobry

(This came up from #30182.)
@gopherbot gopherbot added this to the Unreleased milestone Feb 12, 2019
@gopherbot gopherbot added the Builders x/build issues (builders, bots, dashboards) label Feb 12, 2019
This was referenced Feb 12, 2019
Open
Closed
@gopherbot
Copy link

Change https://golang.org/cl/161906 mentions this issue: cmd/gopherbot: handle 404 GitHub issues in freezeOldIssues task

@dmitshur dmitshur added the NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. label Feb 12, 2019
gopherbot pushed a commit to golang/build that referenced this issue Feb 12, 2019
@dmitshur
cmd/gopherbot: handle 404 GitHub issues in freezeOldIssues task
a528637 
A GitHub issue can become 404. Attempting to lock it will produce a
404 response from the GitHub API. Don't treat it as a fatal error
when it happens.

Add a check for the NotExist field. This will help after golang/go#30184
is resolved.

Updates golang/go#30182

Change-Id: Ia04c59879909b1de00bd681606bfa331fe642cd4
Reviewed-on: https://go-review.googlesource.com/c/161906
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
@bobvawter bobvawter mentioned this issue Jun 28, 2019
Closed
@jmdobry jmdobry mentioned this issue Feb 22, 2020
Open
@dmitshur dmitshur changed the title x/build/maintner: GitHub issue becoming 404 is not reflected in model x/build/maintner: GitHub issue becoming 404 (e.g., due to being deleted) is not reflected in model Oct 11, 2022
@gopherbot
Copy link

Change https://go.dev/cl/442156 mentions this issue: cmd/gopherbot: skip more deleted issues in addGitHubComment

gopherbot pushed a commit to golang/build that referenced this issue Oct 14, 2022
@dmitshur @gopherbot
cmd/gopherbot: skip more deleted issues in addGitHubComment
1aa9c6a 
The deleted issue 55403 wasn't being detected as deleted and causing
errors. Add a little logic to handle this category of deleted issues.

For golang/go#28320.
Updates golang/go#30184.

Change-Id: I37f06a1330b6993e5526ea196074102c96fe98f9
Reviewed-on: https://go-review.googlesource.com/c/build/+/442156
Run-TryBot: Dmitri Shuralyov <dmitshur@golang.org>
Reviewed-by: Carlos Amedee <carlos@golang.org>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Dmitri Shuralyov <dmitshur@google.com>
Auto-Submit: Dmitri Shuralyov <dmitshur@golang.org>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Builders x/build issues (builders, bots, dashboards) NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one.
Projects
None yet
Milestone
Unreleased
Development

No branches or pull requests
2 participants
@dmitshur @gopherbot