Day 29: A Pull Request Needs Two People, and Peering Needs Two Routes
Some things cannot be finished alone, by design. A change that only you have seen is not reviewed. A network connection that only works in one direction is not a connection. Day 29 was two versions of the same idea: both sides have to agree, and half of it is worse than none because half of it looks like it should work. One Git task, one AWS task. Open a pull request, get it reviewed and merged,…
Day 29 in the KodeKloud Engineer platform highlighted the importance of collaboration in software development. A single change required two people to review and merge it, and connecting networks required two routes. The tasks originated from the platform itself.
A pull request (PR) is not a Git feature; it is a workflow layer implemented by hosting platforms like Gitea, GitHub, and GitLab. A PR serves as a gate, not just a merge. It allows for a named reviewer, recorded approval, discussion, and a permanent record of the agreement between parties. In this task, the reviewer was another user who logged in to approve and merge the change.
When opening a PR, it is crucial to remember that base and compare get reversed. The base represents the destination code, while the compare represents the source code. Additionally, the reviewer must be a different user, as approving one's own PR cancels the mechanism's meaning.
Another lesson learned was about security group rules in AWS. A rule with an empty IpRanges field and populated UserIdGroupPairs indicates that the rule is security group-referenced, not CIDR-based. External traffic cannot access the instance under these conditions.
For VPC peering, a connection must be active, and routes must be created on both sides of the connection. The target security group must allow traffic from the source CIDR. Peering is bidirectional but not configured that way, and it is not transitive.
Written by urgent.news from Dev.to's reporting — not their text. Machine-written — may contain errors; check the original before relying on it.
