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

Add option to disable 'report.html.html' and 'report.html.json' created when using cli #3685

Closed
red8888 opened this issue Jun 9, 2023 · 8 comments

Comments

@red8888
Copy link

red8888 commented Jun 9, 2023

Which version and edition of Flyway are you using?

redgate/flyway:9.19

I'm confused why this was closed? #3652

Its still an outstanding feature request no? It wasn't "resolved", I still have no way of controlling if report files are generated and where they are written.

There should be a toggle to disable all together. This is messing up my deployment automation and writing files into app repos when run locally users need to remember to delete. Its not even in a subfolder to make it easy to git ignore.

@Phil-Factor
Copy link

I had an interesting example the other day. A 10 Mb file of insert statements for test data failed due to a system-named constraint having a different name. That generated an error that not only explained what was wrong but provided the entire contents of the file onscreen. When Flyway does that, I just hit ^C to stop the endless scrolling, but now , errors like this get into these files and bulks them up enormously from then on. You can delete them in Flyway Teams with an afterInfo batch file but that is scarecly the point - it slows down Flyway Info to write them

@red8888
Copy link
Author

red8888 commented Jul 26, 2023

@Phil-Factor I pay for flyway so I have teams, what is this "afterInfo batch file" you speak of?

I currently just chain the flyway command with an rm command to clean up but that sucks.

@Phil-Factor
Copy link

@red8888 Hah! Me too. I've always wanted to find a use for beforeInfo and afterInfo callbacks so I took pleasure it writing one. However I now find that I need to delete the awful files after every operation.

@sam-artuso
Copy link

This is annoying. Please fix it!

@bekoeppel
Copy link

You can disable the reports with the flyway.reportEnabled option:

public static final String REPORT_ENABLED = "flyway.reportEnabled";

On the CLI: flyway -reportEnabled=false migrate.

@Mario-Eis
Copy link

It should be disabled by default. Always typing this flag in the CLI is annoying.

@Phil-Factor
Copy link

I notice that '-outputType=json' has been removed from the documentation. Is this now deprecated? If so, what is the new way of doing it? Do we now have to use report.json? If so, how do we switch off the annoying 'report.html' and make report.json optional?

@JasonLuo-Redgate
Copy link

In V10 release, the report is disabled by default.

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

No branches or pull requests

6 participants