Conversation
It must be called after setup because it relies on the contents of PMEMOBJ_CONF which is set there.
test: call setup before require_sds
Signed-off-by: Jan Michalski <jan-marian.michalski@hpe.com>
common: update before install (aarch64)
test: unblock obj_sds tests for non-pmem
test: check if mock functions are used
Signed-off-by: Oksana Salyk <oksana.salyk@hpe.com>
common: update workflow files
+ minor adjustments to error/warning messages Signed-off-by: Oksana Salyk <oksana.salyk@hpe.com>
replace the outdated include command with the import_tasks command Signed-off-by: Oksana Salyk <oksana.salyk@hpe.com>
replace the outdated include command with the import_tasks command Signed-off-by: Oksana Salyk <oksana.salyk@hpe.com>
common: switch self hosted runners to the new repo location
Signed-off-by: Oksana Salyk <oksana.salyk@hpe.com>
common: fix typo
Due to limited hardware availability, the previous monolithic workflow could not meet the 24-hour limit imposed by GitHub. Signed-off-by: Oksana Salyk <oksana.salyk@hpe.com>
Reshuffle the jobs to fit a workflow containing the longest-running jobs into a 24-hour window. Hence there is no longer a clear cut (with Valgrind vs without Valgrind). They are simply called part 1 and part 2. Signed-off-by: Oksana Salyk <oksana.salyk@hpe.com>
Signed-off-by: Oksana Salyk <oksana.salyk@hpe.com>
Signed-off-by: Oksana Salyk <oksana.salyk@hpe.com>
Signed-off-by: Oksana Salyk <oksana.salyk@hpe.com>
common: add fixes to workflow files
Signed-off-by: Oksana Salyk <oksana.salyk@hpe.com>
https://daosio.atlassian.net/browse/DAOS-17449 Ref: #5 Unclean cherry-pick of 0bf7f8a Signed-off-by: Jan Michalski <jan-marian.michalski@hpe.com>
- add missing log entry Ref: #14 Signed-off-by: Jan Michalski <jan-marian.michalski@hpe.com>
…aster-fix test: expand the sds.at_create CTL to also cover pmemobj_open() (fix)
grom72
left a comment
grom72
left a comment
There was a problem hiding this comment.
@grom72 reviewed 1 of 82 files at r1.
Reviewable status: 1 of 82 files reviewed, 1 unresolved discussion (waiting on @janekmi)
ChangeLog line 12 at r1 (raw file):
- Conversely, pmemobj_open() does not issue a warning when attempting to open a pool with SDS disabled while the feature is force-disabled.
Worth to mention that
- new home for pmdk software has been set up
- SDS tests has been fixed/improved/etc
grom72
left a comment
grom72
left a comment
There was a problem hiding this comment.
Reviewable status: 1 of 82 files reviewed, 1 unresolved discussion (waiting on @janekmi and @osalyk)
ChangeLog line 12 at r1 (raw file):
Previously, grom72 (Tomasz Gromadzki) wrote…
Worth to mention that
- new home for pmdk software has been set up
- SDS tests has been fixed/improved/etc
- PMem HW tests has been reorganized
grom72
left a comment
grom72
left a comment
There was a problem hiding this comment.
@grom72 reviewed 81 of 82 files at r1, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @janekmi and @osalyk)
Signed-off-by: Oksana Salyk <oksana.salyk@hpe.com>
Signed-off-by: Oksana Salyk <oksana.salyk@hpe.com>
osalyk
force-pushed
the
release_2.1.2-rc1
branch
from
1b9ce12 to
f9eb131
Compare
osalyk
left a comment
osalyk
left a comment
There was a problem hiding this comment.
Reviewable status: 81 of 82 files reviewed, 1 unresolved discussion (waiting on @grom72 and @janekmi)
ChangeLog line 12 at r1 (raw file):
Previously, grom72 (Tomasz Gromadzki) wrote…
- PMem HW tests has been reorganized
Done.
janekmi
left a comment
janekmi
left a comment
There was a problem hiding this comment.
@janekmi reviewed 81 of 82 files at r1, 1 of 1 files at r2, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @grom72)
grom72
left a comment
grom72
left a comment
There was a problem hiding this comment.
@grom72 reviewed all commit messages.
Reviewable status:complete! all files reviewed, all discussions resolved (waiting on @osalyk)
grom72
left a comment
grom72
left a comment
There was a problem hiding this comment.
@grom72 reviewed 1 of 1 files at r2.
Reviewable status:
3 participants
This change is