New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
HPCC-23717 Change dali to use new config #13502
HPCC-23717 Change dali to use new config #13502
Conversation
https://track.hpccsystems.com/browse/HPCC-23717 |
while (held) | ||
Sleep(5); | ||
} | ||
#endif |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@richardkchapman - moved here so I could use via values.yaml - any objections?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's already supported in jptree via values.yaml - see line 7881 or so.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ah - never mind I see what you did. The only downside of moving it here is that it will make it harder to debug a situation where it's crashing in config merging code.
You could always leave it in both places...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The only downside of moving it here is that it will make it harder to debug a situation where it's crashing in config merging code.
true, yes in _DEBUG why not. I'll change.
StringBuffer s("Failure whilst preparing dali backup location: "); | ||
LOG(MCoperatorError, unknownJob, e, s.append(mirrorPath).append(". Backup disabled").str()); | ||
serverConfig->removeProp("SDS/@remoteBackupLocation"); | ||
e->Release(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Much of the above is different because indentation/scope change only.
@richardkchapman - please review |
@jakesmith This is waiting for a change you promised... |
e452f24
to
eaeb4ac
Compare
@richardkchapman - changed --hold so picked up in either place. |
Automated Smoketest: ❌ Build error: |
Signed-off-by: Jake Smith <jake.smith@lexisnexisrisk.com>
eaeb4ac
to
a52bd9f
Compare
Automated Smoketest: ✅ Unit tests result:
Regression test result:
HPCC Stop: OK
|
Signed-off-by: Jake Smith jake.smith@lexisnexisrisk.com
Type of change:
Checklist:
Smoketest:
Testing: