Snowflake-Labs / sfquickstarts

Follow along with our tutorials to get you up and running with the Snowflake Data Cloud.
Apache License 2.0
332 stars 630 forks source link

Missing grant statements at step 5 #115

Open sfc-gh-tsteinberg opened 3 years ago

sfc-gh-tsteinberg commented 3 years ago

Describe the bug Missing grant statements at step 5

URL of where you see the bug https://quickstarts.snowflake.com/guide/vhol_snowflake_salesforce_tcrm/index.html?index=..%2F..index#4 To Reproduce Steps to reproduce the behavior: Run the flow. Once running step 6 - you will get errors.

Expected behavior add those two lines: GRANT IMPORTED PRIVILEGES ON DATABASE SAFEGRAPH_STARBUCKS TO ROLE CRM_ANALYST_ROLE;

GRANT IMPORTED PRIVILEGES ON DATABASE WEATHERSOURCE TO ROLE CRM_ANALYST_ROLE;

sfc-gh-jwest commented 3 years ago

@andriesengelbrecht can you take alook at this issue?

sfc-gh-aengelbrecht commented 3 years ago

If the user uses the CRM_ANALYST_ROLE to get the Marketplace dataset the privileges are automatically be granted to the role. I was trying to limit the number of steps for simplicity.

@tomersteinbergsnow were you having issues when you used the CRM_ANALYST_ROLE to get the Marketplace data, or did you use SYSADMIN or ACCOUNTADMIN to get the data?

sfc-gh-tsteinberg commented 3 years ago

I will need to check it out... not sure, I believe that I was using the sysadmin.

On Thu, 9 Sept 2021 at 17:38, Andries @.***> wrote:

If the user uses the CRM_ANALYST_ROLE to get the Marketplace dataset the privileges are automatically be granted to the role. I was trying to limit the number of steps for simplicity.

@tomersteinbergsnow https://protection.greathorn.com/services/v2/lookupUrl/76cf649d-1829-4937-ab9a-88ce7f11a02a/361/e5cda9095f24381150d76804efe78dea30f073cd?domain=github.com&path=/tomersteinbergsnow were you having issues when you used the CRM_ANALYST_ROLE to get the Marketplace data, or did you use SYSADMIN or ACCOUNTADMIN to get the data?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://protection.greathorn.com/services/v2/lookupUrl/41bc2295-8116-47ad-a7f6-861d6d48978c/361/e5cda9095f24381150d76804efe78dea30f073cd?domain=github.com&path=/Snowflake-Labs/sfquickstarts/issues/115, or unsubscribe https://protection.greathorn.com/services/v2/lookupUrl/365c8920-cc5e-4d11-97fc-4b951cc3b499/361/e5cda9095f24381150d76804efe78dea30f073cd?domain=github.com&path=/notifications/unsubscribe-auth/AT25I65SF47W77CQS4SAM63UBDIIXANCNFSM5DDMJBQA . Triage notifications on the go with GitHub Mobile for iOS https://protection.greathorn.com/services/v2/lookupUrl/35e7a4e7-12bc-4a65-905e-2968204ad80c/361/e5cda9095f24381150d76804efe78dea30f073cd?domain=apps.apple.com&path=/app/apple-store/id1477376905 or Android https://protection.greathorn.com/services/v2/lookupUrl/50fc185d-eddd-4a99-98cb-ff7f35dd657f/361/e5cda9095f24381150d76804efe78dea30f073cd?domain=play.google.com&path=/store/apps/details.

--

TOMER STEINBERGSales Engineer Majors Benelux

MOBILE: +31 6 21 17 5430 <+31+6+21+175430> *EMAIL: @.***

jdanielmyers commented 2 years ago

@tomersteinbergsnow is this still an issue? Thanks

sfc-gh-tsteinberg commented 2 years ago

Hi Daniel,

I do not remember. Sorry about that. I'm sure I worked around it.

Best, Tomer.

On Thu, 16 Dec 2021 at 00:22, Daniel Myers @.***> wrote:

@tomersteinbergsnow https://github.com/tomersteinbergsnow is this still an issue? Thanks

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/Snowflake-Labs/sfquickstarts/issues/115#issuecomment-995293110, or unsubscribe https://github.com/notifications/unsubscribe-auth/AT25I6YJIX2IE3J455UMYVLUREPMZANCNFSM5DDMJBQA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

--

TOMER STEINBERGSales Engineer Majors Benelux

MOBILE: +31 6 21 17 5430 <+31+6+21+175430> *EMAIL: @.***