dentproject / dentOS

dentOS SwitchDev based NOS
Other
200 stars 59 forks source link

tc persistency #182

Closed taraschornyiplv closed 1 year ago

taraschornyiplv commented 1 year ago

Overview Support to save and restore tc rules configuration

Use Case allow to save and restore tc configuration after reboot

paulmenzel commented 1 year ago

How are tc rules currently configured?

taraschornyiplv commented 1 year ago

At this moment tc configuration is runtime only

paulmenzel commented 1 year ago

At this moment tc configuration is runtime only

Yes, I understood that.

So, currently, every user executes tc commands.

To make this persistent, all they have to do, is put these commands in a script, and create a systemd service executing the script? (Or write the tc commands directly as ExecStart= lines into the script.)

Is that, what you want?

paulmenzel commented 1 year ago

systemd.network also has some options to configure a subset

I think you should elaborate on the usecase.

taraschornyiplv commented 1 year ago

At this moment tc configuration is runtime only

Yes, I understood that.

So, currently, every user executes tc commands.

To make this persistent, all they have to do, is put these commands in a script, and create a systemd service executing the script? (Or write the tc commands directly as ExecStart= lines into the script.)

Is that, what you want?

no. the idea is to have behavior similar to iptables-save.

taraschornyiplv commented 1 year ago

systemd.network also has some options to configure a subset

I think you should elaborate on the usecase.

this looks interesting. however, I see it only supports tc-cake and not tc-flower

paulmenzel commented 1 year ago

no. the idea is to have behavior similar to iptables-save.

Is it already possible to dump the traffic control state, and just needs to be integrated into dentOS?

systemd.network also has some options to configure a subset I think you should elaborate on the usecase.

this looks interesting. however, I see it only supports tc-cake and not tc-flower

No idea. I guess support could be easily added to systemd though, it it’s clear what is wanted.

Again, please update the issue description with your use cases.

taraschornyiplv commented 1 year ago

there is a Petunia https://github.com/dentproject/petunia which implements this. @snobleaws @sonoble can we have more info about it. this feature request is created to have a track record about petunia

paulmenzel commented 1 year ago

What commit fixed the issue?