Details
- Reviewers
carlavilla debdrup pauamma_gundo.com grahamperrin - Group Reviewers
docs - Commits
- R9:6b006b34bf9b: status/report-2022-01-2022-03: Add report
Diff Detail
- Repository
- R9 FreeBSD doc repository
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
If you have any comment regarding the order of reports, please also check this GitHub pull request:
https://github.com/freebsd/freebsd-quarterly/pull/471
That is the file that establishes what the order is.
The report has been generated using the new automated tools, that I will soon commit to the GitHub repository. They are still at the first version: most probably, I am going to add new features to them.
I've made some suggestions for where various entries belong, but that's about it I think.
Also, to make it clear to everyone: @salvadore has blanket permission from me to commit things related to quarterly reports in the doc/website/content/en/status for these reports, as well as making changes to doc/website/data/en/news/news.toml which is needed to make announcements appear on the front-page.
website/content/en/status/report-2022-01-2022-03/_index.adoc | ||
---|---|---|
66 | Having seen several talks from @cperciva on this topic, I think it's fair to say that boot performance belongs thoroughly in the projects category above, since it spans both the boot loader, kernel, and userland. :) | |
127 | We have a category for third-party projects, that this seems to belong in, more-so than the ports category - that's at least how I read the entry, about it being primarily a third-party project that just so happens to be in ports. | |
140 | Entries from helloSystem have in prior reports been in the third-party category as well, so I'd like to see them moved there. |
website/content/en/status/_index.adoc | ||
---|---|---|
16 | Realistically: I can't see that single-paragraph summaries are a norm. The preceding paragraph relates to the operating system, for which there is a single repository. This paragraph relates to development within and beyond multiple repositories. | |
website/content/en/status/report-2022-01-2022-03/clusteradm.adoc | ||
28 | Would the non-mirror URL be better? http://ftp-archive.freebsd.org/pub/FreeBSD-Archive/old-docs/ | |
website/content/en/status/report-2022-01-2022-03/doceng.adoc | ||
30 | Is this intentionally a duplicate of the point that appears in clusteradm.adoc above? | |
website/content/en/status/report-2022-01-2022-03/freebsd-foundation.adoc | ||
36 | ||
152–153 | Open Source Voices Episode 29: Deb Goodkin - Executive Director of The FreeBSD Foundation | |
website/content/en/status/report-2022-01-2022-03/gunion.adoc | ||
2 | Instead of Facility: Utility? (All subsequent descriptions of gunion(8) are of a utility.) | |
website/content/en/status/report-2022-01-2022-03/hellosystem.adoc | ||
25 | This is effectively a duplicate of the second bullet point. | |
website/content/en/status/report-2022-01-2022-03/iwlwifi.adoc | ||
7 | Uppercase .ORG is non-conventional. | |
24 | ||
website/content/en/status/report-2022-01-2022-03/ocf-wg.adoc | ||
4 | Who? https://www.freebsd.org/status/report-2021-10-2021-12/#_kernel_crypto_changes_to_support_wireguard was:
| |
website/content/en/status/report-2022-01-2022-03/portconfig.adoc | ||
24 | Presumably written by Alfonso, one of the two named contacts. | |
website/content/en/status/report-2022-01-2022-03/pot.adoc | ||
2 | ||
website/content/en/status/report-2022-01-2022-03/releng.adoc | ||
5 | ||
website/content/en/status/report-2022-01-2022-03/rtw88.adoc | ||
8 | Uppercase .ORG is non-conventional. | |
13 |
I'm undecided about the because-it's-late updates (changes in 13.1 status, etc.).
website/content/en/status/_index.adoc | ||
---|---|---|
16 |
I can. Notice it says "includes", not "limits itself to".
While on that topic, though, I'd use less restrictive language than "developers". The explicit list "developers, documenters, and ports maintainers" feels awkward, so maybe "regular contributors"? ("Regular" because occasional contributors would be less likely to submit a report or need to, I think.) | |
website/content/en/status/report-2022-01-2022-03/gunion.adoc | ||
2 | I'm not sure. I'm tempted, if they need to be the same, to use "facility" everywhere, since there's more to it than the administration utility. This is really about the geom class aw a whole. But considering this change should really be cleared with the author, I would ignore it to save a roundtrip. | |
website/content/en/status/report-2022-01-2022-03/hellosystem.adoc | ||
25 |
| |
website/content/en/status/report-2022-01-2022-03/iwlwifi.adoc | ||
7 | A recent email to me used
So while I can sort of see s/ORG/org/, I'd let the rest be. | |
website/content/en/status/report-2022-01-2022-03/ocf-wg.adoc | ||
4 | Good catch. And yes, jhb@. See https://github.com/freebsd/freebsd-quarterly/commit/083338351060b0cba28b33cf01b274b7bdda2f8d . | |
website/content/en/status/report-2022-01-2022-03/rtw88.adoc | ||
8 | As above, I'd use FreeBSD.org. |
Thank you very much to all of you for your feedback. I should be able to address all of your comments in the next 48 hours.
@debdrup: I have updated the GitHub pull request with your comments. As soon as I have fixed the other issues too this review will also be updated.
website/content/en/status/report-2022-01-2022-03/_index.adoc | ||
---|---|---|
140 | I wanted to put helloSystem in third-party category too, but I got confused and put it in the wrong one. The same issue applies to pot.adoc below: in https://www.freebsd.org/status/report-2021-10-2021-12/ the report was in the third-party project and not in miscellaneous. I have moved that too. |
I am going to try to leave the original text as it is to reflect what the state of the projects was in 2022q1. But since at the same time it is true that we are late and things have changed, I am also adding notes from the status reports team.
For example,
The upcoming 0.8.0 release will be based on FreeBSD 13.1-RELEASE once it is available.
would become
The upcoming 0.8.0 release will be based on FreeBSD 13.1-RELEASE once it is available [note from status report: link:https://www.freebsd.org/news/newsflash/#2022-05-12:1[13.1-RELEASE has now been released]]
Let me know what do you think of this solution.
website/content/en/status/_index.adoc | ||
---|---|---|
16 | Since there is no immediate agreement on this fix, I suggest we deal with it in another review and focus now on the 2022q1 quarterly report. Personally, I also interpret the actual text as implying that reports are one paragraph only, so, although other interpretations are possible, I think it can be improved so that it becomes unambiguous. As for the "developers" word being too restrictive, I would go with "developers and contributors" or just "contributors". Indeed, those are the words that are used in the FreeBSD contributors list, which include developers: @grahamperrin_gmail.com: Since you spotted the issue, I think it would make sense that you create the new revision. May I ask you to do it please? | |
website/content/en/status/report-2022-01-2022-03/clusteradm.adoc | ||
28 | I think the mirror has been chosen on purpose. I think this should help some users to access the site more smoothly. | |
website/content/en/status/report-2022-01-2022-03/doceng.adoc | ||
30 | It seems it is. I think it makes sense:
| |
website/content/en/status/report-2022-01-2022-03/gunion.adoc | ||
2 | I am not sure either. My interpretation is that a new facility called "gunion" has been added and that you interact with it through an utility also called "gunion". As I am not a native English speaker, I don't feel very confident with this interpretation however and in general I would simply trust that the author choose his words with care. But since in this case the author is on phabricator, it is easy enough to ask him. Moreover, the editing of the report still requires some time, so asking him should not slow things down much. @mckusick: Do you confirm that the words "facility" in your quarterly status report title and "utility" in the text are the terms you want to use or do you prefer to change something? | |
website/content/en/status/report-2022-01-2022-03/portconfig.adoc | ||
24 | You must be right, he also submitted the pull request: |
Let me know if this helps clarify gunion.
website/content/en/status/report-2022-01-2022-03/gunion.adoc | ||
---|---|---|
2 | gunion provides a new capability to GEOM. Much like raid, or stripping, or mirroring. I propose that all instances of "gunion(8) utility" be changed to "gunion facility" Here is my proposed new description: A New GEOM Facility, gunionContact: Marshall Kirk McKusick <mckusick@mckusick.com> The gunion facility is used to track changes to a read-only disk on a writable disk. The gunion facility can be especially useful if you have a large disk with a corrupted filesystem that you are unsure of how to repair. Another use of the gunion facility is to try out upgrades to your system. The gunion(8) utility is used to create and manage an instance of a gunion. Further details and usage examples can be found in the gunion(8) manual page. Netflix sponsored the development of gunion. |
@mckusick: Thank you very much for your quick response. I have updated the report with your proposed new description, which is clear for me.
@grahamperrin, @debdrup, @pauamma_gundo.com: Let me know if you have any more comments. Once we agree on the final version, I will commit the new quarterly status report and also push the edits we are discussing in the GitHub repository, which I am already committing locally as the process reviewing goes forward.
I don't think there's anything else I remarked on that needs to be addressed, so if everyone else would accept it if theirs has been dealt with, we can get the needful done.
With the nits above fixed, LGTM. Thanks for the IRC ping.
website/content/en/status/report-2022-01-2022-03/freebsd-foundation.adoc | ||
---|---|---|
152 | *donning unofficial accessibility advocate hat briefly* | |
website/content/en/status/report-2022-01-2022-03/gunion.adoc | ||
24 | IIRC, the usual wording (see end of report-sample.adoc) is
| |
website/content/en/status/report-2022-01-2022-03/hellosystem.adoc | ||
25 | Uh. Forgot what I meant to write here. |
Report has finally been committed, with the last edit from Pau Amma. Thanks everyone for your work.
@debdrup: In the commit message I wrote "Approved by: debdrup (status blanket)". Since you actually approved explicitely, this is a mistake, I should not have used the blanket: sorry about that. On the other hand, at least the status blanket approval has been seen for the first time, so maybe it is not that bad.