Update gcp integration error messages. b/146443208 (#680) · HpcDataLab/docker-python@b017e4e · GitHub

HpcDataLab / docker-python Public

forked from Kaggle/docker-python

• Notifications You must be signed in to change notification settings
• Fork 0
• Star 0

• Code
• Pull requests
• Actions
• Projects
• Security and quality
• Insights

Additional navigation options

Commit b017e4e

Browse filesBrowse the repository at this point in the historyBrowse files

vimota

authored

Update gcp integration error messages. b/146443208 (Kaggle#680)

1 parent da9dc06 commit b017e4e
Copy full SHA for b017e4e

1 file changed

+4-4Lines changed: 4 additions & 4 deletions

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• patches
  • kaggle_gcp.py

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎patches/kaggle_gcp.py‎

Copy file name to clipboardExpand all lines: patches/kaggle_gcp.py

+4-4Lines changed: 4 additions & 4 deletions

Original file line number	Diff line number	Diff line change
@@ -75,7 +75,7 @@ def refresh(self, request):
75	75	if (not get_integrations().has_integration(self.target)):
76	76	Log.error(f"No {self.target.service} integration found.")
77	77	print(
78		- f"Please ensure you have selected a {self.target.service} account in the Kernels Settings sidebar.")
	78	+ f"Please ensure you have selected a {self.target.service} account in the Notebook Add-ons menu.")
79	79	raise RefreshError('Unable to refresh access token.') from e
80	80
81	81
@@ -96,7 +96,7 @@ def api_request(self, *args, **kwargs):
96	96	return super().api_request(*args, **kwargs)
97	97	except Forbidden as e:
98	98	msg = ("Permission denied using Kaggle's public BigQuery integration. "
99		- "Did you mean to select a BigQuery account in the Kernels Settings sidebar?")
	99	+ "Did you mean to select a BigQuery account in the Notebook Add-ons menu?")
100	100	print(msg)
101	101	Log.info(msg)
102	102	raise e
@@ -131,7 +131,7 @@ def init_bigquery():
131	131	if not (is_proxy_token_set or is_user_secrets_token_set):
132	132	return bigquery
133	133
134		- # If this Kernel has bigquery integration on startup, preload the Kaggle Credentials
	134	+ # If this Notebook has bigquery integration on startup, preload the Kaggle Credentials
135	135	# object for magics to work.
136	136	if get_integrations().has_bigquery():
137	137	from google.cloud.bigquery import magics
@@ -162,7 +162,7 @@ def monkeypatch_bq(bq_client, *args, **kwargs):
162	162	if (not has_bigquery):
163	163	Log.info("No bigquery integration found, creating client anyways.")
164	164	print('Please ensure you have selected a BigQuery '
165		- 'account in the Kernels Settings sidebar.')
	165	+ 'account in the Notebook Add-ons menu.')
166	166	if explicit_project_id is None:
167	167	Log.info("No project specified while using the unmodified client.")
168	168	print('Please ensure you specify a project id when creating the client'

0 commit comments

Comments

0 (0)