incus icon indicating copy to clipboard operation
incus copied to clipboard

Port `network_forwards` to the database generator

Open stgraber opened this issue 9 months ago • 3 comments

Incus uses a built-in code generator for most of its database interactions. Unfortunately not all entities have yet been moved over to that mechanism, causing us to have far more SQL code in our code tree than we'd really like.

The goal here is to define new code generated functions in internal/server/db/cluster/ and then remove the old implementation from internal/server/db/ transitioning everything over to the new generated functions.

A recent example of the database generator being used can be found here: https://github.com/lxc/incus/pull/655

Note that some of the syntax has since evolved, so you may want to look at the current version of networks_integrations.go to see how to invoke the generator.

stgraber avatar Mar 17 '25 20:03 stgraber

Good afternoon, My friend @tonyn10 and I are taking Professor Chidambaram's Virtualization Class at UT Austin and were interested in tackling this issue for our class project. Would we be able to get this issue assigned to us?

arojas2003 avatar Apr 03 '25 19:04 arojas2003

Hi, I'm the other person who's also interested on being assigned for this issue.

tonyn10 avatar Apr 03 '25 19:04 tonyn10

Done!

stgraber avatar Apr 03 '25 20:04 stgraber