-
Notifications
You must be signed in to change notification settings - Fork 1.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
plugins: expand PluginStatusV1
to include runtime status details
#42962
Conversation
Generally looks good. What will the semantics of status updates be? E.g. if a code path emits the old barebones status (just the error code), will it override existing detailed integration-specific fields? |
We have no way of controlling this. Since this runs in auth, auth itself is responsible for updating the status field. If it's not aware of this change, it will replace whatever is in the db |
This PR extends Teleport PluginStatusV1 to include details about the status of the runtime status of a plugin. These details include: - error message - last sync time - number of objects imported in gitlab Signed-off-by: Tiago Silva <tiago.silva@goteleport.com>
6275617
to
5dfacd8
Compare
…42962) * plugins: expand `PluginStatusV1` to include runtime status details This PR extends Teleport PluginStatusV1 to include details about the status of the runtime status of a plugin. These details include: - error message - last sync time - number of objects imported in gitlab Signed-off-by: Tiago Silva <tiago.silva@goteleport.com> * fix field refs --------- Signed-off-by: Tiago Silva <tiago.silva@goteleport.com>
…42962) * plugins: expand `PluginStatusV1` to include runtime status details This PR extends Teleport PluginStatusV1 to include details about the status of the runtime status of a plugin. These details include: - error message - last sync time - number of objects imported in gitlab Signed-off-by: Tiago Silva <tiago.silva@goteleport.com> * fix field refs --------- Signed-off-by: Tiago Silva <tiago.silva@goteleport.com>
…42962) * plugins: expand `PluginStatusV1` to include runtime status details This PR extends Teleport PluginStatusV1 to include details about the status of the runtime status of a plugin. These details include: - error message - last sync time - number of objects imported in gitlab Signed-off-by: Tiago Silva <tiago.silva@goteleport.com> * fix field refs --------- Signed-off-by: Tiago Silva <tiago.silva@goteleport.com>
…42962) (#43471) * plugins: expand `PluginStatusV1` to include runtime status details This PR extends Teleport PluginStatusV1 to include details about the status of the runtime status of a plugin. These details include: - error message - last sync time - number of objects imported in gitlab * fix field refs --------- Signed-off-by: Tiago Silva <tiago.silva@goteleport.com>
…42962) (#43470) * plugins: expand `PluginStatusV1` to include runtime status details This PR extends Teleport PluginStatusV1 to include details about the status of the runtime status of a plugin. These details include: - error message - last sync time - number of objects imported in gitlab * fix field refs --------- Signed-off-by: Tiago Silva <tiago.silva@goteleport.com>
…42962) (#43469) * plugins: expand `PluginStatusV1` to include runtime status details This PR extends Teleport PluginStatusV1 to include details about the status of the runtime status of a plugin. These details include: - error message - last sync time - number of objects imported in gitlab * fix field refs --------- Signed-off-by: Tiago Silva <tiago.silva@goteleport.com>
This PR extends Teleport PluginStatusV1 to include details about the status of the runtime status of a plugin. These details include: