-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Start Docker from Java for server RPM tests #2064
Conversation
presto-server-rpm/src/test/java/io/prestosql/server/rpm/ServerIT.java
Outdated
Show resolved
Hide resolved
presto-server-rpm/src/test/java/io/prestosql/server/rpm/ServerIT.java
Outdated
Show resolved
Hide resolved
presto-server-rpm/src/test/java/io/prestosql/server/rpm/ServerIT.java
Outdated
Show resolved
Hide resolved
// install RPM | ||
"rpm -i /target/" + rpm + "\n" + | ||
// create Hive catalog file | ||
"mkdir /etc/presto/catalog\n" + |
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.
/etc/presto/catalog
should be part of RPM
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 not today, and I don't see anything which should be creating it. Perhaps we should do so in postinstall
?
"hive.metastore.uri=thrift://localhost:9083\n" + | ||
"EOT\n" + | ||
// start server | ||
"/etc/init.d/presto start\n" + |
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.
service presto start
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.
Looks like our image does not include that command
presto-server-rpm/src/test/java/io/prestosql/server/rpm/ServerIT.java
Outdated
Show resolved
Hide resolved
@@ -55,6 +55,8 @@ | |||
<dep.tempto.version>166</dep.tempto.version> | |||
<dep.testng.version>6.10</dep.testng.version> | |||
<dep.assertj-core.version>3.8.0</dep.assertj-core.version> | |||
<!-- TODO: remove after updating to Airbase 94 --> | |||
<dep.slf4j.version>1.7.28</dep.slf4j.version> |
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.
how this relates to test change?
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.
These version updates are needed to satisfy RequireUpperBoundDeps
due to testcontainers
using newer versions.
No description provided.