Skip to content
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

platformify validate command fails with config file produced with platformify #145

Open
gilzow opened this issue Aug 30, 2023 · 4 comments
Labels
bug Something isn't working requires-more-info

Comments

@gilzow
Copy link
Contributor

gilzow commented Aug 30, 2023

Building latest version from main

After running platformify and answering the questions, if I then try to run platformify validate it reports a validation failure:

Platform.sh config validation failed: validation failed for utils/ify/platformifier/templates/generic/.platform.app.yaml: yaml: line 20: could not find expected ':'

Running the same commands for upsunify reports a valid config file.

@gilzow
Copy link
Contributor Author

gilzow commented Aug 30, 2023

if I add an incorrect property to my .platform.app.yaml file and run platformify validate it does correctly report the issue as well as the one mentioned above:

Platform.sh config validation failed: validation failed for .platform.app.yaml: mounts.Platformify_Cookies/static: source_path is required
mounts.Platformify_Cookies/static: Additional property path is not allowed
validation failed for utils/ify/platformifier/templates/generic/.platform.app.yaml: yaml: line 20: could not find expected ':'

@akalipetis
Copy link
Member

@gilzow can you please provide the repository you're running the command against and the answers you're giving? It might be a corner-case we're not currently handling.

@akalipetis akalipetis added bug Something isn't working requires-more-info labels Sep 18, 2023
@gilzow
Copy link
Contributor Author

gilzow commented Sep 18, 2023

it is project dmz5nuanqilnu https://console.platform.sh/paul-gilzowatplatform-sh/dmz5nuanqilnu
Let me know if I need to add you to the project.
The command I'm running is validate after building platformify from the main, built from commit:

commit 515b17540008a49705a699a2738ee2f5cf44a561 (HEAD -> main, tag: v0.2.1, origin/main, origin/HEAD)

The line it is erroring on is https://github.com/platformsh/platformify/blob/main/platformifier/templates/generic/.platform.app.yaml#L20

@akalipetis
Copy link
Member

@gilzow we have figured out that this was because of cloning the Platformify repository inside your project. If you still face the issue, let me know.

Closing for now.

@akalipetis akalipetis closed this as not planned Won't fix, can't repro, duplicate, stale Sep 21, 2023
@akalipetis akalipetis reopened this Jan 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working requires-more-info
Projects
None yet
Development

No branches or pull requests

2 participants