Skip to content

Conversation

@anantha1999
Copy link

No description provided.

@anantha1999 anantha1999 force-pushed the mariadb-logical-backup branch from ef4503b to 6926f45 Compare January 1, 2026 08:53
gcc \
libffi-dev \
&& curl -sL https://aka.ms/InstallAzureCLIDeb | bash \
&& pip3 install --upgrade pip \
Copy link
Member

Choose a reason for hiding this comment

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

we need to pin to a version, so next image build will have same version of pip

Copy link
Author

Choose a reason for hiding this comment

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

made the changes in both postgres and mariadb logical backup dockerfiles

@anantha1999 anantha1999 force-pushed the mariadb-logical-backup branch from 4c9938c to 525fc7b Compare January 1, 2026 11:01
az_upload /tmp/azure-backup.sql.gz
else

# Backup all the databases owned by the user if PGDATABASE is not set
Copy link
Author

Choose a reason for hiding this comment

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

F.ex in opendesk multiple apps use the same postgres instance with different users. This handles that.

Copy link
Author

Choose a reason for hiding this comment

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

have removed these, as found a few more changes required. Will raise a separate PR for that.

@anantha1999 anantha1999 force-pushed the mariadb-logical-backup branch from 525fc7b to 70372ee Compare January 1, 2026 11:06
@anantha1999 anantha1999 changed the title feat: add mariadb logical backup feat: add mariadb logical backup and handle user specific database backup for postgres Jan 1, 2026

echo "Uploading dump to s3"
aws s3 cp - "$PATH_TO_BACKUP" "${args[@]//\'/}"
aws s3 cp - "$PATH_TO_BACKUP" "${args[@]}"
Copy link
Author

Choose a reason for hiding this comment

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

fixes below

Uploading dump to s3
fatal error: invalid literal for int() with base 10: '34613139 --endpoint-url=http://localhost:9000 --region=us-west-1'

Copy link
Author

Choose a reason for hiding this comment

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

have removed these, as found a few more changes required. Will raise a separate PR for that.

"$PG_BIN"/pg_dump --verbose
else
"$PG_BIN"/pg_dumpall --exclude-database='postgres'
"$PG_BIN"/pg_dumpall --exclude-database='postgres' --verbose
Copy link
Author

Choose a reason for hiding this comment

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

added verbose for logs in the cronjob

Copy link
Author

Choose a reason for hiding this comment

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

have removed these, as found a few more changes required. Will raise a separate PR for that.

@anantha1999 anantha1999 force-pushed the mariadb-logical-backup branch from 70372ee to 27a28ec Compare January 2, 2026 04:54
@anantha1999 anantha1999 changed the title feat: add mariadb logical backup and handle user specific database backup for postgres feat: add mariadb logical backup Jan 2, 2026
@anantha1999 anantha1999 force-pushed the mariadb-logical-backup branch from 27a28ec to 580f06b Compare January 2, 2026 08:27
@anantha1999 anantha1999 force-pushed the mariadb-logical-backup branch from 580f06b to 1e23dac Compare January 2, 2026 08:35
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.

3 participants