Skip to content
This repository was archived by the owner on Dec 2, 2025. It is now read-only.

Fix catalogInfoPath in template.yaml#6

Closed
jayonthenet wants to merge 1 commit intohumanitec-architecture:mainfrom
jayonthenet:main
Closed

Fix catalogInfoPath in template.yaml#6
jayonthenet wants to merge 1 commit intohumanitec-architecture:mainfrom
jayonthenet:main

Conversation

@jayonthenet
Copy link
Contributor

Update the catalogInfoPath in template.yaml to point to the correct location.

Signed-off-by: jayonthenet <clemens@humanitec.com>
Copy link
Contributor

@mathieu-benoit mathieu-benoit left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, out of curiosity, is it something that should be done for the other templates?

@ajmoorexyz
Copy link

ajmoorexyz commented Feb 14, 2025

@jayonthenet thanks for putting in this fix. I can attest that I wasn't able to get a clean run of the 5min IDP - NodeJS Service Template following pe-course-labs/pocketidp.md. I eventually tracked the issue down this fix as well. I compared the 5min-podinfo/template.yaml to this template. Pretty cool to see how everything wires up. This should definitely get merged before the next PE Fundamentals cohort 😜

@mathieu-benoit no issues with the 5min Podinfo Service Template per the pe-course-labs/pocketidp.md setup. That's not to say that there could be other templates with a botched path used in other places. 🤷‍♂️

@jayonthenet jayonthenet closed this by deleting the head repository Feb 17, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants