Add AGENTS.md#570
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: fmount The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
fmount
force-pushed
the
agentsmd
branch
3 times, most recently
from
58e7f24 to
2a8ee4b
Compare
|
|
||
| - [lib-common](https://github.com/openstack-k8s-operators/lib-common): shared modules for conditions, endpoints, TLS, secrets, etc. | ||
| - [infra-operator](https://github.com/openstack-k8s-operators/infra-operator): topology APIs. | ||
| - [dev-docs/developer.md](https://github.com/openstack-k8s-operators/dev-docs/blob/main/developer.md): developer guide and coding conventions. |
There was a problem hiding this comment.
should we also mention neutron-operator here?
There was a problem hiding this comment.
yeah makes sense, let me update this and also mention pre-commit like we did in neutron-operator.
There was a problem hiding this comment.
But neutron-operator is dependent on the ovn-operator not the reverse so seems we don't need to add here, if we add then it will also require octavia-operator.
It is something like almost all operator depends on keystone-operator, and in keystone-operator all operators needs to be added as dependency which doesn't look so here checking keystone-operator similar PR.
There was a problem hiding this comment.
ah ok, yes, if the dependency is the reverse we don't need to overcomplicate. for deeper inspection I assume the LLM can simply go through go.mod and build the dependency graph, but here is mostly to give a direction about the key dependency we want to establish for the project.
So yes, let's not have other dependencies here other than what we added at the beginning, and then we can improve in a follow up in case we missed something.
| |------|---------| | ||
| | `OVNNorthd` | Manages the OVN Northd deployment (logical-to-physical flow translation). | | ||
| | `OVNDBCluster` | Manages OVN database clusters (northbound and southbound OVSDB). | | ||
| | `OVNController` | Manages the OVN Controller daemonset (per-node OVS flow management). | |
There was a problem hiding this comment.
here we should extend ovs and metric daemonsets as well
This patch adds AGENTS.md to the repo to ensure any AI agent can follow the conventions defined for this operator. At the same time, it allows to not waste time navigating the repo to find initial information and save tokens. Signed-off-by: Francesco Pantano <fpantano@redhat.com>
3 participants
This patch adds
AGENTS.mdto the repo to ensure anyAI agentcan follow the conventions defined for this operator. At the same time, it allows to not waste time navigating the repo to find initial information and save tokens.