Stream: dev

Topic: database settings improvements #11654


view this post on Zulip Philip Durbin ๐Ÿš€ (Jul 30 2025 at 13:55):

@Oliver Bertuch thanks for PR #11654 where you're improving how database settings are handled and for explaining it during tech hours yesterday!

view this post on Zulip Philip Durbin ๐Ÿš€ (Jul 30 2025 at 13:56):

I picked it up for review and I'm thinking I'd like to see Jenkins successfully running the API test suite. @Don Sizemore how do you feel about an extra Jenkins job pointed at that branch and an upcoming ansible branch specifically? We've done this for Payara upgrades in the past.

view this post on Zulip Philip Durbin ๐Ÿš€ (Jul 30 2025 at 13:57):

@Oliver Bertuch have you created an issue in the ansible repo yet? If not, do you mind?

view this post on Zulip Oliver Bertuch (Jul 30 2025 at 14:03):

There's already a PR ready to go! https://github.com/gdcc/dataverse-ansible/pull/399

view this post on Zulip Philip Durbin ๐Ÿš€ (Jul 30 2025 at 14:04):

Ah! Thanks! I looked at issues but not PRs.

view this post on Zulip Oliver Bertuch (Jul 30 2025 at 14:04):

Am I spoiling @Don Sizemore too much? :partying_face:

view this post on Zulip Philip Durbin ๐Ÿš€ (Jul 30 2025 at 14:05):

Ha, you can never spoil him enough. We owe him so much. :dataverse_man:

view this post on Zulip Philip Durbin ๐Ÿš€ (Sep 02 2025 at 15:11):

@Oliver Bertuch I gave this PR back to you. Can you please look at https://github.com/IQSS/dataverse/pull/11654#discussion_r2263981716 ?

view this post on Zulip Philip Durbin ๐Ÿš€ (Sep 03 2025 at 20:30):

Also, it seems a bit disruptive. Do you still want to try to get it in 6.8 (last call for core devs is tomorrow) or should we push it to 6.9?

view this post on Zulip Oliver Bertuch (Sep 08 2025 at 12:02):

I'm fine with pushing it to 6.9. :wink:

view this post on Zulip Philip Durbin ๐Ÿš€ (Sep 08 2025 at 12:10):

Pushed! Thanks!

view this post on Zulip Philip Durbin ๐Ÿš€ (Sep 08 2025 at 12:12):

I also moved it from "in progress" to "on hold". Happy to chat about it whenever! I feel like it's close.

view this post on Zulip Philip Durbin ๐Ÿš€ (Sep 08 2025 at 12:12):

I'm just not sure if you really want to change how WorkflowsAdmin#IP_WHITELIST_KEY works or not. :grimacing:

view this post on Zulip Oliver Bertuch (Sep 08 2025 at 12:22):

Not the first time I'm taking out the broom to cleanup oopsies in the codebase :stuck_out_tongue_wink:

view this post on Zulip Philip Durbin ๐Ÿš€ (Sep 08 2025 at 12:55):

nope

view this post on Zulip Philip Durbin ๐Ÿš€ (Sep 26 2025 at 19:32):

@Oliver Bertuch I merged develop into #11654 for you and then moved it to "ready for QA".

view this post on Zulip Philip Durbin ๐Ÿš€ (Oct 24 2025 at 13:31):

@Oliver Bertuch I resolved merge conflicts for you yesterday (just in changelog.rst).

Meanwhile, @Omer M Fahim and I noticed this error:

TASK [dataverse : STORAGE | Get infos on container] ********
fatal: [localhost]: FAILED! => {"changed": false, "msg": "Error connecting: Error while fetching server API version: Not supported URL scheme http+docker"}

Any ideas? Or maybe @Don Sizemore knows? Here's the latest run, which I just left a comment about: https://jenkins.dataverse.org/job/IQSS-Dataverse-11639/22/console

view this post on Zulip Don Sizemore (Oct 24 2025 at 14:54):

Philip Durbin ๐Ÿš€ said:

Oliver Bertuch I resolved merge conflicts for you yesterday (just in changelog.rst).

Meanwhile, Omer M Fahim and I noticed this error:

TASK [dataverse : STORAGE | Get infos on container] ********
fatal: [localhost]: FAILED! => {"changed": false, "msg": "Error connecting: Error while fetching server API version: Not supported URL scheme http+docker"}

Any ideas? Or maybe Don Sizemore knows? Here's the latest run, which I just left a comment about: https://jenkins.dataverse.org/job/IQSS-Dataverse-11639/22/console

The http+docker error should've been fixed in https://github.com/gdcc/dataverse-ansible/commit/916b3db27ded3434b84f43b456a0022afd86509d โ€” I'll take a look at that job and see why it's popped back up?

view this post on Zulip Philip Durbin ๐Ÿš€ (Oct 24 2025 at 14:55):

Oh, ok. Yes, please take a look. Thanks, Don! :dataverse_man:


Last updated: Nov 01 2025 at 14:11 UTC