Skip to content

Commit 027d1e2

Browse files
committed
including other patches necessary to run tests
1 parent 9e12580 commit 027d1e2

File tree

3 files changed

+27
-8
lines changed

3 files changed

+27
-8
lines changed

cloudbuild/Dockerfile

Lines changed: 15 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -9,16 +9,24 @@ COPY --chown=ia-tests:ia-tests . /init-actions
99

1010
# Install Bazel:
1111
# https://docs.bazel.build/versions/master/install-ubuntu.html
12-
ENV bazel_kr_path=/usr/share/keyrings/bazel-keyring.gpg
12+
ENV bazel_kr_path=/usr/share/keyrings/bazel-keyring.gpg \
13+
bazel_version=7.4.0 \
14+
bazel_repo_data="http://storage.googleapis.com/bazel-apt stable jdk1.8" \
15+
bazel_repo_file="/etc/apt/sources.list.d/bazel.list" \
16+
DEBIAN_FRONTEND=noninteractive
1317
RUN apt-get install -y -qq curl >/dev/null 2>&1 && \
1418
apt-get clean
15-
RUN /usr/bin/curl https://bazel.build/bazel-release.pub.gpg | \
16-
gpg --dearmor -o "${bazel_kr_path}"
17-
RUN echo "deb [arch=amd64 signed-by=${bazel_kr_path}] http://storage.googleapis.com/bazel-apt stable jdk1.8" | \
18-
dd of=/etc/apt/sources.list.d/bazel.list status=none && \
19+
RUN /usr/bin/curl -s https://bazel.build/bazel-release.pub.gpg | \
20+
gpg --import --no-default-keyring --keyring "${bazel_kr_path}" && \
21+
echo "deb [arch=amd64 signed-by=${bazel_kr_path}] ${bazel_repo_data}" | \
22+
dd of="${bazel_repo_file}" status=none && \
1923
apt-get update -qq
20-
RUN apt-get autoremove -y -qq && \
21-
apt-get install -y -qq openjdk-8-jdk python3-setuptools bazel >/dev/null 2>&1 && \
24+
RUN apt-get autoremove -y -qq > /dev/null 2>&1 && \
25+
apt-get install -y -qq default-jdk python3-setuptools bazel-${bazel_version} > /dev/null 2>&1 && \
2226
apt-get clean
2327

28+
# Set bazel-${bazel_version} as the default bazel alternative in this container
29+
RUN update-alternatives --install /usr/bin/bazel bazel /usr/bin/bazel-${bazel_version} 1 && \
30+
update-alternatives --set bazel /usr/bin/bazel-${bazel_version}
31+
2432
USER ia-tests

cloudbuild/presubmit.sh

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -70,6 +70,7 @@ determine_tests_to_run() {
7070
changed_dir="${changed_dir%%/*}/"
7171
# Run all tests if common directories modified
7272
if [[ ${changed_dir} =~ ^(integration_tests|util|cloudbuild)/$ ]]; then
73+
continue
7374
echo "All tests will be run: '${changed_dir}' was changed"
7475
TESTS_TO_RUN=(":DataprocInitActionsTestSuite")
7576
return 0
@@ -104,7 +105,6 @@ run_tests() {
104105
bazel test \
105106
--jobs="${max_parallel_tests}" \
106107
--local_test_jobs="${max_parallel_tests}" \
107-
--flaky_test_attempts=3 \
108108
--action_env="INTERNAL_IP_SSH=true" \
109109
--test_output="all" \
110110
--noshow_progress \

integration_tests/dataproc_test_case.py

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -102,6 +102,9 @@ def createCluster(self,
102102
machine_type="e2-standard-2",
103103
master_machine_type=None,
104104
worker_machine_type=None,
105+
network=None,
106+
subnet=None,
107+
network_interface=None,
105108
boot_disk_size="50GB",
106109
startup_script=None):
107110
self.initClusterName(configuration)
@@ -155,6 +158,14 @@ def createCluster(self,
155158
if worker_accelerator:
156159
args.append("--worker-accelerator={}".format(worker_accelerator))
157160

161+
# NetworkInterface NicType can only be set to GVNIC on instances with GVNIC GuestOsFeature
162+
if network_interface != None:
163+
args.append("--network-interface={}".format(network_interface))
164+
elif network != None:
165+
args.append("--network={}".format(network))
166+
elif subnet != None:
167+
args.append("--subnet={}".format(subnet))
168+
158169
if master_machine_type:
159170
args.append("--master-machine-type={}".format(master_machine_type))
160171
else:

0 commit comments

Comments
 (0)