Chris Calloway

Results 42 comments of Chris Calloway

> @cbcunc Was wondering if you did any test using postgres 14 with existing data. @pkdash I approved these changes. But it's not enough to merge. Please don't merge yet....

> > @cbcunc Was wondering if you did any test using postgres 14 with existing data. > > @pkdash I approved these changes. But it's not enough to merge. Please...

Should conf_postgres be removed? I believe it only applies to the old PG image. I don't think it is healthy to simply overwrite pg_hba.conf and postgresql.conf from an old version...

https://groups.google.com/g/irod-chat/c/uZpNuQn6ytw

@trel All the tests that invoke the rule fail with ```django_irods.icommands.SessionException: (SessionException(...), "Error processing IRODS request: -1. stderr follows:\n\nb' ERROR: parseMsInputParam: inpParam hydroshare/irods/ruleGenerateBagIt_HS.r format error\\n ERROR: Invalid input parameter list...

Possibly. I'll try that. I'm curious that the last three arguments aren't positional. self.session.run is a call into django_irods that Jason wrote.

Ah, yeah, we already tried that in the very first commit. The arguments are positional. https://github.com/hydroshare/hydroshare/pull/5287/commits/99fa4f0e4f70a92b3fd0c1ec2a9063063c62f0f8

I will try it again, though, as it looks like it should work: https://github.com/hydroshare/hydroshare/blob/874d8239f0b8860bf9198b1514ce74197bc27063/django_irods/icommands.py#L152

@trel So now that each we're back to each space-delimited argument to irule is a separate parameter, this error on every every test involving this generated rule. @devincowan Can you...

https://github.com/hydroshare/hydroshare/pull/5287