Skip to content

test/realms: Bump memory to 2000MB - #23499

Merged
tomasmatus merged 1 commit into
cockpit-project:mainfrom
Venefilyn:realms/memory-bump
Jul 16, 2026
Merged

test/realms: Bump memory to 2000MB#23499
tomasmatus merged 1 commit into
cockpit-project:mainfrom
Venefilyn:realms/memory-bump

Conversation

@Venefilyn

Copy link
Copy Markdown
Member

When testing there were stderr about memory issues for the freeipa
container. Not evident by looking at the logs themselves but saw it
without detached mode on.

Tested using bots/vm-run services -M 2000. Might not work on your
local machine as I also modified services image.

See-also: cockpit-project/bots#9260
Signed-off-by: Freya Gustavsson freya@venefilyn.se

@tomasmatus

Copy link
Copy Markdown
Member

kerberos test "times out" because it cannot nslookup the DNS entry, digging deeper...

+ sleep 19
+ for x in $(seq 1 20)
+ nslookup -type=SRV _ldap._tcp.cockpit.lan
+ sleep 20
+ echo foobarfoo
+ realm join -vU admin cockpit.lan
 * Resolving: _ldap._tcp.cockpit.lan
 * Resolving: cockpit.lan
 * No results: cockpit.lan
realm: No such realm found
+ systemctl --quiet is-fail

Comment thread test/verify/check-system-realms Fixed
@tomasmatus tomasmatus added the no-test For doc/workflow changes, or experiments which don't need a full CI run, label Jul 14, 2026
@tomasmatus
tomasmatus force-pushed the realms/memory-bump branch 2 times, most recently from 30a34b6 to 20a545e Compare July 14, 2026 14:16
@tomasmatus

Copy link
Copy Markdown
Member

well ofc now it goes green :D

@tomasmatus

Copy link
Copy Markdown
Member

All is green even with amplification, dropping debug commits and we can merge this.

@tomasmatus tomasmatus removed the no-test For doc/workflow changes, or experiments which don't need a full CI run, label Jul 15, 2026
@tomasmatus
tomasmatus force-pushed the realms/memory-bump branch from 6e62fd7 to 9ecd859 Compare July 15, 2026 08:15
When testing there were stderr about memory issues for the freeipa
container. Not evident by looking at the logs themselves but saw it
without detached mode on.

Tested using `bots/vm-run services -M 2000`. Might not work on your
local machine as I also modified services image.

See-also: cockpit-project/bots#9260
Signed-off-by: Freya Gustavsson <freya@venefilyn.se>
@tomasmatus
tomasmatus force-pushed the realms/memory-bump branch from 9ecd859 to a3a837a Compare July 15, 2026 08:18
@tomasmatus

tomasmatus commented Jul 15, 2026

Copy link
Copy Markdown
Member

I've also bumped the services VM memory in the testKerberos class, it also runs the freeipa container so we'd likeky run into memory issues there too.

@tomasmatus
tomasmatus requested review from allisonkarlitskaya and mvollmer and removed request for allisonkarlitskaya July 15, 2026 09:39

@mvollmer mvollmer left a comment

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.

More is more!

@tomasmatus
tomasmatus merged commit 773496a into cockpit-project:main Jul 16, 2026
93 of 94 checks passed
@Venefilyn
Venefilyn deleted the realms/memory-bump branch July 16, 2026 09:51
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