Skip to content

Enhance launch script for multi-tier deployment.#688

Merged
sbrossie merged 1 commit into
killbill:v3from
vnandwana:v3
Jul 14, 2026
Merged

Enhance launch script for multi-tier deployment.#688
sbrossie merged 1 commit into
killbill:v3from
vnandwana:v3

Conversation

@vnandwana

Copy link
Copy Markdown
Contributor

No description provided.

db_password="<DB passwordd>"
kb_admin_password="<Kaui admin login password>"

KB_PLUGINS_TEMP_DIR="/var/lib/killbill/bundles/temp"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Do we typically use temp or tmp - does it matter?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This is a directory we create during AMI creation (we can give any name) -- https://github.com/killbill/killbill-cloud/blob/master/ansible/roles/killbill/tasks/plugins/kpm_plugin.yml#L10


echo "Installing plugin DDLs..."

find "${KB_PLUGINS_TEMP_DIR}" \

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Are we guarantee that all ddl files will be there - the loop wait for the first one if I understand correctly?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Ansible copies these DDL files during AMI creation: https://github.com/killbill/killbill-cloud/blob/master/ansible/roles/killbill/tasks/plugins/kpm_plugin.yml#L18. If the AMI is created successfully, these files should be present when the instance is launched.

@sbrossie
sbrossie merged commit f0ae048 into killbill:v3 Jul 14, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants