NetworkingDsc icon indicating copy to clipboard operation
NetworkingDsc copied to clipboard

[New Resource] QoS Policies

Open tysonjhayes opened this issue 8 years ago • 10 comments

Add QoS Policy to the list of resources.

tysonjhayes avatar Jan 04 '16 21:01 tysonjhayes

Got a couple of resources here.

  1. MSFT_xNetworkAdapterQos - Enables/Disables QoS on a network adapter.
  2. MSFT_xNetQosTrafficClass - Manage traffic classes creation and deletion, algorithm / priority management and so on.

but, the question I have is, should these be in xNetworking or go into a module of their own like NetworkQoSDsc?

rchaganti avatar Nov 14 '16 11:11 rchaganti

Hi @rchaganti - good question.

Can you forsee the need for any other Network QoS resources? If there is likely to be a whole load of them it might be better to keep them in a new Resource Module. But if these two are all there are likely to be if might be better to include them in xNetworking (one day to be NetworkingDsc I hope).

Ultimately it should be down to discoverability - e.g. where would you expect to find them if you were looking. Because the cmdlets that configure QoS are -Net cmdlets I'd first go looking for the resources in the Networking repo. But I guess if they are in the NetworkQoSDsc that is equally discoverable. But because xNetworking is fairly well known/used, people might go looking there first.

@tysonjhayes, @kwirkykat, @mbreakey3, @TravisEz13 ?

PlagueHO avatar Nov 15 '16 05:11 PlagueHO

What I am looking at is a larger collection that spans across three different modules.

NetQoS Get-NetQosPolicy New-NetQosPolicy Remove-NetQosPolicy Set-NetQosPolicy

NetAdapter Disable-NetAdapterQos Enable-NetAdapterQos Get-NetAdapterQos Set-NetAdapterQos

DcbQos Disable-NetQosFlowControl Enable-NetQosFlowControl Get-NetQosDcbxSetting Get-NetQosFlowControl Get-NetQosTrafficClass New-NetQosTrafficClass Remove-NetQosTrafficClass Set-NetQosDcbxSetting Set-NetQosFlowControl Set-NetQosTrafficClass Switch-NetQosDcbxSetting Switch-NetQosFlowControl Switch-NetQosTrafficClass

While these are physically separate, they are all used in a QoS Configuration for example in a S2D cluster. I have the MSFT_NetAdapterQoS and MSFT_NetQosTrafficClass in a xNetworking fork but wanted to see community thoughts on the grouping. I am working on MSFT_NetQoSDcbxSetting, MSFT_NetQosPolicy, and MSFT_NetQosFlowControl. So, it will be total 5 resources.

rchaganti avatar Nov 15 '16 11:11 rchaganti

@rchaganti - in that case, maybe a separate QoS resource module might be the way to go. But would be good to get some thoughts from the MSFT team.

PlagueHO avatar Nov 16 '16 06:11 PlagueHO

@rchaganti @PlagueHO Yeah looking at that list of functions, NetQos or NetworkQos should be its own resource module.

kwirkykat avatar Jan 20 '17 00:01 kwirkykat

Interested in using these..

BarnumD avatar Dec 01 '17 14:12 BarnumD

@rchaganti - did you end up implementing or releasing any NetQos DSC Resources?

PlagueHO avatar Dec 03 '17 08:12 PlagueHO

Yes. I implemented a bunch of these resources (NetQosDSC) and will publish my repo this week. I will need help in writing some tests though. I don't have the full coverage.

rchaganti avatar Dec 04 '17 00:12 rchaganti

Thanks @rchaganti - awesome! :smile: Could you link to them in this issue when they're published? I'd contribute some tests for you but I'm miles behind in the backlog for the other repos I maintain! :cry:

PlagueHO avatar Dec 04 '17 02:12 PlagueHO

Alright. Here it is: https://github.com/rchaganti/netqosdsc I added some basic examples and no tests at all. I am adding tests but cannot publish them until holidays are over.

rchaganti avatar Dec 18 '17 16:12 rchaganti