-
Notifications
You must be signed in to change notification settings - Fork 367
Bug 1923923 - Allow creation or update (annotation) of an internal issue #8532
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
base: master
Are you sure you want to change the base?
Conversation
This is still a Work In Progress, the below screenshot shows the design I had in mind. |
In the current design, the user has to focus on two things:
|
adafa57
to
ea5686b
Compare
ea5686b
to
249a4f0
Compare
I'm actually stuck by an obscure JS error (that seems related to webpack build & react) that happens while submitting the form (when user is logged on treeherder):
@Archaeopteryx does that mean anything to you? |
Could this be the callback which currently adds the bug number returned by the bug filer to the pinboard? |
249a4f0
to
aa737b2
Compare
I recorded a video using the UI to create and reference an internal issue: https://github.com/user-attachments/assets/ce94f6b0-b2a8-4fba-a5d9-fc24e27cdd13.
Next (and last for this feature) work is to link a bugzilla ID to the internal issue once it gets creates, then update its attributes when Bugscache is automatically filled. |
Thank you for the video and the description. Does a third classification with a bug automatically reopen the bug filer? |
Actually it's the fourth classification that opens the bug filer (which is shown at the end of the video). About step 6, indeed it reuses the existing internal issue, then the UI also create a classification with the selected job. |
0c877ce
to
e0dfeb0
Compare
I rebased this branch, it is fine by me. I think I reproduced the bug I mentioned on thursday. It only happens with some special summary (e.g. |
Getting back to this tomorrow. |
Thank you. The changes have been deployed to prototype. |
I played a bit on prototype: https://prototype.treeherder.nonprod.cloudops.mozgcp.net/jobs?repo=autoland&revision=cc5967ffe3435273d130a6b4e992606e1948fe81&selectedTaskRun=aX1fdZkFQyuwljxSLA6e4Q.0 |
Remaining issues from testing the stack and PR 8595:
|
If I understand correctly, cache for suggestions happens at job level in treeherder: https://github.com/mozilla/treeherder/blob/master/treeherder/model/error_summary.py#L125. I see no summary based cache for the suggestions query. To my understanding already opened jobs should not show the internal issue, but a new job with a TextLogError matching the summary should be visible. I'm implementing the 3 other points. |
I implemented the suggestions, except for the last point (number of occurrences is reached via pinning). I'll work on it next week. I had trouble to log in from my local application (making UI devs more complex to test…). It was working before, now I have the above error at https://auth.mozilla.auth0.com: |
0960261
to
9dda37c
Compare
I rebased on top of master and finally managed to trigger bug suggestion from a pinned bug, through a custom event (when the suggestion is available). @aryx can you have a look once again please? |
No description provided.