Page MenuHomeSolus

Update mongo-tools to 4.0.5
ClosedPublic

Authored by der_eismann on Dec 24 2018, 12:54 PM.
Tags
None
Referenced Files
F11046607: D4812.diff
Thu, Aug 10, 1:47 PM
F11021556: D4812.diff
Mon, Aug 7, 3:34 PM
F11001283: D4812.diff
Sun, Jul 30, 1:40 AM
F10887607: D4812.id12201.diff
Jun 26 2023, 2:27 PM
F10886632: D4812.id11751.diff
Jun 26 2023, 12:54 AM
F10883241: D4812.diff
Jun 24 2023, 2:56 AM
F10831133: D4812.id12201.diff
Jun 5 2023, 7:15 AM
F10816427: D4812.diff
Jun 1 2023, 2:21 AM
Subscribers

Details

Summary

Click here to have an overview of the commits

Test Plan
  • Enabled unit tests
  • Restore a database with mongorestore --archive=test.20150715.archive --db test
  • Dump a datbase with mongodump --archive=test.20150715.archive --db test

Diff Detail

Repository
R3585 mongo-tools
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Packaging go stuff in solbuild can really be a PITA, especially when upstream changes their build scripts.
It's working now, but the unit tests fail for mongorestore and mongoreplay and I have no idea why:

+ cd mongorestore
+ go test
# _/home/build/YPKG/root/mongo-tools/build/mongo-tools.git/mongorestore
./filepath.go:363: Logvf call has arguments but no formatting directives
FAIL	_/home/build/YPKG/root/mongo-tools/build/mongo-tools.git/mongorestore [build failed]
Command '['/bin/bash', '--norc', '--noprofile', '/tmp/ypkg-check6FwVCX']' returned non-zero exit status 2
[Build] check failed


+ cd mongoreplay
+ go test
# _/home/build/YPKG/root/mongo-tools/build/mongo-tools.git/mongoreplay
./packet_handler.go:66: logWrapper.Logvf call has arguments but no formatting directives
FAIL	_/home/build/YPKG/root/mongo-tools/build/mongo-tools.git/mongoreplay [build failed]
Command '['/bin/bash', '--norc', '--noprofile', '/tmp/ypkg-checkx4YBfH']' returned non-zero exit status 2
[Build] check failed

When you take a look at the source you can see there *is* a formatting directive. So I disabled unit tests for now.

This revision is now accepted and ready to land.Dec 26 2018, 1:54 PM
This revision was automatically updated to reflect the committed changes.