-
Notifications
You must be signed in to change notification settings - Fork 71
Outreach NDSU 2023Q1 meetings
Notes:
- All dates should be in ISO 8601 format (YYYY-MM-DD).
- See Outreach-meeting-template for the meeting template.
Note: Last meeting! :(
- Round robin
Person | Update | Blockers |
---|---|---|
Alec & Jenny | ||
Chris & Max | ||
Gabe | ||
Treva | ||
James | Out this week |
Note: penultimate meeting! :(
- Round robin
Person | Update | Blockers |
---|---|---|
Alec & Jenny | ||
Chris & Max | ||
Gabe | Sent apologies as has a dental appointment. | |
Treva | ||
James | Out this week |
-
Alec and Jenny need to update their
kata-ctl iptables
PR and get it building withcargo
. We recreated their feature branch as it was conflicted and updated the PR branch which now has a single commit as expected. -
Chris and Max are working through the review feedback for their
check-markdown
PR. -
Gabe's
log-parser
PR needs to be run throughrustfmt
to fix a lint issue.
- Round robin
Person | Update | Blockers |
---|---|---|
Alec & Jenny | ||
Chris & Max | ||
Gabe | ||
Treva | ||
James | Out this week |
-
James has approved Gabe's PR, so it only needs one more "ack" to land.
-
James has re-reviewed Alec and Jenny's iptables PR and we discussed some of the next steps briefly.
-
James has re-reviewed Chris and Max's markdown checker PR and made a list of suggestions on the issue.
- James to send out the link to a clap derive test program: https://github.com/jamesodhunt/test-clap-derive-cli-parsing
- Round robin
Person | Update | Blockers |
---|---|---|
Alec & Jenny | Raised two iptables command draft PRs | |
Chris & Max | Lots of updates to the rust markdown parser | |
Gabe | Absent today | |
Treva | ||
James |
-
We talked through the changes that Alec and Jenny need to perform on their draft PR.
-
Chris gave a demo of his and Max's rust markdown parser showing that it extracts headings and links, performs some checks and also dump some statistics.
We also turned it into a draft PR in the meeting.
-
We also talked about the fact that Kata builds rust code by setting
RUSTFLAGS=--deny warnings
, which makes even unused variables to cause a compile fail. To work around it, either runcargo build
iniitially (rather thanmake
(which sets the deny flag)), or comment out / remove the--deny warnings
in theMakefile
temporarily.Related to this, James mentioned that you can also "annotate" variables/functions with the magic "allow" line below to disable the compiler warnings:
// XXX: The line below stops the compiler complaining. #[allow(dead_code)] let an_unused_variable = "foo";
However, when you are ready to raise a PR, you'll have to clean up all the warnings to keep the CI and the Kata reviewers happy! 😄
-
We also talked about
make
and it's somewhat obtuse syntax, plus why Kata createsMakefile
's for all the tools and components.
-
James to review Chris and Max's rust markdown parser PR.
-
Alec and Jenny to go through James' review feedback to tweak their branch.
-
Gabe to update his log parser PR based on review feedback.
- Round robin
Person | Update | Blockers |
---|---|---|
Alec & Jenny | ||
Chris & Max | ||
Gabe | ||
Treva | ||
James |
- Max absent as unwell today.
- Jenny absent as away on drill.
- Alec is having problems with rust lifetimes and is planning to share his code with James to see if he can help.
- Chris is making progress.
- Gabe's PR is mostly done. He just needs to update the doc and write some more unit tests. At that point, the PR can be taken out of draft and can be reviewed by the community.
- Treva asked for us to give her hoodie sizes as she has some Kata swag!
- Round robin
Person | Update | Blockers |
---|---|---|
Alec & Jenny | Making progress with some help from Gabe. | |
Chris & Max | Looking at golang markdown tool for inspiration. | |
Gabe | More progress on draft rust log parser PR | |
Treva | ||
James |
-
We discussed PRs and the fact that you can raise a single PR and use that "continuously" for the draft work up until the final version after all the reviewer feedback has been applied: just
git push -f
and GitHub will update the PR for you. Related, once a PR is ready for a final review, just remove thewip
/rfc
labels or comments in the PR summary line. -
Chris absent as unwell.
-
Jenny let us know she will be out on Friday for drill.
- Round robin
Person | Update | Blockers |
---|---|---|
Alec & Jenny | Have implemented a lot of the code and added unit tests | |
Chris & Max | Looking at next stage of markdown checker | |
Gabe | Lots of progress | |
Treva | ||
James |
We talked through the next stages of what Chris and Max will need to do for the markdown checker. James has updated the issues with further details on this here:
We set up Chris' system by adding Chris's comrak
test program to
Chris' tests
clone, configuring a chris
remote (pointing to his
tests
fork, and
pushed all the files after ensuring cargo build
worked.
Gabe talked us through some of his latest changes to his rust log parser.
-
James needs to re-review Gabe's PR.
-
James to speak to Chelsea about the Alec & Jenny's latest mail on current direction of the
iptables
code.
- Updates on actions from previous meetings.
- Round robin
- Progress
- Problems
- Plan
- Discussions
- Presentations
Person | Update | Blockers |
---|---|---|
Alec | Started implementing kata-ctl iptables command |
|
Chris | Wrote rust tool using comrak crate to walk markdown doc nodes |
|
Gabe | Continuing to work on the rust log parser | |
Jenny | Absent for exams today. | |
Max | Starting to work on rust tool using markdown-rs crate |
|
Treva | Absent. | |
James |
-
Gabe kindly offered to have a meeting with the team to explain how to set up a multi developer git environment that avoids rebasing until the very last stage (to ensure each user can
git push
andgit pull
without causing conflicts due to rebase rewriting history). -
James gave a quick demo of
kata-runtime iptables
including proving that firewall rules applied by runningiptables -vL
inside VM using the debug console. Also mentioned that Alec and Jenny should enable full debug to help with testingkata-ctl iptables [get|set] ...
.
Note:
ℹ️ Gah! James away due to Easter public holiday days in the UK.
- James on holiday so no meeting?
Person | Update | Blockers |
---|---|---|
Alec | ||
Chris | ||
Gabe | ||
Jenny | ||
Max | ||
Treva | ||
James |
Note:
ℹ️ James away - back on 5th April! ℹ️
- Round robin
Person | Update | Blockers |
---|---|---|
Alec | ||
Chris | ||
Gabe | ||
Jenny | ||
Max | ||
Treva | ||
James | ℹ️ James away - back on 5th April |
Note:
ℹ️ James away this week. ℹ️
- Round robin
Person | Update | Blockers |
---|---|---|
Alec | ||
Chris | ||
Gabe | ||
Jenny | ||
Max | ||
Treva | ||
James | ℹ️ James away this week |
Note:
ℹ️ James away this week. ℹ️
- Round robin
Person | Update | Blockers |
---|---|---|
Alec | ||
Chris | ||
Gabe | ||
Jenny | ||
Max | ||
Treva | ||
James | ℹ️ James away this week |
- Round robin
Person | Update | Blockers |
---|---|---|
Alec | ||
Chris | ||
Gabe | ||
Jenny | ||
Max | ||
Treva | ||
James | Out this week |
- Round robin
Person | Update | Blockers |
---|---|---|
Alec | Starting to get familiar with the tasks for the kata-ctl iptables command
|
|
Chris | Considering working on the markdown checker rewrite | |
Gabe | Great progress on log parser rewrite | |
Jenny | Unable to attend today | |
Max | Considering working on improvements to the data collection script | |
Treva | Unable to attend due to AC meeting clash | |
James |
- Unfortunately, the timezone difference meant that the meeting clashed with the Kata AC meeting. However, next week the problem will resolve itself as the UK switches to BST/DST.
- Round robin.
- Chris went through some questions to help get feedback on the project status.
- James gave a five minute overview of Coco.
- James out all next week.
ℹ️ No meeting due to NDSU spring break. ℹ️
Person | Update | Blockers |
---|---|---|
Alec | ||
Chris | ||
Gabe | ||
Jenny | ||
Max | ||
Treva | ||
James |
- Round robin
Person | Update | Blockers |
---|---|---|
Alec | ||
Chris | ||
Gabe | ||
Jenny | ||
Max | ||
Treva | ||
James |
-
Round robin
-
ℹ️ approximate project midpoint [*] ℹ️
- Assess progress (and discuss contingencies if necessary).
-
James to do a demo of
agent-ctl
(andkata-ctl
/kata-runtime
if there is time): https://github.com/jamesodhunt/kata-and-tool-overview -
Post demo, let's discuss the following issues:
-
https://github.com/kata-containers/kata-containers/issues/5343 (
kata-ctl
, Alec) -
https://github.com/kata-containers/kata-containers/issues/6230 (
agent-ctl
, Jenny).
-
[*] - We can't discuss next week due to spring break.
Person | Update | Blockers |
---|---|---|
Alec | ||
Chris | ||
Gabe | ||
Jenny | ||
Max | ||
Treva | ||
James |
James gave presentation with demo which took rather longer than expected. Treva recorded it so Jenny can view later.
No meeting next week due to mid-term break.
- Round robin
Person | Update | Blockers |
---|---|---|
Alec | ||
Chris | ||
Gabe | ||
Jenny | ||
Max | ||
Treva | ||
James |
Alec, Chris, Jenny and Max are starting to investigate a rust issue to work on.
Gabe gave us a sneak preview of his rust log parser code and we tried to debug what was causing the dragonball failure. Gabe has now raised an issue so we can investigate this further:
We agreed that James would do a brief demo of agent-ctl
and
kata-ctl
/ kata-runtime
on Friday. This will help with the
familiarisation of these two tools, and hopefully stimulate interest
in some of the ideas for these tools:
Since Jenny probably won't be able to attend, Treva will try to record Friday's session.
- Round robin
Person | Update | Blockers |
---|---|---|
Alec | Landed first PR! | |
Chris | Landed first PR! | |
Gabe | Working on rust log parser rewrite | |
Jenny | Landed her first PR in the meeting! ;) | |
Max | Landed first PR! | |
Treva | ||
James |
- Round robin
- Discuss first PRs.
- Question: Does everyone now have access to a Linux system?
- If time, James to present on Kata unit testing in rust.
Person | Update | Blockers |
---|---|---|
Alec | ||
Chris | ||
Gabe | ||
Jenny | ||
Max | ||
Treva | ||
James |
We discussed some of the open PRs and why the CI was failing for them (and discussed the solution).
James talked over:
- https://github.com/jamesodhunt/rust-unit-tests-example
- https://github.com/kata-containers/kata-containers/blob/main/docs/presentations/unit-testing/kata-containers-unit-testing.md
- Round robin
- Discuss best day+time for a 2nd weekly meeting.
- If time, James to present on Kata unit testing in rust.
Person | Update | Blockers |
---|---|---|
Alec | Raised his first PR. | |
Chris | Getting familiar with Linux and ssh. | |
Gabe | Continuing work on rust log parser. Rust crate familiarisation. | |
Jenny | Absent today. | |
Max | Getting back to speed after being out sick. | |
Treva | Unable to attend today. | |
James |
Still waiting for real hardware to arrive. Some students now using a cloud instance for now.
Discussed having a second weekly meeting. The best time seems to be immediately after the Kata Architecture Committee meeting.
Alec offered to help Chris with Linux command line familiarisation.
- James to ask Treva to setup a second weekly meeting on Tuesday immediately after the Kata AC meeting (10:00 CST / 16:00 UTC).
- Round robin
- Discuss:
- Potential work items
- "good first issues" - raise that "one-liner" PR! ;)
- OpenStack demo from Treva.
- If time, James to present on Kata unit testing in rust.
Person | Update | Blockers |
---|---|---|
Alec | Linux system setup issues. Starting to look at kata-manager.sh
|
|
Chris | Out (travelling?) | |
Gabe | Raised two PRs this week! | |
Jenny | Still needs h/w. Continuing to learn rust. | |
Max | Linux system setup issues. Couldn't attend meeting as unwell. | |
Treva | Chasing h/w for Jenny. Set up OpenStack system and demo'ed it to us. | |
James | Reviewed Gabe's PRs. |
- Round robin
- Discuss:
- Potential work items
- "good first issues" - raise that "one-liner" PR! ;)
- If time, James to present on Kata unit testing in rust.
Person | Update | Blockers |
---|---|---|
Alec | Installing Kata using kata-manager . |
|
Chris | ||
Gabe | Installing Kata using kata-manager . |
|
Jenny | Going to try to borrow some hardware. | |
Max | working on dual boot Linux setup. Needs Sata cables. | |
Treva | Going to send some loaned hardware to Jenny. | |
James | Wrote up work items list. |
-
James to write up wiki page on follow-on tasks after installing Kata and share with team.
-
Team to look at the work items list and try to find one or more that look interesting to them.
- Ask questions on the wiki page or in the linked issues to get more info.
- General status updates (learning rust).
- Review of Capstone project Initiation doc.
- Problems obtaining access to a usable Linux environment.
- General status updates (learning rust).
- Discussion around Capstone project Initiation doc.
Kick off meeting.
- Students introduced themselves.
- James explained how the outreach project will work and a little about Kata.