Skip to content

ADBM-2955: Update pgbackrest tests according to changed docker inspect output#102

Merged
GirgerS merged 2 commits into2.54-cifrom
ADBM-2955
Mar 11, 2026
Merged

ADBM-2955: Update pgbackrest tests according to changed docker inspect output#102
GirgerS merged 2 commits into2.54-cifrom
ADBM-2955

Conversation

@GirgerS
Copy link

@GirgerS GirgerS commented Feb 25, 2026

The IP address has been moved in newer versions of Docker but this expression will fetch the IP address from the old or new location.

(cherry picked from commit 2b6a1de295160ddb016a7d65c8f5c08ff408b02c)

Note:

  • This api was deprecated since Docker 1.9.0, which was released back in 2015.
  • This branch is intended to merged with rebase.

The IP address has been moved in newer versions of Docker but this expression will fetch the IP address from the old or new location.

(cherry picked from commit 2b6a1de295160ddb016a7d65c8f5c08ff408b02c)
@GirgerS GirgerS changed the title Use more generic expression to fetch container IP addresses. ADBM-2955: ADBM-2955: Update pgbackrest tests according to changed docker inspect output Feb 25, 2026
@GirgerS GirgerS changed the title ADBM-2955: ADBM-2955: Update pgbackrest tests according to changed docker inspect output ADBM-2955: Update pgbackrest tests according to changed docker inspect output Feb 25, 2026
@ShapatinAA
Copy link

why did it failed one test? Seems like this #101 PR was meant to fix it right?

@GirgerS
Copy link
Author

GirgerS commented Mar 10, 2026

why did it failed one test? Seems like this #101 PR was meant to fix it right?

These changes were meant to fix RH8 documentation build, and the job works as expected.

The last failing job is fixed here, it is a separate issue (and ticket).

P.S. we also had to fix caching issue for Debian/Ubuntu, only because other changes required it

@GirgerS GirgerS merged commit fa80caa into 2.54-ci Mar 11, 2026
15 of 16 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.

4 participants