modern-data-warehouse-dataops icon indicating copy to clipboard operation
modern-data-warehouse-dataops copied to clipboard

Archive synapse samples, update readme to remove "Modern Data Warehouse" terminology, replace with "Medallion"

Open devlace opened this issue 1 year ago • 1 comments

Type of PR

  • Documentation changes

Purpose

  • Updates the main readme to remove references to "Modern Data Warehouse" in preparation for repo rename.
  • Tidied up README
  • Updated diagram to drawio
  • Archived Synapse samples

Does this introduce a breaking change? If yes, details on what can break

Author pre-publish checklist

  • [ ] Added test to prove my fix is effective or new feature works
  • [X] No PII in logs
  • [X] Made corresponding changes to the documentation

Validation steps

  • ...

Issues Closed or Referenced

  • Closes #issue_number
  • References #issue_number

devlace avatar Oct 11 '24 04:10 devlace

@alexott can you help here

krupakar1329 avatar Aug 22 '24 10:08 krupakar1329

We're also seeing this issue. After a bit of digging, it also seems the API itself (https://docs.databricks.com/api/azure/account/workspaces/update) is not functioning as expected.

Scenario: We have a workspace that is linked to an existing dummy NCC (...21bb). We like to change this NCC to a production NCC (...915e). We tried to do this through Terraform resulting in the issue as listed above.

When running the following CURL request:

curl  -X PATCH \
  'https://accounts.azuredatabricks.net/api/2.0/accounts/<ACCOUNT_ID>/workspaces/<WORKSPACE_ID>' \
  --header 'Accept: */*' \
  --header 'Authorization: Bearer <REDACTED_BEARER_TOKEN>' \
  --header 'Content-Type: application/json' \
  --data-raw '{
  "network_connectivity_config_id": "...915e"
}'

We're getting the following response:

200 OK

<!doctype html>
<html lang="en">
  <head>
    <meta charset="utf-8">
    <meta name="viewport" content="width=device-width,initial-scale=1">
    <meta name="theme-color" content="#000000">
    <meta name="description" content="Databricks Sign in">
    <title>Databricks - Sign in</title>
    <script>window.__DATABRICKS_CONFIG__={"isCuttingEdge":false,"publicPath":{"accounts-console":"https://databricks-ui-assets.azureedge.net/"}}</script>
    <link rel="icon"
      href="https://databricks-ui-assets.azureedge.net/favicon.ico">
    <script defer
      src="https://databricks-ui-assets.azureedge.net/static/js/8843.9334ce07.js"></script>
    <script defer
      src="https://databricks-ui-assets.azureedge.net/static/js/802.55bbb00f.js"></script>
    <script defer
      src="https://databricks-ui-assets.azureedge.net/static/js/9806.4bdefc97.js"></script>
    <script>
  function setNoCdnAndReload() {
      document.cookie = `x-databricks-cdn-inaccessible=true; path=/; max-age=86400`;
      const metric = 'cdnFallbackOccurred';
      const browserUserAgent = navigator.userAgent;
      const browserTabId = window.browserTabId;
      const performanceEntry = performance.getEntriesByType('resource').filter(e => e.initiatorType === 'script').slice(-1)[0]
      sessionStorage.setItem('databricks-cdn-fallback-telemetry-key', JSON.stringify({ tags: { browserUserAgent, browserTabId }, performanceEntry}));
      window.location.reload();
  }
</script>
    <script>
  // Set a manual timeout for dropped packets to CDN
  function loadScriptWithTimeout(src, timeout) {
     return new Promise((resolve, reject) => {
        const script = document.createElement('script');
          script.defer = true;
          script.src = src;
          script.onload = resolve;
          script.onerror = reject;
          document.head.appendChild(script);
          setTimeout(() => {
              reject(new Error('Script load timeout'));
          }, timeout);
      });
  }
  loadScriptWithTimeout('https://databricks-ui-assets.azureedge.net/static/js/login.10991e62.js', 10000).catch(setNoCdnAndReload);
</script>
    <link
      href="https://databricks-ui-assets.azureedge.net/static/css/8843.013d6ea4.css"
      rel="stylesheet">
    <link
      href="https://databricks-ui-assets.azureedge.net/static/css/9806.ec03b64f.css"
      rel="stylesheet">
    <link
      href="https://databricks-ui-assets.azureedge.net/static/css/login.5a13953a.css"
      rel="stylesheet">
  </head>
  <body>
    <noscript>
      You need to enable JavaScript to run this app.
    </noscript>
    <div id="login"></div>
    <script>const telemetryEndpoint="/telemetry-unauth?t=",uiModuleName="accountsConsoleLogin";function shouldIgnoreError(e){return!1}function generateUuidV4(){const e=window.crypto?.randomUUID?.();return e||"xxxxxxxx-xxxx-4xxx-yxxx-xxxxxxxxxxxx".replace(/[xy]/g,(e=>{const n=16*Math.random()|0;return("x"===e?n:3&n|8).toString(16)}))}function networkConnectivityTags(){const e=window.navigator.onLine,n=window.navigator.connection?.rtt??-1,t=window.navigator.connection?.downlink??-1;return{browserNavigatorOnline:e,browserConnectionEstimatedRtt:n,browserConnectionEstimatedDownlink:t,browserConnected:e&&n>0&&t>0}}function createTelemetryRequestBody(e,n={},t=null){const o=Math.round(Date.now()/1e3),r={eventId:generateUuidV4(),metric:e,tags:{...n,...networkConnectivityTags(),browserTabId:window.browserTabId,browserUserAgent:navigator.userAgent},ts:o};return t&&(r.blob=t),JSON.stringify({uploadTime:o,items:[JSON.stringify(r)]})}function recordTelemetry(e,n={},t=""){const o={method:"POST",credentials:"include",body:createTelemetryRequestBody(e,n,t),headers:{"Content-Type":"application/json","X-Databricks-Org-Id":"AC"}};fetch(telemetryEndpoint+Date.now(),o)}window.__databricks_networkConnectivityTags=networkConnectivityTags,Object.defineProperty(window,"browserTabId",{value:generateUuidV4()}),window.recordTelemetry=recordTelemetry,recordTelemetry("uiInit",{uiModule:uiModuleName,eventId:"init",eventClientSource:uiModuleName,eventType:"init"});let logCount=0;function error_handler(e,n,t,o,r){logCount++>4||shouldIgnoreError(e)||recordTelemetry("uncaughtJsException",{eventType:"jsExceptionV3",jsExceptionMessage:e,jsExceptionSource:n,jsExceptionLineno:t,jsExceptionColno:o,jsExceptionBeforeInit:!0},r&&r.stack&&r.stack.toString())}function sendBeaconOnPageExit(e){if(navigator.sendBeacon){const n=e&&e.type||"unknown",t=(Math.round(Date.now()/1e3),createTelemetryRequestBody("uiInit",{eventType:"pageExitBeforeAppInitComplete",eventName:n,eventClientSource:uiModuleName}));navigator.sendBeacon(telemetryEndpoint+Date.now(),t)}}window.onerror=error_handler,window.onunhandledrejection=function(e){error_handler(String(e.reason),null,null,null,e.reason)},window.addEventListener("beforeunload",sendBeaconOnPageExit),window.addEventListener("unload",sendBeaconOnPageExit),window.addEventListener("pagehide",sendBeaconOnPageExit),window.cleanupAfterAppInit=()=>{window.removeEventListener("beforeunload",sendBeaconOnPageExit),window.removeEventListener("unload",sendBeaconOnPageExit),window.removeEventListener("pagehide",sendBeaconOnPageExit)}</script>
  </body>
</html>

While all seems okay from the 200 OK response, after checking the workspace itself, we see the NCC is not updated to the ...915e NCC, and is still reporting the initial ...21bb dummy NCC. Furthermore, the API itself is responding with a login page (similar to the one that caused the provider to panic), so my take is that it's likely an API issue rather than a provider issue

TMRert avatar Aug 22 '24 13:08 TMRert

it seems a problem with accounts api backend

alexott avatar Aug 22 '24 13:08 alexott

Tried with other api like listing users . Same issue with that as well

krupakar1329 avatar Aug 22 '24 13:08 krupakar1329

We have the same issue since today morning in our terraform jobs that seems to be affecting the account level resources.


│ Error: cannot read mws workspaces: cannot read data mws workspaces: unexpected error handling request: invalid character '<' looking for beginning of value. This is likely a bug in the Databricks SDK for Go or the underlying REST API. Please report this issue with the following debugging information to the SDK issue tracker at https://github.com/databricks/databricks-sdk-go/issues. Request log:
│ ```
│ GET /login?account_id=_REDACTED_&next_url=/api/2.0/accounts/_REDACTED_/workspaces

.....

tunayokumus avatar Aug 22 '24 15:08 tunayokumus

Started seeing an error just like this from @tunayokumus at ~2pm CST on August 21st.

We have the same issue since today morning in our terraform jobs that seems to be affecting the account level resources.


│ Error: cannot read mws workspaces: cannot read data mws workspaces: unexpected error handling request: invalid character '<' looking for beginning of value. This is likely a bug in the Databricks SDK for Go or the underlying REST API. Please report this issue with the following debugging information to the SDK issue tracker at https://github.com/databricks/databricks-sdk-go/issues. Request log:
│ ```
│ GET /login?account_id=_REDACTED_&next_url=/api/2.0/accounts/_REDACTED_/workspaces

.....

awbarbeau avatar Aug 22 '24 21:08 awbarbeau

Databricks SDK had issue with new deployment, rolled out on Aug 21st.

They have reverted the changes, and it should be working fine now.

poornimaputtaswamy avatar Aug 23 '24 00:08 poornimaputtaswamy

I'm seeing API calls succeed as of ~6pm CST today.

awbarbeau avatar Aug 23 '24 00:08 awbarbeau

The issue has indeed been resolved for us, both API and Terraform work as expected again

TMRert avatar Aug 23 '24 10:08 TMRert

Seeing this issue again Planning failed. Terraform encountered an error while generating this plan.╷│ Error: cannot read mws network connectivity config: unexpected error handling request: invalid character '<' looking for beginning of value. This is likely a bug in the Databricks SDK for Go or the underlying REST API. Please report this issue with the following debugging information to the SDK issue tracker at https://github.com/databricks/databricks-sdk-go/issues. Request log:

@alexott @tanmay-db please help here

krupakar1329 avatar Sep 26 '24 10:09 krupakar1329

I'm seeing it too, since approximately 6pm UTC 25th September. Contacted our partner manager first thing this morning.

hashicorp/azurerm provider version: v3.47.0 databricks provider version: v1.52.0 (latest) TF version: 1.9.5 x64

I've also seen what looks like a truncated version of the same error when browsing the Databricks Account page, but I also think I've seen that at other times, so maybe not related.

bentleysj avatar Sep 26 '24 10:09 bentleysj

Yes version v1.52.0 - I've edited my reply.

On Thu, Sep 26, 2024 at 11:27 AM mohi @.***> wrote:

Does it appear in latest databricks provider version as well ?

— Reply to this email directly, view it on GitHub https://github.com/databricks/terraform-provider-databricks/issues/3934#issuecomment-2376558691, or unsubscribe https://github.com/notifications/unsubscribe-auth/AP5Z7X5ABC5JLIXXQXB4SKTZYPOSXAVCNFSM6AAAAABM535K5WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNZWGU2TQNRZGE . You are receiving this because you are subscribed to this thread.Message ID: <databricks/terraform-provider-databricks/issues/3934/2376558691@ github.com>

bentleysj avatar Sep 26 '24 11:09 bentleysj

Same both in TF and cli since this morning

# databricks.exe account network-connectivity list-network-connectivity-configurations

Error: unexpected error handling request: invalid character '<' looking for beginning of value. This is likely a bug in the Databricks SDK for Go or the underlying REST API. Please report this issue with the following debugging information to the SDK issue tracker at https://github.com/databricks/databricks-sdk-go/issues. Request log:

GET /login?account_id=f******&next_url=/api/2.0/accounts/******/network-connectivity-configs
> * Host:
> * Accept: application/json
> * Authorization: REDACTED
> * Referer: https://accounts.azuredatabricks.net/api/2.0/accounts/******/network-connectivity-configs?
> * Traceparent: ******
> * User-Agent: cli/0.222.0 databricks-sdk-go/0.43.0 go/1.21.11 os/windows cmd/account_network-connectivity_list-network-connectivity-configurations sdk-feature/pagination auth/pat
< HTTP/2.0 200 OK

emailisabu avatar Sep 26 '24 11:09 emailisabu

Same both in TF and cli since this morning

# databricks.exe account network-connectivity list-network-connectivity-configurations

Error: unexpected error handling request: invalid character '<' looking for beginning of value. This is likely a bug in the Databricks SDK for Go or the underlying REST API. Please report this issue with the following debugging information to the SDK issue tracker at https://github.com/databricks/databricks-sdk-go/issues. Request log:

GET /login?account_id=f******&next_url=/api/2.0/accounts/******/network-connectivity-configs
> * Host:
> * Accept: application/json
> * Authorization: REDACTED
> * Referer: https://accounts.azuredatabricks.net/api/2.0/accounts/******/network-connectivity-configs?
> * Traceparent: ******
> * User-Agent: cli/0.222.0 databricks-sdk-go/0.43.0 go/1.21.11 os/windows cmd/account_network-connectivity_list-network-connectivity-configurations sdk-feature/pagination auth/pat
< HTTP/2.0 200 OK

@krupakar1329 @bentleysj @emailisabu this should be resolved now (this was an issue with the underlying API which has since been resolved). could you try again?

annechung avatar Sep 26 '24 20:09 annechung

@annechung Thank you!, the issue has been fixed.

hieuly27g avatar Sep 27 '24 03:09 hieuly27g

Same both in TF and cli since this morning

# databricks.exe account network-connectivity list-network-connectivity-configurations

Error: unexpected error handling request: invalid character '<' looking for beginning of value. This is likely a bug in the Databricks SDK for Go or the underlying REST API. Please report this issue with the following debugging information to the SDK issue tracker at https://github.com/databricks/databricks-sdk-go/issues. Request log:

GET /login?account_id=f******&next_url=/api/2.0/accounts/******/network-connectivity-configs
> * Host:
> * Accept: application/json
> * Authorization: REDACTED
> * Referer: https://accounts.azuredatabricks.net/api/2.0/accounts/******/network-connectivity-configs?
> * Traceparent: ******
> * User-Agent: cli/0.222.0 databricks-sdk-go/0.43.0 go/1.21.11 os/windows cmd/account_network-connectivity_list-network-connectivity-configurations sdk-feature/pagination auth/pat
< HTTP/2.0 200 OK

@krupakar1329 @bentleysj @emailisabu this should be resolved now (this was an issue with the underlying API which has since been resolved). could you try again?

Hi, thanks. Confirmed to be working for me.

bentleysj avatar Sep 27 '24 08:09 bentleysj

Same both in TF and cli since this morning

# databricks.exe account network-connectivity list-network-connectivity-configurations

Error: unexpected error handling request: invalid character '<' looking for beginning of value. This is likely a bug in the Databricks SDK for Go or the underlying REST API. Please report this issue with the following debugging information to the SDK issue tracker at https://github.com/databricks/databricks-sdk-go/issues. Request log:

GET /login?account_id=f******&next_url=/api/2.0/accounts/******/network-connectivity-configs
> * Host:
> * Accept: application/json
> * Authorization: REDACTED
> * Referer: https://accounts.azuredatabricks.net/api/2.0/accounts/******/network-connectivity-configs?
> * Traceparent: ******
> * User-Agent: cli/0.222.0 databricks-sdk-go/0.43.0 go/1.21.11 os/windows cmd/account_network-connectivity_list-network-connectivity-configurations sdk-feature/pagination auth/pat
< HTTP/2.0 200 OK

@krupakar1329 @bentleysj @emailisabu this should be resolved now (this was an issue with the underlying API which has since been resolved). could you try again?

Hi, thanks. Confirmed to be working for me.

Everything is working fine now. Hopefully, this issue won’t happen again, and the product team will avoid breaking the underlying API in the future

emailisabu avatar Sep 27 '24 08:09 emailisabu

fixed

alexott avatar Jan 06 '25 07:01 alexott