Skip to content

Fix exposing artifacts with dot slash paths

Erick Bajao requested to merge eb-fix-expose-as-path into master

What does this MR do and why?

Resolves #419627 (closed)

When artifact paths specified in the CI config starts with the ./ prefix, the pattern matching for metadata entries don't work properly. This fixes the bug to make sure that we always use the path stripped off of the prefix.

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

Screenshot based on the CI config from eb-tests/test-expose-as!1.

Before After
Screenshot_2023-08-19_at_1.06.52_AM Screenshot_2023-08-19_at_1.07.06_AM

How to set up and validate locally

  1. Clone the test project from eb-tests/test-expose-as!1, and open the same MR so you can test the exposed artifacts.
  2. As seen on the screenshot, the build-artifacts-2 should show up even though all its artifact paths include the ./ prefix.
  3. Then click on the my artifacts link for build-artifacts-2 and it should redirect you to the browse page. There should be all the artifacts there namely hello_dot_1.txt and hello_dot_2.txt.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Merge request reports