Skip to content

Fix access to logs when multiple pods exist

Adrien Kohlbecker requested to merge ak/fix-multi-pod into master

What does this MR do?

Related issue #210048 (closed)

The right way to find the first object matching a block is .find not .first in ruby.

This prevented access to any logs besides the first pod returned by the kubernetes API.

Added a new spec to catch that bug in the future.

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
Edited by 🤖 GitLab Bot 🤖

Merge request reports