| opendevreview | Sofia Sarhiri proposed openstack/security-doc master: Migrate OSSN txt files to build pipeline https://review.opendev.org/c/openstack/security-doc/+/1000155 | 17:15 |
|---|---|---|
| opendevreview | Sofia Sarhiri proposed openstack/security-doc master: Migrate OSSN txt files to build pipeline https://review.opendev.org/c/openstack/security-doc/+/1000155 | 17:39 |
| opendevreview | Sofia Sarhiri proposed openstack/security-doc master: Migrate OSSN txt files to build pipeline https://review.opendev.org/c/openstack/security-doc/+/1000155 | 17:42 |
| opendevreview | Sofia Sarhiri proposed openstack/security-doc master: Migrate OSSN txt files to build pipeline https://review.opendev.org/c/openstack/security-doc/+/1000155 | 17:45 |
| opendevreview | Sofia Sarhiri proposed openstack/security-doc master: Migrate OSSN txt files to build pipeline https://review.opendev.org/c/openstack/security-doc/+/1000155 | 17:47 |
| opendevreview | Sofia Sarhiri proposed openstack/security-doc master: Migrate OSSN txt files to build pipeline https://review.opendev.org/c/openstack/security-doc/+/1000155 | 17:52 |
| opendevreview | Sofia Sarhiri proposed openstack/security-doc master: Migrate OSSN txt files to build pipeline https://review.opendev.org/c/openstack/security-doc/+/1000155 | 18:14 |
| JayF | fungi: can I steal your brain for a second? | 18:43 |
| JayF | fungi: at the intersection of gerrit weird behavior and this OSSN migration | 18:43 |
| JayF | tl;dr I have whitespace in the GERRIT copy of a change that isn't in the LOCAL copy of the change that git thinks is the same | 18:43 |
| opendevreview | Jay Faulkner proposed openstack/security-doc master: Migrate OSSN txt files to build pipeline https://review.opendev.org/c/openstack/security-doc/+/1000155 | 18:44 |
| JayF | fungi: nevermind, apparently I hadn't git-review'd (WTF) | 18:45 |
| JayF | me and sarhiri having shared hallucinations over here in zoom | 18:45 |
| JayF | fungi: gouthamr: rosmaita: Please make a final review of https://review.opendev.org/c/openstack/security-doc/+/1000155 -- I will merge and integate this into our workflow this week once it gets +2s | 18:45 |
| rosmaita | ack | 18:46 |
| fungi | JayF: if in doubt, `git status` ;) | 18:50 |
| JayF | git status doesn't show me I didn't git review :D | 19:20 |
| JayF | git status was telling me one thing, gerrit was telling me another. git-review fixed it all lol | 19:20 |
| gouthamr | JayF: ++ | 19:21 |
| JayF | just get +2s on it and I'll merge on a day I can observe and do the follow-ups | 19:21 |
| JayF | maybe tomorrow? IDK but not today for sure :D | 19:21 |
| * gouthamr can likely look before the VMT meeting today, if not, will be prio after | 19:22 | |
| JayF | we literally went down the review comments you/I made at PS4 and resolved them | 19:23 |
| JayF | so I think it's good | 19:23 |
| gouthamr | yeah, i'm overthinking.. let's get this in and if i find something, i can follow up | 19:24 |
| gouthamr | oh look at that, you got security-notes/OSSN-0108.yaml also :D | 19:24 |
| fungi | JayF: the other troubleshooting step i use is comparing `git show-ref` to the ref gerrit reports for the most recent patchset | 19:26 |
| fungi | that would have raised that your local commit was different from the last commit gerrit got pushed | 19:26 |
| opendevreview | Merged openstack/ossa master: Process clarifications on impact descriptions https://review.opendev.org/c/openstack/ossa/+/989537 | 22:09 |
| opendevreview | Merged openstack/ossa master: Retitle and clarify embargoed patch guidelines https://review.opendev.org/c/openstack/ossa/+/993654 | 22:10 |
| opendevreview | Merged openstack/ossa master: Add yamllint for advisories https://review.opendev.org/c/openstack/ossa/+/1001584 | 22:22 |
| opendevreview | Merged openstack/ossa master: Reporting vulnerabilities in other software https://review.opendev.org/c/openstack/ossa/+/977152 | 23:04 |
| *** mrunge_ is now known as mrunge | 23:25 | |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!