Skip to content

Rework pod logs navigation scheme

Adrien Kohlbecker requested to merge ak/logs-search into master

What does this MR do?

In preparation for the creation of a generic logs api (in #36342 (closed)), a number of changes are necessary:

  • Switch the logging route from /project/environment_id/pod_name/container_name to a generic /project/logs?environment_name=..&pod_name=..
  • Use the name of the environment rather than the id
  • Ensure the logs view works without page reloads (changing the environment does not require loading a new url)

A further MR will rework the query parameters to a generic filter[]=environment_name:foo&filter[]=... as this MR is already big enough.

Screenshots

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

If this MR contains changes to processing or storing of credentials or tokens, authorization and authentication methods and other items described in the security review guidelines:

  • Label as security and @ mention @gitlab-com/gl-security/appsec
  • The MR includes necessary changes to maintain consistency between UI, API, email, or other methods
  • Security reports checked/validated by a reviewer from the AppSec team

Related to #5696 (closed)

Edited by 🤖 GitLab Bot 🤖

Merge request reports

Loading