this post was submitted on 15 May 2024
89 points (95.9% liked)

Programming

17340 readers
329 users here now

Welcome to the main community in programming.dev! Feel free to post anything relating to programming here!

Cross posting is strongly encouraged in the instance. If you feel your post or another person's post makes sense in another community cross post into it.

Hope you enjoy the instance!

Rules

Rules

  • Follow the programming.dev instance rules
  • Keep content related to programming in some way
  • If you're posting long videos try to add in some form of tldr for those who don't want to watch videos

Wormhole

Follow the wormhole through a path of communities [email protected]



founded 1 year ago
MODERATORS
 

I've heard it thrown around in professional circles and how everybody's doing it wrong, so.. who actually does use it?

For smaller teams

"scaled" trunk based development

top 50 comments
sorted by: hot top controversial new old
[–] [email protected] 39 points 5 months ago* (last edited 5 months ago) (1 children)

Second diagram, yes absolutely.

Short lived (1-2 day) branches, and a strong CI systems to catch regressions.

Be warned, the strength in the CI lies in its capacity to detect when some functionality that previously worked doesn't work anymore. So, the flow must be green always, and it must evolve as the features evolve. Without good CI you're destined for failure.

[–] [email protected] 5 points 5 months ago

lies in its* capacity

[–] [email protected] 27 points 5 months ago (3 children)

I do, on a 900+ developer mono repo. Works like a charm.

We just have a CD that allows to deliver each project each micro service individually.

[–] [email protected] 9 points 5 months ago (1 children)

Holy crap that's a lot of devs 😳

[–] [email protected] 5 points 5 months ago

Yeah, the biggest problem is keeping up to date.

That's where the mono repo really shines. We have a folder for common stuff that everyone depends upon. A modification is automatically applied/compatible with every micro service. Really streamline the lib updates problem ^^

[–] [email protected] 9 points 5 months ago (1 children)

You deliver your software on CDs?

[–] [email protected] 12 points 5 months ago (1 children)

Most likely CD is intended to mean continuous delivery, which commonly means automation in processes that deliver your software to it’s target audience.

[–] [email protected] 2 points 5 months ago

It does indeed ^^

[–] [email protected] 3 points 5 months ago (2 children)

Out of curiosity, how long are CI and CD runs? And are there any particularities in the way of working for example every PR/MR is created by pair programmers, or the use of josh to cut down on time to clone, stuff like that.

Anti Commercial-AI license

[–] [email protected] 4 points 5 months ago

If cloning a repo is an issue, you're using CI wrong. --shallow has it's purpose.

Anyway, in my project a complete CI run including local integration tests takes about an hour. We could cut that down by running things in parallel, but we never bothered to add more runners.

I would say, if your tests hold you back, you might want to reconsider testing. Staged testing is an option, or just reevaluate whether you really need all those test cases. Many integration tests are not really testing that much, because 95% of them overlap.

[–] [email protected] 1 points 5 months ago

No problem for the cloning.

A CI run is around 8 mins. We just use bazel to determine what to run and to keep cache of unchanged tests.

[–] [email protected] 19 points 5 months ago

I’ve been doing this for the past 10 years or so. When I joined my current company a few years ago, it was one of the first things I pushed for. It made cycle times go down drastically and value is being delivered to end users at a much higher rate now.

With enough tests and automation, there is almost no reason not do on the web. On embedded or mobile platforms this might be a bit more difficult, although not entirely impossible.

The use of feature toggles also greatly enhanced our trust in being able to turn a feature off again if it turned out to be faulty somehow. Although we usually opt for patching bugs, it gives the business as a whole more confidence.

[–] [email protected] 12 points 5 months ago (2 children)

Here there's main. You branch off. Do your work. Make a PR to main. Build passes and someone approves, merge to main. Production release is done by tagging main.

The branches are short lived because the units of work we select are small. You have like one pr for an endpoint. You don't wait until the entire feature with 20 endpoints is ready to merge.

Seems to work fine. I think this is different than trunk based development but honestly I'm not sure I understand trunk.

[–] [email protected] 1 points 5 months ago

This is enough for most projects.

load more comments (1 replies)
[–] [email protected] 10 points 5 months ago (4 children)

We do, for two 2-3 person projects, where no code reviews are done. This is mostly because (a) it's "just" a rewrite and (b) most new functionality is small and well-defined. For bigger features a local branch is checked out and then merged back later. Commits are always up-to-date, which makes it much easier to test integration of new featues.

load more comments (4 replies)
[–] [email protected] 10 points 5 months ago (1 children)

I haven't worked on any teams where all members committed "every 24 hours", and there have always been some branches that live longer than we'd like (usually just unfinished work that got deprioritized but still kept as an eventual "todo"), but most teams I've worked on have indeed followed the basic pattern of only one long-lived branch, reviews and CI required prior to merge, and all feature-branches being short-lived.

[–] [email protected] 15 points 5 months ago (1 children)

A hard timeline on commit strikes me as less than ideal.

People are people. They have issues, they screw up, but they still write good code.

Seems like a brutal metric that encourages minimal commits without real change.

load more comments (1 replies)
[–] [email protected] 9 points 5 months ago* (last edited 5 months ago)

Team of three. We do the feature branches, pull requests and code reviews. With the right culture, it keeps devs informed about the various projects and gives a nice space to discuss about the practice.

EDIT: we do automated linting and formatting checks too. Keeps things coherent.

[–] [email protected] 9 points 5 months ago* (last edited 5 months ago) (2 children)

Yes. We use SVN. I hate it. I'm trying to build a case to switch to git. We're a small team, but a growing team

[–] [email protected] 17 points 5 months ago

Welcome time traveler. If you're able to return to your own timeline, buy some Apple stock.

/joking

But I hope your team does migrate. SVN is pretty bad, compared to git.

[–] [email protected] 4 points 5 months ago* (last edited 5 months ago) (2 children)

Do you really need to make a case? Does your company not trust devs? Is there people leading that have no idea about technology? SVN is dead. Many devs won't touch it. It's best way to say to new candidates your company is backwards. Many would refuse to work in a company that uses a version control system that has been dead 7 years.

[–] [email protected] 3 points 5 months ago (2 children)

It's effort to switch, and we don't benefit from having separate copies of the repo bc we're so small. No one steps on eachother's toes, so distributed version control isn't necessary.

Now, the fact that most devs know git and SVN is dead is not lost on our CTO, but putting the effort to switch over doesn't provide direct value to the customer, so I have to make the case that switching to git would do enough from a productivity and maintenance standard to effect customers.

load more comments (2 replies)
[–] [email protected] 2 points 5 months ago (1 children)

My company still uses SVN, but we have almost 20 years of history in the repository, not including the autogenerated commits from when we migrated from CVS.

My department would like for us to move to git (some sub projects have) but it's important for our process to retain the history and nobody has had the time to figure out if the migration would be clean then update all of our auto-testing infrastructure (which itself is over a decade old) to use git, all while not stopping active development.

load more comments (1 replies)
[–] [email protected] 6 points 5 months ago

Team of one here tends to work on master only for very small stuff.

There's a branch for the next release which will get merged once everything's done. Occasionally there are smaller branches that fork off of the release branch and get merged back.

Meanwhile master is, most of the time, an old copy of the new release branch so merging goes without issue. Unless there's a problem in prod, then it gets fixed in master and backported to the feature branch.

I should use feature toggles more (usually #ifdef, sometimes if (config_Flag)), occasionally a big feature creeps in and i know management will change it at least 3 times and 2-3 new releases will come out in between...

[–] [email protected] 6 points 5 months ago

At my last company, we used the scaled TBD. For personal projects I do the same. It’s honestly really nice. Not having to worry about merging issues between a dev branch and main branch was probably the biggest benefit. The code base also felt more accessible to the team. Cherry-picking a particular commit that a teammate worked on that’s been merged but I needed on my feature/bug branch was also painless.

[–] [email protected] 5 points 5 months ago (2 children)

I’ve been a fan of git-flow for a long time. It makes the master consistently stable and production ready, gives mechanisms for hotfixing, patching, releasing, tagging, and regular feature dev with a running develop branch. This tends to be more stable than Wild West commits into dev direct, since you work on a feature in isolation, and then merge the feature in when it’s ready, and keeps prod in its own lane so there’s no risk of a feature accidentally nuking something.

[–] [email protected] 1 points 5 months ago (1 children)

Doesn't the "scaled trunk-based development" do that too, with feature branches and merge requests? Trunk is your production-ready branch there.

load more comments (1 replies)
[–] [email protected] 1 points 5 months ago

I'm another git-flow fan. Have not encountered a situation that would motivate me to change workflow. We use submodules all over the place, too.

[–] [email protected] 5 points 5 months ago

We've used the "scaled" approach for a whilen and we make tags to release. Every time we have a long-lived branch that isn't main/master something gets fucked up or we spend an extra few days just to reconcile the conflicts. Luckily we're service based so supporting multiple versions of a product is very rare.

[–] [email protected] 5 points 5 months ago

This actually seems like git flow, but easier while achieving the same objectives.

[–] [email protected] 5 points 5 months ago (1 children)

Only for repos where there are no code reviews or releases. E.g small single developer utilities, configuration repos, etc.

[–] [email protected] 1 points 5 months ago (1 children)

Were there at least tests (where possible) if there were no code reviews?

Anti Commercial-AI license

[–] [email protected] 1 points 5 months ago

In my experience if there are tests there are code reviews and branches.

[–] [email protected] 5 points 5 months ago

Google operates on a trunk model, according to this:

https://dev.to/tbodt/comment/1j4e

[–] [email protected] 4 points 5 months ago

Scaled trunk based all year long, every year, every team.

Life is too short to tolerate long-lived branches.

[–] [email protected] 3 points 5 months ago

We have hundred of individual repos and use git flow: short lived feature branches but also long lived develop, master and support branches (for LTS releases).

[–] [email protected] 3 points 5 months ago

Scaled trunk based development for sure, we're a two man team, I do backend the other guy does frontend, but still we use that.

[–] [email protected] 3 points 5 months ago (2 children)

Worked with both, both worked fine.

load more comments (2 replies)
[–] [email protected] 2 points 5 months ago* (last edited 5 months ago) (1 children)

We don't have release branches, the commit is just tagged as being currently deployed in production.

People merge their feature branches to master during working hours (by merging CI validated PRs) and release when they get a chance. Normally do about 10-20 releases per day.

[–] [email protected] 3 points 5 months ago

For those of us that develop products that are sold to customers (and run on-premise), you will end up with customers on various releases. It is imperative that we can fix bugs in a release without requiring the customers to upgrade versions (major.minor).

[–] [email protected] 2 points 5 months ago (3 children)

Not saying anything about good or bad, but trunk-based development doesn't work when the business requires you to have multiple releases under development concurrently.

[–] [email protected] 2 points 5 months ago (1 children)

I don't think anything works in that situation.

[–] [email protected] 2 points 5 months ago* (last edited 5 months ago)

In TBD, it's not a "release" until its production ready. The methodology and philosophy doesn't prevent you from developing multiple feature branches at once or even deploying a work in progress feature branch to a dev environment.

All TBD requires in that case is once the feature branch is production ready, it's merged to the trunk. You may need to add a feature toggle if there are multiple release like for different architectures. And you also might benefit from using git tags and deploying to production from a git tag instead of the most recent commit on a branch.

Exactly what you need to do is going to depend on the project's exact needs but TBD is totally possible in that example.

[–] [email protected] 2 points 5 months ago

Scaled absolutely. Works really well. We have a pr ci pipeline. And cd set up on the trunk. Everything is PRed before being merged to the trunk.

load more comments
view more: next ›