Skip to content

Conversation

robshakir
Copy link
Member

 * (A) Dockerfile.aite
   - Dockerfile to build Aite. This implementation requires
     kernel-level privileges, which can be added through
     --privileged, but --cap-add NET_ADMIN may be sufficient (for
     Docker), requires additional examination.
 * (A) srv/srv.go
   - Initial server implementation of Aite.
 * (A) proto/aite
   - Protobuf gRPC API for Aite. This interface allows a test to
     interface with Aite.
 * (A) cmd/aite
   - Binary acting as the Aite main.

To add:

  • Some form of testing -- likely in the form of an e2e test that can
    validate functionality integrates with the kernel since there is
    limited package logic here.

 * (A) Dockerfile.aite
   - Dockerfile to build Aite. This implementation requires
     kernel-level privileges, which can be added through
     --privileged, but --cap-add NET_ADMIN may be sufficient (for
     Docker), requires additional examination.
 * (A) srv/srv.go
   - Initial server implementation of Aite.
 * (A) proto/aite
   - Protobuf gRPC API for Aite. This interface allows a test to
     interface with Aite.
 * (A) cmd/aite
   - Binary acting as the Aite main.
@robshakir
Copy link
Member Author

robshakir commented Nov 25, 2023

Revised API examples:

➜  ~ grpcurl -plaintext -d '{"name": "eth0", "params": {"state": "IS_UP", "latency_msec": 64,  "loss_pct": 10 }}' localhost:60061 openconfig.aite.Aite.SetInterface
{
  "name": "eth0",
  "params": {
    "state": "IS_UP",
    "latencyMsec": 64,
    "lossPct": 10
  }
}

➜  ~ grpcurl -plaintext -d '{"name": "eth0", "params": {"state": "IS_UP", "latency_msec": 0,  "loss_pct": 0 }}' localhost:60061 openconfig.aite.Aite.SetInterface
{
  "name": "eth0",
  "params": {
    "state": "IS_UP"
  }
}

@robshakir
Copy link
Member Author

Confirmed that we just need to use --cap-add NET_ADMIN rather than --privileged. I believe kind will give us more permissions than this, since we create interfaces within those pods in KNE, but to be validated.

@robshakir
Copy link
Member Author

Leaving open for @akrentsel and @alshabib to take a look before submitting.

@robshakir robshakir merged commit 310a912 into main Nov 27, 2023
@robshakir robshakir deleted the poc branch November 27, 2023 17:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants