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 hint about flyway usage #4086

Merged
merged 8 commits into from
Aug 26, 2024
Merged

add hint about flyway usage #4086

merged 8 commits into from
Aug 26, 2024

Conversation

jonathanlukas
Copy link
Contributor

@jonathanlukas jonathanlukas commented Jul 31, 2024

Description

Context: https://camunda.slack.com/archives/C0693F1NFK5/p1722411282734599

TL;DR: A customer was facing the issue that web modeler was not running as it claimed "the database schema is not empty" which can be due to functions being present in the database schema.

The newly added section should give the user (infra engineer) hints on what could be the problem and how it could be solved.

When should this change go live?

  • This is a bug fix, security concern, or something that needs urgent release support.
  • This is already available but undocumented and should be released within a week.
  • This on a specific schedule and the assignee will coordinate a release with the DevEx team. (apply hold label or convert to draft PR)
  • This is part of a scheduled alpha or minor. (apply alpha or minor label)
  • There is no urgency with this change and can be released at any time.

PR Checklist

  • My changes are for an already released minor and are in /versioned_docs directory.
  • My changes are for the next minor and are in /docs directory (aka /next/).

@jonathanlukas jonathanlukas self-assigned this Jul 31, 2024
@jonathanlukas jonathanlukas added the component:web-modeler-sm Issues related with Web Modeler Self-Managed label Jul 31, 2024
@akeller
Copy link
Member

akeller commented Jul 31, 2024

DevEx will need more context to do a proper review. For now, we can only review for style and grammar.

@jonathanlukas
Copy link
Contributor Author

@akeller thank you for flagging that. I added some context.

@christinaausley
Copy link
Contributor

@jonathanlukas Are you just pending review from @wollefitz here, or any additional changes coming in?

@christinaausley
Copy link
Contributor

@jonathanlukas Let me know when you're ready for final review and I'll have one more look 👍

Copy link
Contributor

github-actions bot commented Aug 23, 2024

👋 🤖 ✅ Looks like the changes were ported across versions, nice job! 🎉

You can read more about the versioning within our docs in our documentation guidelines.

@jonathanlukas
Copy link
Contributor Author

@wollefitz the section has been moved and slightly renamed
@christinaausley this is ready for a final review I think

Copy link
Contributor

@christinaausley christinaausley left a comment

Choose a reason for hiding this comment

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

Nice 👍 Just uppercased Flyway.

Copy link
Member

@wollefitz wollefitz left a comment

Choose a reason for hiding this comment

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

Thanks for adding this!

@jonathanlukas jonathanlukas merged commit 8f818a3 into main Aug 26, 2024
7 checks passed
@jonathanlukas jonathanlukas deleted the web-modeler/flyway-hint branch August 26, 2024 04:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:web-modeler-sm Issues related with Web Modeler Self-Managed
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

4 participants