Skip to content
This repository has been archived by the owner on Sep 2, 2022. It is now read-only.

"prisma deploy -n" does not migrate pre 1.7.x prisma.yml file #2354

Closed
marktani opened this issue May 5, 2018 · 4 comments
Closed

"prisma deploy -n" does not migrate pre 1.7.x prisma.yml file #2354

marktani opened this issue May 5, 2018 · 4 comments

Comments

@marktani
Copy link
Contributor

marktani commented May 5, 2018

Bug Report

Current behavior

Running prisma deploy -n with an existing prisma.yml file does not convert service, stage and cluster correctly.

Here is what happens:

  • line 19 in the screenshot is added
  • nothing happens to line 1-3
  • no backup file is created

image 11

Expected behavior?

  • backup file with old values should be created
  • line 19 should be added in new file
  • line 1-3 should be removed in new file
@marktani marktani changed the title prisma deploy -n "prisma deploy -n" does not migrate pre 1.7.x prisma.yml file May 5, 2018
@stale
Copy link

stale bot commented Jun 27, 2018

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the status/stale Marked as state by the GitHub stalebot label Jun 27, 2018
@marktani
Copy link
Contributor Author

prisma deploy -n currently comments out the endpoint property and appends the new endpoint in the now commented out line, which should be adjusted as well.

@stale stale bot removed the status/stale Marked as state by the GitHub stalebot label Jun 27, 2018
@divyenduz
Copy link
Contributor

divyendusingh [prisma-2354]$ cat prisma.yml
#endpoint: http://localhost:4466
#endpoint: http://localhost:4466/z/z
endpoint: https://eu1.prisma.sh/divyenduz/prisma-2354/dev
datamodel: datamodel.graphqldivyendusingh [prisma-2354]$ prisma --version
prisma/1.10.2 (darwin-x64) node-v10.4.0

Unable to replicate with Prisma CLI 1.10.2.
Appends the endpoint in newline correctly.

@marktani
Copy link
Contributor Author

marktani commented Jul 6, 2018

The original issue is invalid. Persisted the other one here: #2742.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants