Skip to content

Commit

Permalink
Fix lint.
Browse files Browse the repository at this point in the history
  • Loading branch information
Jeremy Lewi committed Oct 23, 2019
1 parent c069d68 commit 33d3ff2
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 11 deletions.
2 changes: 1 addition & 1 deletion py/kubeflow/testing/cleanup_ci.py
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@
# See https://github.com/kubeflow/testing/issues/444
# We are switching to unique names for auto deployments
# So this matches the new ones.
AUTO_DEPLOY_PATTERN = re.compile("kf-vmaster-(?!n\d\d)")
AUTO_DEPLOY_PATTERN = re.compile(r"kf-vmaster-(?!n\d\d)")

# Regexes that select matching deployments
MATCHING = [re.compile("e2e-.*"), re.compile("kfctl.*"),
Expand Down
2 changes: 1 addition & 1 deletion py/kubeflow/testing/create_kf_instance.py
Original file line number Diff line number Diff line change
Expand Up @@ -110,7 +110,7 @@ def deploy_with_kfctl_go(kfctl_path, args, app_dir, env):
config_spec["spec"] = util.filter_spartakus(config_spec["spec"])

# Remove name because we will auto infer from directory.
if "name" in config_file["metadata"]:
if "name" in config_spec["metadata"]:
logging.info("Deleting name in kfdef spec.")
del config_spec["metadata"]["name"]

Expand Down
11 changes: 2 additions & 9 deletions py/kubeflow/testing/create_unique_kf_instance.py
Original file line number Diff line number Diff line change
Expand Up @@ -13,21 +13,15 @@
import argparse
import datetime
import logging
import json
import os
import re
import requests
import shutil
import subprocess
import tempfile
import uuid
import yaml

from googleapiclient import discovery
from google.cloud import storage
from kubeflow.testing import util
from retrying import retry
from oauth2client.client import GoogleCredentials

@retry(wait_fixed=60000, stop_max_attempt_number=5)
def run_with_retry(*args, **kwargs):
Expand Down Expand Up @@ -183,9 +177,8 @@ def main(): # pylint: disable=too-many-locals,too-many-statements
env.update(os.environ)
env.update(oauth_info)

labels = { "GIT_LABEL": git_describe,
"PURPOSE": "kf-test-cluster",
}
labels = {"GIT_LABEL": git_describe,
"PURPOSE": "kf-test-cluster",}

label_args = []
for k, v in labels.items():
Expand Down

0 comments on commit 33d3ff2

Please sign in to comment.