text:
Interactive bash does not start properly in Windows Insider's built-in SSH shell

Your Windows build number:  Microsoft Windows [Version 10.0.17017.1000]

What you're doing and what's happening:
Enabled built-in SSH server, then:

ssh user@windows-host
user@windows-host's password
Microsoft Windows [Version 10.0.17017.1000]
(c) 2017 Microsoft Corporation. All rights reserved.
C:\Users\user>bash -c "echo hello"
hello
C:\Users\user>bash
]0;user@DESKTOP-XXXXXX: /mnt/c/Users/user[01;32mrbalint@DESKTOP-TL0ATTV[00m:[01;34m/mnt/c/Users/user[00m$

Here i can't type anything and the shell does not respond

What's wrong / what should be happening instead:
The interactive shell should work like when just executing commands with the -c option.

Got same issue here.
It seems win 10 host its own ssh service for all insider version user.
Disable these 2 service on win10:
SSH Server Broker
SSH Server Proxy
It's not related to WSL, but it's really dumb to enable SSH service( with cmd ) to all userㄡ

I tried this on a new build and it worked for me, can you reopen a new issue if you're still having issues?

source: gharchive/issue

authors: benhillis rbalint xyzith

license: MIT

license_type: permissive

license_source: github-api


text:
Caption button in title bar Not clickable when ExtendContentintoTitlebar is given true

Describe the bug
The caption buttons are not clickable when the window is maximized. It works when the window is opened , it stops working after being maximized .
Steps to reproduce the bug
1.When activating a window set ExtendContentintoTitlebar  property to true then activate  the window
2. After the window is opened maximize the window the caption buttons are not clickable
Expected behavior
The caption button to work  even after maximizing the window
Screenshots
No response
NuGet package version
No response
Packaging type
Packaged (MSIX)
Windows version
Windows 10 version 21H2 (19044, November 2021 Update)
IDE
Visual Studio 2022
Additional context
Nuget version 1.01

Hello @vishal2447,
You need to check if title bar customization is supported before calling AppWindowTitleBar.ExtendsContentIntoTitleBar. The reason for that is because titlebar customization with WinAppSDK windowing APIs only works on Win11+. See more here.

Will these APIs be supported for Windows 10 in future?

m_window = new MainWindow();
m_window.Title = "Main Window";
m_window.ExtendsContentIntoTitleBar = true;
I m using this property for extending the content and not using the AppWindowTitleBar.ExtendsContentIntoTitleBar , Is this property also supported only in WIndows 11?

No, WinUI's Window.ExtendsContentIntoTitleBar works on both win10 and win11 and is a good alternative until we are able to bring AppWindowTitleBar API support further downlevel.

Yes ExtendsContentIntoTitleBar works on Windows 10. It was working correctly on WinAppSDK 1.0.0. After WinAppSDK 1.0.1 update Minimize, Maximize, Close buttons clicks but not working on Windows 10.

this should be fixed in both Window custom titlebar as well as AppWindow custom titlebar apis. Both are using the same implementation downlevel.

repo: microsoft/WindowsAppSDK


text:
Error: Failed to analyze MachO file: Unknown load command: 2147483700

appcenter fails with:
Error: Failed to analyze MachO file: Unknown load command: 2147483700
This is server-side analysis I'm quite sure, but you'll start seeing issues about it if people attempt to upload to Test with Xcode 12.2 as the build system.
The XCUITest frameworks and libraries contain an ARM64e slice and two new Load Commands. 0x33 and 0x34
Workaround - remove the arm64e slices from the XCUITest frameworks and dylibs in the Runner.

@palmerc This is very useful info but exactly how does one perform the workaround?

@snoozemoose you can remove the arm64e slice easily enough with the lipo tool. However, AppCenter still has problem with Test Runner's compiled in Xcode 12. So... the work around is to compile your test running with Xcode 11, then copy the *-Runner.app and your Xcode 12 app into a folder and upload the pair. It is not ideal, but does work for us.

@palmerc Thank you, I'll give this a try!

Facing the same issue, Is this fixed now?

When is there an update to this?

When is there an update to this?

Is this error coming from the python macholib package? I recently got an identical error directly from that package which was fixed by https://github.com/ronaldoussoren/macholib/commit/e86c2d8745ef13826df0e4227e2f775edec1ec4f .

Closing as stale.

authors: DmitriyKirakosyan JayakumarKnila palmerc sethrj snoozemoose timroeringDB

repo: microsoft/appcenter-cli


text:
Not able to create new release when access token is given using environment variable

Version 2.6.1
I am trying to publish a new version of app to AppCenter, and the distrubute command fails with error:

Error: failed to create release upload for ....ipa. Missing valid authentication token. Correlation ID: 06e24f3a-a11d-44ca-b100-4c75978d56ef

The same command works with CLI version 2.5.4 (verified by downgrading)
Auth token was provided using environment variable in both cases.

label:Distribute

This could be a side effect of the large changes introduced in #909. We'll circle back with the authors.

authors: jp-andre sameeragrawal


text:
Android SDK Include Custom Session Info with Crashes

Describe the solution you'd like
Please add/allow the ability to include custom information of the doomed session with Crash Reports. The existing getErrorAttachments() is not invoked until the subsequent launch, at which time relevant cache/memory information is no longer available. HockeyApp SDK included getDescription() within its CrashManagerListener which could easily be used for this purpose.
Describe alternatives you've considered
There does not appear to be any public/available functionality within the Android SDK available to replicate the previous behavior.

Hey there!
I've assigned this thread to the product owner of Diagnostics on our team for visibility. She'll take a look at this and investigate accordingly

At the same time don't hesitate to support this :
/// <summary>
/// Collect all header fields for the custom exception report.
/// </summary>
/// <returns>A list which contains the header fields for a log file.</returns>
protected virtual List<string> GetLogHeaders ()

It was present in HockeyApp and it seems not present in AppCenter ? :p

authors: Drenet Zakeelm liszto

repo: microsoft/appcenter


text:
Fully working Auth User Flow

Hey @amchew i was wondering if there is a UserFlow which is fully working with auth?
When trying the default Userflows like Register and Login( default/v2) the login and the register is working, but when pressing the back Button when register or when pressing password forgotten the flow crashes....
Now i wondering if it is possible to add a working Userflow to Auth so the basic functionality is working, and the user won't encounter a crashing bug.
Are custom User flows working at the moment? If so an fully working example would be awesome.
(Also at the moment it's not possible to add Auth or Data to new apps in EUW )

Hi @SOliasS, happy new year! Good to hear from you again. We currently only support the sign-in and sign-up user flow or custom policy. The user flow which you're referring to is the password reset one which we currently don't support, unfortunately. We don’t have an ETA on this feature yet. If it’s a blocking feature for your apps, I’d advise you to use Azure AD B2C and the MSAL SDK directly.
Custom policies are working, but we only support the sign-in and sign-up flow in the custom policy.

@amchew does data work when using the MSAL SDK instead of Auth?

@amchew As it's not possible to switch to msal without breaking data #1561.
Is it possible to mix Auth with msal?
So basically use the working part of Auth and use Msal for the missing part?
Can i get the login Token from auth an pass to Msal?
The missing password reset is really killing the whole solution here...

@amchew happy news for a change ;)
i found a workaround for this issue.
Like i indicated in the last comment i was able to combine Auth with the MSAL SDK.
The Auth returns the errors in the LoginRegister Flow back to the App, where i can catch them and then depending on which error message was thrown i can use msal to open the change password flow, where the user can now change their password...
The problem atm is only that the user has to login again after reseting his password.
With this approach i'm now able to use msal, for all the missing features, while continue using auth for all already integrated features.
P.s. ad b2c /MSAL docu is really horrible/confusing/complex, after crawling throw this mess i really appreciate appcenter.auth approach to make it plug an play like it should have been from the beginning....

i found a workaround for this issue.
Like i indicated in the last comment i was able to combine Auth with the MSAL SDK.

I don't find this work around to be acceptable. The point of AppCenter Auth is not have to deal with the underlying auth process and libraries. This is a big gap for my team.

#1621 Auth is dead

authors: SOliasS amchew askpatrickw


text:
Feature to organize Apps in Folders

Describe the solution you'd like
We have a lot of builds in our My Apps pages and it would be great if we could organize the apps inside folders
Describe alternatives you've considered
Better experience on Categorize apps
Additional context
nothing to say

Thanks for the feedback! We don't have any short term plans to add this functionality but we'll consider this in our future planning.
Linking a few other similar requests:
https://github.com/microsoft/appcenter/issues/1752
https://github.com/microsoft/appcenter/issues/352

authors: gmmsantos winnie


text:
Unintuitive UI/UX for "Advanced Options" in Agent Configuration

What feature would you like to be added?
The Agent Configuration UI in AutogenStudio lacks clarity, particularly for the "Advanced Options" section. As a user, it was not immediately obvious that "Advanced Options" is clickable, given the current theme and design. This caused significant confusion and time delay while trying to locate and utilize the feature.
Steps to Reproduce

Open the Agent Configuration panel in AutogenStudio (dark theme)
Look for the "Advanced Options" section at the bottom of the form.
Observe that there is no visual indicator or affordance to suggest it is clickable or expandable.

Actual Behavior
The "Advanced Options" section appears static and non-interactive, making it difficult for users to identify it as a clickable feature.
Why is this needed?
Expected Behavior

There should be a clear visual cue, such as an arrow or highlighting, to indicate that "Advanced Options" is an interactive element.
The design should be intuitive for new users, avoiding unnecessary confusion.

Screenshot from twitter:

Hi @gagb
Can you please look into https://github.com/microsoft/autogen/discussions/4406
Thanks!

Hi @himanshud2611 ,
Thanks for the note. That is indeed a bug that needs to be fixed.
The issue seems to be with darkmode.  (that section is legible when you switch from dark to light mode).
P.S. An updated version of AutoGen Studio is being written (see here). It is still early but this is where much of the development effort will be focused going forward. To learn more about it, see issue #4006 and discussion #4208 .

authors: gagb himanshud2611 victordibia

repo: microsoft/autogen


text:
Env failure due to lack of support for NAT Gateway

Please provide us with the following information:

This issue is a: (mark with an x)

[X] bug report -> please search issues before submitting
[ ] documentation issue or request
[ ] regression (a behavior that used to work and stopped in a new release)

Issue description
Container Apps Creation with Existing Vnet fails without reason.
Steps to reproduce

Create a Vnet
Create Subnet with NSG defined as in [here](https://learn.microsoft.com/en-us/azure/container-apps/firewall-integration
Create Container apps env, it fails without reason.

Expected behavior [Should Create Container Apps env without issue]
Actual behavior [Container Apps env not getting created.]
Screenshots
If applicable, add screenshots to help explain your problem.
Vnet with subnets

subnet

NSG rules

CAE creation failed.

command used to to create cae
az containerapp env create --name prefix-dev-cae --resource-group prefix-dev-rg --logs-workspace-id workspace-id --logs-workspace-key key --location centralus --logs-destination log-analytics --infrastructure-subnet-resource-id /subscriptions/<sid>/resourceGroups/prefix-dev-rg/providers/Microsoft.Network/virtualNetworks/prefix-dev-vn/subnets/prefix-dev-con-sn --tags app.environment=dev

Additional context
Same behavior both in portal and cli

Today I got the following error
deploymentErrors": "ErrorCode: ManagedEnvironmentApiServerConnectionBlocked, Message: Fail to create managed environment because connection to the managed cluster API server was blocked, it could be caused by UDR, refer to https://go.microsoft.com/fwlink/?linkid=2198255 for more detail.",

Same issue even without a NSG attached to a subnet

@mndeem Seems you have configured virtual network NAT gateway for the subnet which is not supported yet. Can you remove the NAT from subnet and add NSG to try again.
BTW, we are working on to support NAT gateway, we will have private preview soon.

Thanks @chinadragon0515  couple of questions

With out NAT, will the subnet would be able to talk to outside world ?
Does AKS also does not support NAT gateway as of now?
Any time line on when NAT gateway would be supported (at least in preview) for ACA ?

@mnadeem,

Yes
AKS supports NAT gateway. ACA does not support yet due to potential security risk.
We are actively working on this issue but no solid target date to share now.

Thanks a bundle @chinadragon0515

Hi @chinadragon0515
After adding file storage, the revision is in provision state for ever, with out file storage it works fine

I have got storage with two subnets

In storage subnet I have configured the storage account and in container subnet I have configured the aca
Storage subnet nsg

Container subnet nsg

Am I missing anything? Do I have to Add NSG rule to allow container subnet to talk to storage subnet ?

Sent email with detail

Hi @chinadragon0515,
We are expririence the same issue with container app env vnet integration when subnet configured with NAT gateway.
We have stantard tier Redis cache and using Redis firewall rules to grant the access for resources from subnets to Redis based on NAT gateway IP. As it's non-premium tier we can't use vnet integration for Redis. Wondering what other options do we have to grant access to Redis for container apps?
Thanks a lot in advance!

@chinadragon0515
As communicate over the email there is one more issue.
Have storage account in one subnet, and ACA in another subnet, the movement you mount azure files to container app, container app seizes to provision.
Not able to mount storage on conatiner app, seems to be blocker and very critical issue

@strofimo  if you configure NAT on subnet, the env could not be created successfully now. If you met issue without NAT, can you send your sub/rg name/env name/app name/ failed time to acasupport@microsoft.com, so we can check more?

@mnadeem we have received the detail and investigated the issue,
The mount error(13): permission denied is generally caused by cluster can not connect to azure file share.
Can you confirm following points:

Storage access key is right
It seems you are using nsg. Make sure the azure file share address and port 445 is not blocked.
Make sure the azure file share is accessible from the cluster subnet.

@strofimo if you configure NAT on subnet, the env could not be created successfully now. If you met issue without NAT, can you send your sub/rg name/env name/app name/ failed time to acasupport@microsoft.com, so we can check more?

hi @chinadragon0515 , sorry for the delay. Yes we tried with NAT on subnet. We did it because our apps in ACA needs to access on-premise resources so it much easy to configure NAT and whitelist NAT IPs upfront. As it's not possible atm do we have any other way to achive it? Whitelist ACA outbound ip is not very convenient.
Thanks a lot in advance!

@chinadragon0515 is there an issue or ticket we can track the progress on NAT gateway support on ACA? Thanks.

For anyone banging their head against a wall like I've been for the last week, I think I've found a fix...
Investigation
I'm using the AzAPI provider with Terraform, and was using the following TF:
resource "azapi_resource" "ca_env" {
  type      = "Microsoft.App/managedEnvironments@2022-10-01"
  name      = "containerappenvname"
  parent_id = "/subscriptions/xyz/resourceGroups/rg-name"
  location  = "northeurope"

  body = jsonencode({
    properties = {
      appLogsConfiguration = {
        destination = "log-analytics"
        logAnalyticsConfiguration = {
          customerId = "law_workspace_id"
          sharedKey  = "law_primary_shared_key"
        }
      }
      vnetConfiguration = {
        internal               = true
        infrastructureSubnetId = "subnetResourceId"
        dockerBridgeCidr       = "172.16.0.1/16"
        platformReservedCidr   = "172.17.0.0/16"
        platformReservedDnsIP  = "172.17.0.2"
      }
    }
  })

  depends_on = [azurerm_virtual_network.main]

  response_export_values  = ["properties.defaultDomain", "properties.staticIp"]
  ignore_missing_property = false
}

Which would produce the exact same issue as logged by @mnadeem .
I also tested with a NAT Gateway on the subnet (not realising this was not supported), which led to this error:
Message: Fail to create managed environment because outbound connectivity was blocked
I decided to deploy a temporary Container App Env via the portal, and compare the JSON from the produced resources, which highlighted this was not automatically populated:
vnetConfiguration = {
  ...
  outboundSettings = {
    outBoundType = "LoadBalancer"
  }
}

The MS docs don't seem to indicate this is a required attribute, but valid values are documented here.
I've only just had the requirement to start using this, so I can't be sure, but I'd assume at some point this was auto-populuated to be outBoundType = "loadBalancer" and now no longer is, which is why we're being caught out by it.
The Fix
Using the following TF, I've just deployed a Container App Environment successfully:
resource "azapi_resource" "ca_env" {
  type      = "Microsoft.App/managedEnvironments@2022-10-01"
  name      = "containerappenvname"
  parent_id = "/subscriptions/xyz/resourceGroups/rg-name"
  location  = "northeurope"

  body = jsonencode({
    properties = {
      appLogsConfiguration = {
        destination = "log-analytics"
        logAnalyticsConfiguration = {
          customerId = "law_workspace_id"
          sharedKey  = "law_primary_shared_key"
        }
      }
      vnetConfiguration = {
        internal               = true
        infrastructureSubnetId = "subnetResourceId"
        dockerBridgeCidr       = "172.16.0.1/16"
        platformReservedCidr   = "172.17.0.0/16"
        platformReservedDnsIP  = "172.17.0.2"
        outboundSettings = {
          outBoundType = "LoadBalancer"
        }
      }
    }
  })

  depends_on = [azurerm_virtual_network.main]

  response_export_values  = ["properties.defaultDomain", "properties.staticIp"]
  ignore_missing_property = false
}

The interesting thing that also now appears, is on the overview page of the ACA Environment there is now a property in the Essentials part of the resource overview, showing the load balancer static IP.
@chinadragon0515 FYI

FYI - NAT Gateway support is currently in public preview for the Consumption + Dedicated plan structure. Documentation

We are facing this same issue when our vnet's subnet is configured with a route table with a user-defined route to a firewall. We are not using NAT gateway or NSG on the subnet. Is there a resolution for this scenario?

Closing this because NAT gateway is now supported. @l3ender we're tracking your issue via #881.

repo: microsoft/azure-container-apps


text:
Terraform: It is impossible to create AKS via terraform using Azure DevOps service principal

Extension: Terraform
We have service principal credentials in Azure DevOps services. In terraform main file there is a section with "azurerm_kubernetes_cluster" resource. The mandatory fields are:
service_principal {
client_id     = "${var.client_id}"
client_secret = "${var.client_secret}"
}
By security reason there is no possibility to take the client secret from azurerm provider. We can run only terraform scripts via Terraform extension in Azure DevOps services. So, is there any way to create a cluster in our case?
Thank you!

@WindMaster "Can run only terraform scripts via Terraform extension in Azure DevOps services" No you can't We require service endpoint for backend configuration. Current time we are not supporting local backend configuration.
If you still face any issue or blocked please ping ur skypeId on RM_Customer_Queries@microsoft.com. So we have a quick call.

authors: RoshanKumarMicrosoft WindMaster

repo: microsoft/azure-pipelines-extensions


text:
Cancelling the build does not kill the maven process

Required Information
Entering this information will route you directly to the right team and expedite traction.
Question, Bug, or Feature?
Type: Bug
Enter Task Name: Maven
Environment

Server - Azure Pipelines
https://dev.azure.com/semarchy/Build Project/_build/results?buildId=3799

Agent - Hosted agent Agent Pool: "Azure Pipelines" https://dev.azure.com/semarchy/Build Project/_settings/agentqueues?queueId=9&view=jobs

Issue Description
Cancelling the build does not kill the maven task which runs until the timeout kills it
Task logs
Not available
Troubleshooting
Checkout how to troubleshoot failures and collect debug logs: https://docs.microsoft.com/en-us/vsts/build-release/actions/troubleshooting
Error logs
No log available

Closed as duplicate of #14228

Closed as duplicate of #14228

Sorry about the duplicate I thought github didn't create the first one because I received a 404 when I clicked "create"

Sorry about the duplicate I thought github didn't create the first one because I received a 404 when I clicked "create"

authors: egor-bryzgalov sarod

repo: microsoft/azure-pipelines-tasks


text:
Can not connect after changing Microsoft password

Issue Type: Bug
Keep getting a token retrieval error even though I have reauthenticated.
Azure Data Studio version: azuredatastudio 1.28.0 (13e362762762e5cb76a5c8afd2bc650f48c2d3d8, 2021-04-15T00:25:54.683Z)
OS version: Darwin x64 18.7.0
Extensions: none

Hi @lioras, is this for an Azure SQL DB?  Have you tried removing your Microsoft account and then re-adding it?

this is a duplicate of https://github.com/microsoft/azuredatastudio/issues/12637

authors: cssuh lioras

repo: microsoft/azuredatastudio


text:
Database trees no longer expand after recent update

Issue Type: Bug
I just updated Azure Data Studio to 1.39.0 and now none of my database connections show the tree view with tables, views etc. None of them will expand. I click on them, get a spinning cursor while it connects, then nothing is displayed.
Azure Data Studio version: azuredatastudio 1.39.0 (9fdb2161d49b4d4ab6798809faa042a898ca2eae, 2022-08-20T01:46:47.606Z)
OS version: Windows_NT x64 10.0.22000
Restricted Mode: No

System Info

Item
Value

CPUs
11th Gen Intel(R) Core(TM) i7-11850H @ 2.50GHz (16 x 2496)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.73GB (2.74GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (1)

Extension
Author (truncated)
Version

import
Mic
1.5.0

Hi @dbainscp, is this for an azure DB or a local DB?

I encounter the same issues, although it does seem to work for some servers if given enough time. The SQL server instances are all  local on my computer (SQL2017/2019) or on premise (running on dedicated DB servers at our office). None of them are Azure DBs.

@dbainscp could you also include screenshots of the issue?

Here is the zipped logging folder & the console logs:
exthost.zip
vscode-app-1661464134991.log

And here is a screen shot of my connections. The one in the red box should be showing the Tree  below it:

And here is the screen shot. The connection with the red box should be showing the tree but it's not.

Hi @dbainscp , is this happening for all your servers?  Have you tried closing and restarting ADS? seems like that might have fixed it for @woutkroon.  Have you been able to successfully connect to these servers before?

@dbainscp the log files don't appear to show any errors, or even attempts to connect.  Could you please check the Help->Developer Tools->Console UI after repro'ing this bug to confirm there are no error messages there?   All the DevTools failed to load source map errors can be ignored, the relevant errors should say something about Connections or Object Explorer sessions.  Thanks!

Hi @dbainscp , is this happening for all your servers? Have you tried closing and restarting ADS? seems like that might have fixed it for @woutkroon. Have you been able to successfully connect to these servers before?

I have logged out & in multiple times over the past three days. I even uninstalled & reinstalled, same thing. This is happening on every one of my servers, all of which are SQL server. Thanks -

@dbainscp the log files don't appear to show any errors, or even attempts to connect. Could you please check the Help->Developer Tools->Console UI after repro'ing this bug to confirm there are no error messages there? All the DevTools failed to load source map errors can be ignored, the relevant errors should say something about Connections or Object Explorer sessions. Thanks!

Going back into the console, these are the only two errors I see:
Failed to load resource: net::ERR_FILE_NOT_FOUND
ERR [Extension Host] (node:30544) [DEP0005] DeprecationWarning: Buffer() is deprecated due to security and usability issues. Please use the Buffer.alloc(), Buffer.allocUnsafe(), or Buffer.from() methods instead.
(Use azuredatastudio --trace-deprecation ... to show where the warning was created)
console.ts:137 [Extension Host] (node:30544) [DEP0005] DeprecationWarning: Buffer() is deprecated due to security and usability issues. Please use the Buffer.alloc(), Buffer.allocUnsafe(), or Buffer.from() methods instead.
(Use azuredatastudio --trace-deprecation ... to show where the warning was created)
log.

@dbainscp thanks for the additional information!  This is unusual as there should be various messages generated when trying to establish the connection, which could provide additional context on what's happening.  Could you please try create a new connection to one of the servers using the Connection Dialog and see if that generates an error message?  Also, is the product basically working in other areas that don't require a connection, such as an offline SQL file or a Notebook file?

OK this g

@dbainscp thanks for the additional information! This is unusual as there should be various messages generated when trying to establish the connection, which could provide additional context on what's happening. Could you please try create a new connection to one of the servers using the Connection Dialog and see if that generates an error message? Also, is the product basically working in other areas that don't require a connection, such as an offline SQL file or a Notebook file?

OK this gets weirder. When I launch ADS, I can connect to one of my databases. The tree won't expand, but I can open a new query and get data from that connection. However, only one connection will work at a time. Any second connection I click on spins endlessly. I can exit ADS then launch it again, and successfully launch the connection that just spun...as long as it's the first one I try.
I tried creating a new connection and as long as I haven't tried to connect to one of my existing servers in the same session, the connection creates successfully  and the tree works properly. Go figure, makes no sense to me. If I try and create a second connection, same thing, it times out I guess and I have to exit Azure & then I can create a second one that works correctly. With my new connections, I can connect to more than one at a time. Very odd.

This weird.  It sounds like something is causing the SQL Tools Service to get in a bad state and be unable to handle future messages.  You may try to restore back to "1st-launch" state, which would clear out everything in the application cache using a procedure such as following.

Run ADS and backup your connection metadata and server groups from the user settings file.  For example, open the "Open Settings (JSON)" file and copy the  "datasource.connectionGroups" and  "datasource.connections" properties to a temporary file.  Close ADS.
Delete the application cache at %APPDATA%\azuredatastudio
Run ADS and restore your connection metadata.  Restart ADS.  Backup/restore connection metadata is option but would save the hassle to recreate.

If that doesn't work, please verify that the Mssql: Tracing Level is set to All in the user settings and try to reproduce the error.  Then close ADS.  Then open the newest folder in %APPDATA%\azuredatastudio\logs, specifically the exthost1\Microsoft.mssql\sqltools.log file and check for any application errors/exceptions.  Please paste the last several events logged in that file as it could provide an idea if the component was behaving normally.
Thanks for helping investigating and sorry for the inconvenience caused by this bug!

This weird. It sounds like something is causing the SQL Tools Service to get in a bad state and be unable to handle future messages. You may try to restore back to "1st-launch" state, which would clear out everything in the application cache using a procedure such as following.

Run ADS and backup your connection metadata and server groups from the user settings file.  For example, open the "Open Settings (JSON)" file and copy the  "datasource.connectionGroups" and  "datasource.connections" properties to a temporary file.  Close ADS.
Delete the application cache at %APPDATA%\azuredatastudio
Run ADS and restore your connection metadata.  Restart ADS.  Backup/restore connection metadata is option but would save the hassle to recreate.

If that doesn't work, please verify that the Mssql: Tracing Level is set to All in the user settings and try to reproduce the error. Then close ADS. Then open the newest folder in %APPDATA%\azuredatastudio\logs, specifically the exthost1\Microsoft.mssql\sqltools.log file and check for any application errors/exceptions. Please paste the last several events logged in that file as it could provide an idea if the component was behaving normally.
Thanks for helping investigating and sorry for the inconvenience caused by this bug!

I'm not seeing any errors in that log file, at least anything that jumps out ast me. I discovered something accidentally though that also makes no sense. If I have the settings.json file open and on the connection section, when I click on a connection, I get a pop-up that says "unable to write into user settings because the file has unsaved changes. Please save the user settings file first and then try again." If I click save & retry, then try that connection again, it connects & the tree pops up. However, it's short-lived. Exiting ADS & relaunching - the connection still doesn't exhibit the tree.

Been busy all day preparing for my upcoming holidays, but just read up until this point. After reading @dbainscp 's comments, I got curious and had another look.
It seems that I spoke too soon when I said that the issue is resolved for me. When I run ADS, and expand the first (topmost, but not sure if that matters) server connection, the tree for that connection is successfully loaded and I can see the database instances etc. However, when I try to expand another server connection, the issue is still there. The second connection did turn green (indicating the connection is established), but the tree does not expand. Every connection after the second just gives me the progress indicator:

I won't be able to provide more info for the next couple of weeks, because of my absence from work.

I was able to reproduce this in a different environment when connecting to a SQL Server instance over a slower VPN connection.  From what I can tell this appears related to a change https://github.com/microsoft/sqltoolsservice/commit/08c74be51feb0ae1f04652a13fe3dcebe9cf70c6 that updates the STS message dispatcher resulting in out-of-order/dropped events in this situation.  I've posted a PR to revert that change and will try to get an insiders build out for confirmation of the proposed fix.  A private version of STS with this change resolved the issue in my environment.

I was able to reproduce this in a different environment when connecting to a SQL Server instance over a slower VPN connection. From what I can tell this appears related to a change microsoft/sqltoolsservice@08c74be that updates the STS message dispatcher resulting in out-of-order/dropped events in this situation. I've posted a PR to revert that change and will try to get an insiders build out for confirmation of the proposed fix. A private version of STS with this change resolved the issue in my environment.

Thanks very much for the update, I appreciate it.

Same problem.  Downgrading to 1.38 resolved the problem.

@dbainscp @louwill today's insider build has an update that we expect to resolve this issue.  We are planning to release a hotfix today or tomorrow pending test sign-off.  If possible, could you please test the insider build in your repro environment to confirm that the update fixes the problem?  Thanks!  https://azuredatastudio-update.azurewebsites.net/latest/win32-x64-user/insider

@dbainscp @louwill today's insider build has an update that we expect to resolve this issue. We are planning to release a hotfix today or tomorrow pending test sign-off. If possible, could you please test the insider build in your repro environment to confirm that the update fixes the problem? Thanks! https://azuredatastudio-update.azurewebsites.net/latest/win32-x64-user/insider

I installed the insider build on my test box and it appears to be working correctly. I have five DB connections and all of them will expand the tree. I tried several times, closing & reopening and it was still working correctly. I look forward to the hotfix, thanks for the quick turn around :)

The hotfix was just released, so please update to the latest (1.39.1) and if you still see this issue let us know!

The hotfix was just released, so please update to the latest (1.39.1) and if you still see this issue let us know!

Hotfix worked for me! Thanks for the quick turnaround!

Installed the hotfix and everything is working correctly now. Thanks for the quick response, I appreciate it!
David Bains
Analyst/Programmer, Enterprise Application Engineering
Information Technology Services
Cal Poly, San Luis Obispo, CA
Office: 805-756-5849
tech.calpoly.eduhttps://tech.calpoly.edu/

From: Will Green @.>
Sent: Tuesday, August 30, 2022 3:11 PM
To: microsoft/azuredatastudio @.>
Cc: David S. Bains @.>; Mention @.>
Subject: Re: [microsoft/azuredatastudio] Database trees no longer expand after recent update (Issue #20472)
The hotfix was just released, so please update to the latest (1.39.1) and if you still see this issue let us know!
Hotfix worked for me! Thanks for the quick turnaround!
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/azuredatastudio/issues/20472#issuecomment-1232220844, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AILM5NEWSYV6PQSGYY6RPRDV32BIDANCNFSM57REAEKQ.
You are receiving this because you were mentioned.Message ID: @.***>

@dbainscp Did you enable the setting mssql.parallelMessageProcessing when you ran into the issues/.

They are all local, SQL Server databases.
Thanks -
David Bains
Analyst/Programmer, Enterprise Application Engineering
Information Technology Services
Cal Poly, San Luis Obispo, CA
Office: 805-756-5849
tech.calpoly.eduhttps://tech.calpoly.edu/
From: Christopher Suh @.>
Sent: Wednesday, August 24, 2022 5:31 PM
To: microsoft/azuredatastudio @.>
Cc: David S. Bains @.>; Mention @.>
Subject: Re: [microsoft/azuredatastudio] Database trees no longer expand after recent update (Issue #20472)
Hi @dbainscphttps://github.com/dbainscp, is this for an azure DB or a local DB?
—
Reply to this email directly, view it on GitHubhttps://github.com/microsoft/azuredatastudio/issues/20472#issuecomment-1226632570, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AILM5NFBMKISHIWVC5OGLCDV225F3ANCNFSM57REAEKQ.
You are receiving this because you were mentioned.Message ID: @.@.>>

Hi @dbainscp, Previously, we rolled back some changes due to this issue and now we have  re-implemented it, we need you help do some testing using the latest insiders build to see whether the issue still reproduces. Appreciate the help.
@caohai FYI.

authors: Charles-Gagnon alanrenmsft cssuh dbainscp kburtram louwill woutkroon


text:
Notebook Jobs UI does not render

Issue Type: Bug
When I click the "New Notebook Job" button, the form is empty with two buttons at the bottom. Those two buttons are Ok and Cancel.
Extension version: 0.49.0
Azure Data Studio version: azuredatastudio 1.39.1 (7553f799e175f471b7590302dd65c997b838b29b, 2022-08-30T01:11:13.047Z)
OS version: Windows_NT x64 10.0.22000
Restricted Mode: No

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-1065G7 CPU @ 1.30GHz (8 x 1498)

GPU Status
2d_canvas: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: enabledopengl: enabled_onrasterization: enabledskia_renderer: enabled_onvideo_decode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.60GB (2.62GB free)

Process Argv

Screen Reader
no

VM
0%

![Screenshot 2022-10-12 144321](https://user-images.githubusercontent.com/26484126/195423001-a544a6d2-8f48-43f1-a3c6-9e6351b901f7.png)

I downloaded an older version .37, and it works fine there. Meaning some recent changes broke this functionality.

@sqlworldwide the issue has been fixed now, the fix will be included in the December release.

Thank you

authors: alanrenmsft sqlworldwide


text:
Results overlaps a horizontally split window

Running a query of the left window the results will overlap the right window. However, that behavior only appeared once.

Azure Data Studio Version:
Version: 1.5.1 (user setup)
Commit: 6a500715a7f2a9c5466d66b3cc312308bf59846d
Date: 2019-03-16T01:14:53.392Z
VS Code 1.30.1
Electron: 2.0.12
Chrome: 61.0.3163.100
Node.js: 8.9.3
V8: 6.1.534.41
OS: Windows_NT x64 10.0.16299

Steps to Reproduce:
Cannot reproduce the issue. Appeared once.

This should be fixed in the current builds.

authors: Ungerfall kburtram


text:
Alanren/notebook password notice

missed these 2 files in my previous pr

Coverage decreased (-0.07%) to 35.235% when pulling 48f74e112ab415fe3503076b2b9449963f2134a6 on alanren/notebook-password-notice into 20c612d9fb7f4b46b901b99cc49612cdf5284fa5 on main.

source: gharchive/pull-request

authors: alanrenmsft coveralls


text:
Skip fetching access token when opening context menu in OE

This PR fixes https://github.com/microsoft/azuredatastudio/issues/21363
Call to fetch access token can be safely skipped when opening context menu, as it will eventually be fetched when any action is taken on server tree nodes.

Pull Request Test Coverage Report for Build 3879867405

2 of 3 (66.67%)  changed or added relevant lines in 2 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+0.001%) to 42.074%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/sql/workbench/services/objectExplorer/browser/serverTreeActionProvider.ts
0
1
0.0%

Files with Coverage Reduction
New Missed Lines
%

extensions/sql-bindings/src/common/azureFunctionsUtils.ts
1
80.65%

Totals

Change from base Build 3876471857:
0.001%

Covered Lines:
28592

Relevant Lines:
63122

💛  - Coveralls

authors: cheenamalhotra coveralls


text:
CLI deployment filter by service doesn't show results for github repos

Describe the bug:
spk deployment get -s service-name doesn't show results for github repos
To Reproduce:
spk deployment get -s service-name where service-name is the name of a service in a github repository.
Expected behavior:
It shows the correct search result in the table
Seen:
Empty results table
Does this require updates to documentation?:
No

@edaena @samiya So this only affects GitHub repos? If so let's indicate that in the title

Not able to reproduce for github configurations. Closing

authors: andrebriggs edaena samiyaakhtar

repo: microsoft/bedrock


text:
open source compliance: minimum files.

This is about the required and recommended files for open-source repos.

[x] LICENSE
[x] CONTRIBUTING.md
[x] CODE_OF_CONDUCT.md
[x] SECURITY.md
[x] SUPPORT.md
[x] ISSUE_TEMPLATE.md
[x] PULL_REQUEST_TEMPLATE.md

~- [ ] CHANGELOG.md~
~- [ ] PRIVACY.txt~
Decided to not touch "PRIVACY.txt".
CHANGELOG.md will be added with the next major release.

~Remains is to update README.md to accommodate latest changes and follow open-source compliance guidelines.~

[x] README.md

Progress: Remaining is the copyright headers.
Also will defer until the action that checks for copyright headers is completed.

repo: microsoft/bing-search-sdk-for-python


text:
Add CancellationToken to WebSocketClient::ConnectAsync

Here
https://github.com/microsoft/botbuilder-dotnet/blob/86d67b989c331fadc8f54efccf081586fc643c9a/libraries/Microsoft.Bot.Streaming/Transport/WebSocket/WebSocketClient.cs#L106
The cancellation token should be taken in the ConnectAsync methods and passed through to the underlying transport layer. ConnectAsync invokes code in the bot which could take a long time and should be cancel-able.
[enhancement]

Comment didn't work. Removing Stale tag and reopening.

@carlosscastro - are you able to pick this up? Seems like a quick change.

authors: NickEricson cleemullins

repo: microsoft/botbuilder-dotnet


text:
Export LG functions to all actions

Fixes #4730
Refs https://github.com/microsoft/BotFramework-Composer/issues/2413
Refs https://github.com/microsoft/BotFramework-Composer/issues/4958
Refs https://github.com/microsoft/botbuilder-dotnet/issues/3652
Refs #5309
Description
There has been a long ask that to be able to call LG functions in all actions, like SetProperty, IfCondition, ChoiceInput, etc, basically every action. While today the workaround is to use the notions of namespace+exports to manually export LG functions to actions. It requires a manual process and it doesn't auto language fallback, where most users are expecting.
In this change, we inject all LG functions to the dialog's expression engine, thus, it can be called the same way as any other built-in functions.  And we do control the injecting scope in a fine-grain manner, that only the functions in the LG file referred by this particular dialog are visible to this dialog's actions. This will have less chance for a naming conflicts.
If in a case that the LG function name is conflict with built-in functions, we will register lg.name instead of name to be able to refer. This is an approach that aligns with how in current LG system, the naming conflicts are been handled.
Scenarios
With this change, you will be able use LG functions in any actions, for example
// in an IfCondition
{
   "$type": "Microsoft.IfCondition",
   "condition": "=MyTemplateA()"
}
// in a SetProperty
{
  "$type": "Microsoft.SetProperty",
  "value": "=MyTemplateB()"
}
// in ChoiceInput
{
  "$kind": "Microsoft.ChoiceInput",
  "choices": "=MyChoices()"
}

Note for Choices in ChoiceInput
With #5309 (which has made LG into choices), we've already exposing LG functions to choices, with a syntax like "choices":"${MyChoices()}", like the way how SendActivity is been written, which works but actually don't align with the underlying schema that declaring choices is an expression(not an LG field).
In this PR, we support a more aligned approach as "choices":"=MyChoices()", which is the standard expression syntax, and have better align with schema and Composer UX.
Note: this is NOT a breaking change, even to #5039, this is additive.
Specific Changes

Inject LG as functions to all actions
Control the injection scope
Handle the naming conflicts

Testing

Like the overall approach, just not sure you need to put in the dialog scope and it might be worth thinking about a bit in light of the namespace proposal.

Like the overall approach, just not sure you need to put in the dialog scope and it might be worth thinking about a bit in light of the namespace proposal.

Thanks, make sense, replied in context.

        var interrupted = false;

This isn't related to this PR is it?

Refers to: libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/AdaptiveDialog.cs:662 in 08d3ea2. [](commit_id = 08d3ea2714c82a17da554cd482ed4a6933d5ac23, deletion_comment = False)

Pull Request Test Coverage Report for Build 240902

0 of 0   changed or added relevant lines in 0 files are covered.
146 unchanged lines in 11 files lost coverage.
Overall coverage increased (+0.07%) to 78.992%

Files with Coverage Reduction
New Missed Lines
%

/libraries/integration/Microsoft.Bot.Builder.Integration.AspNet.Core/Skills/SkillHttpClient.cs
1
95.0%

/libraries/Microsoft.Bot.Builder.LanguageGeneration/Analyzer.cs
3
96.1%

/libraries/Microsoft.Bot.Builder.LanguageGeneration/StaticChecker.cs
3
98.11%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Actions/EmitEvent.cs
6
70.97%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/AdaptiveDialogBot.cs
6
80.33%

/libraries/Adapters/Microsoft.Bot.Builder.Adapters.Slack/SlackHelper.cs
8
86.31%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Generators/TemplateEngineLanguageGenerator.cs
9
72.6%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Input/OAuthInput.cs
9
52.68%

/libraries/Microsoft.Bot.Builder.Dialogs.Adaptive/Actions/SignOutUser.cs
10
54.55%

/libraries/AdaptiveExpressions/FunctionUtils.cs
13
95.03%

Totals

Change from base Build 234300:
0.07%

Covered Lines:
18304

Relevant Lines:
23172

💛  - Coveralls

Closing this one. Related issue is closed too


text:
[PORT] Only use resource.id as the default dialog id if the dialog doesn't have an id.

Port this change from botbuilder-dotnet/master branch:
https://github.com/microsoft/botbuilder-dotnet/pull/4314

Fixes #4250
Description
only apply resource.id as default .id if there is no id in the incoming json payload.
Specific Changes
Change conditional for auto assigning dialog.id
Testing
added unit test to verify.
Changed projects

Microsoft.Bot.Builder.Dialogs.Declarative
Microsoft.Bot.Builder.Dialogs.Declarative.Tests

The qna recognizer stuff will be blocked without this feature. so mark it as P0

repo: microsoft/botbuilder-js


text:
fix: export default orchestrator bot component

Fixes https://github.com/microsoft/botbuilder-js/issues/3598

Pull Request Test Coverage Report for Build 771917813

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+1.3%) to 84.466%

Files with Coverage Reduction
New Missed Lines
%

libraries/botbuilder-lg/src/templateExtensions.ts
1
84.48%

Totals

Change from base Build 771517078:
1.3%

Covered Lines:
18931

Relevant Lines:
21333

💛  - Coveralls

authors: coveralls joshgummersall


text:
Calculation error in simple operation with parenteshis

Describe the bug
Some simple operations with parenthesis are wrong. For example: 100 * (100 -20) = 8000 but Windows calculator returns 80.
Steps To Reproduce
Open the calculator, I tried in scientific mode. Test the above equation from the keyboard, keys: 1, 0, 0, +, (, 1, 0, 0, -, 2, 0, ), intro
Expected behavior
The result should be 8000, not 80
Screenshots

Device and Application Information

OS Build: 10.0.19042.0
Architecture: X64
Application Version: 10.2103.8.0
Region: es-ES
Dev Version Installed: False

Additional context

Requested Assignment
I'm just reporting this problem.  I don't want to fix it.

This is your friendly Microsoft Issue Bot. I've seen this issue come in and have gone to tell a human about it.

is this problem fixed? I checked it out, when the equal button is clicked I got the correct output... :)

Yep, it's currently working. It looks like I didn't in fact pressed intro key on my keyboard, which it's like clicking in equal button. However, IMHO, I find the UX a bit weird. I think that showing the result of a part of the expression, in place of the whole expression, seems a bit confusing, because some PC calculators will show the entire expression result if the expressions is valid (i.e. doesn't have unclosed parenthesis or so), without need for pressing intro or the equal button.

repo: microsoft/calculator


text:
Fix erroneous check for empty trusted stack.

The check in pop_trusted_stack_frame needs to compare the trusted stack offset against the offset of frames field in TrustedStack to check whether we have reached the top of the trusted stack  (which grows upwards). This is the same as the check in handle_error.
Spotted because tests were raising several exceptions before eventually exiting due to failure to spot empty trusted stack.

This makes me suspicious of the decrement of the frame offset at line 689 because it does not have a similar check...

The one at 689 looks fine, it's looking inside the current trusted stack frame to find something.  This must exist because if it didn't then we wouldn't be running any code.  We should be checking for underflow when we pop (which, apparently, we're doing incorrectly).

Hmm, interesting.  We probably should push a trusted stack frame when we call into thread_exit.

Or maybe prevent popping to the dummy stack frame and call thread_exit earlier?

switcher re-enters error handler and this time detects fault in switcher (via PCC base) so attempts to unwind again. Now there is no trusted stack frame to unwind so call scheduler's thread_exit. Note that thread_exit runs with trusted_stack_frameoffset == TrustedStack_offset_frames.

Hmm, I'm not sure that's actually the case.  The scheduler's thread-exit is not an entry point, it's just an exception code in the scheduler's exception handler.  The exception handler is not allowed to make cross-compartment calls or to trap (if it does, it should die).  While it's running, the trusted stack register should be null, which means that entering the switcher via the exception path is treated as a double fault.  Once we have a mechanism for performing a reset, that path will reset the CPU.

To see what happened added a trap to the scheduler in the thread_exit case. The switcher immediately noticed the null mtdc and entered an exception loop. I guess that's fine as the scheduler is already trusted for availability.

Yup, that's the expected behaviour.  Once we have a way of asserting reset from the core, that's what we'll do.

repo: microsoft/cheriot-rtos


text:
considers Newtonsoft.Json to be available on Core, but not on Desktop.

I ran it on the desktop version of the dll in this package:
https://www.nuget.org/packages/Microsoft.AspNet.WebApi.Client/5.2.3
It says it's 100% compatible with .NET Standard and .NET Core - which is great. But it says it's only 89.7% compatible with .NET Framework. Which makes no sense as it's a .NET Framework asset.
It looks like it considers Newtonsoft.Json to be available on Core, but not on Desktop.

I have the same problem.

Closing as API Port was deprecated in favor of binary analysis in .NET Upgrade Assistant.

authors: danmoseley terrajobst ustcltx

repo: microsoft/dotnet-apiport


text:
fix: AdditionalAttributes in FluentSliderLabel component not working

🐛 Bug Report
Use of AdditionalAttributes with the FluentSliderLabel component doesn't affect thoses attributes to the fluent-slider-label component.
💻 Repro or Code Sample
<FluentSlider Min=0 Max=2 Step=1 @bind-Value="_model.ProcessToRun">
    <FluentSliderLabel Position=0 AdditionalAttributes="sliderLabelAttr">
        Data Load (MRI Acturial)
    </FluentSliderLabel>

    <FluentSliderLabel Position=1>
        MRI - Accounting
    </FluentSliderLabel>

    <FluentSliderLabel Position=2>
        Post to MRI Sub-Ledger
    </FluentSliderLabel>
</FluentSlider>

and
private Dictionary<string, object> sliderLabelAttr = new() { { "disabled", true } };

🤔 Expected Behavior
The disabled property should be affected to the fluent-slider-label.
😯 Current Behavior
The fluent-slider-label has the default disabled property affected and not my overload.

💁 Possible Solution
Bind the disabled property directly with a proper Parameter into the FluentSliderLabel component.
🔦 Context
Can't disable a step with my slider component so I can't guide the user into a proper business process.
🌍 Your Environment

OS & Device: Windows
Browser Edge
.NET and FAST Version .NET 6 / FAST 1.1.0

After further investigation, seems that FluentSlider can't interprets disabled labels, even if FluentSliderLabel has a disabled attribute affected.
Closing this.

repo: microsoft/fast-blazor


text:
Expose Button properties

Platforms Impacted

[ ] iOS
[X] macOS

Description of changes
Although Mac design toolkit is still work in progress, let's expose some of the fundamental properties that FluentUI React Native will need to customize.
Let's expose:
contentTintColor - button content tint color
restBackgroundColor - button background color
cornerRadius - button corner radius
Verification
Added customized button test cases to demo app:

Tested exposed props in JS

Pull request checklist
This PR has considered:

[X] Light and Dark appearances
[ ] VoiceOver and Keyboard Accessibility
[ ] Internationalization and Right to Left layouts
[ ] Different resolutions (1x, 2x, 3x)
[ ] Size classes and window sizes (iPhone vs iPad, notched devices, multitasking, different window sizes, etc)

Microsoft Reviewers: Open in CodeFlow

Might be a stretch, but it would be cool if we added  section where there is just the one custom button and you could customize various properties and see it reflected on the button, kinda like what we have for DatePicker. Totally not a blocker though

authors: Saadnajmi chiuam

repo: microsoft/fluentui-apple


text:
instantaneous_ops_per_sec is not valid

Describe the bug
instantaneous_ops_per_sec return 100 times less value than the number of processed operations per second (with pipeline and w/o):
instantaneous_ops_per_sec: 923821
# time memtier_benchmark -s n01-test-podman -p6379 -Predis -c 10 -t 100 -n 1000000 --ratio 1:2 --data-size-pattern=S --key-minimum=100 --key-maximum=100000 --key-pattern=G:G --key-stddev=2000 --key-median=10000 --hide-histogram -x 1 --pipeline=64
Writing results to stdout
[RUN #1] Preparing benchmark client...
[RUN #1] Launching threads now...
[RUN #1 100%,  10 secs]  0 threads:  1000000000 ops, 95205155 (avg: 93434567) ops/sec, 6.49GB/sec (avg: 6.37GB/sec),  0.66 (avg:  0.67) msec latency

100       Threads
10        Connections per thread
1000000   Requests per client

ALL STATS
============================================================================================================================
Type         Ops/sec     Hits/sec   Misses/sec    Avg. Latency     p50 Latency     p99 Latency   p99.9 Latency       KB/sec
----------------------------------------------------------------------------------------------------------------------------
Sets     34719080.15          ---          ---         0.67346         0.44700         4.44700        19.07100   2593747.51
Gets     69437951.99  69416703.95     21248.03         0.67309         0.44700         4.44700        19.07100   4847748.81
Waits           0.00          ---          ---             ---             ---             ---             ---          ---
Totals  104157032.14  69416703.95     21248.03         0.67322         0.44700         4.44700        19.07100   7441496.31

Steps to reproduce the bug

configure Garnet to output metrics per second
run test operations
view and compare metrics and test results

Expected behavior
Garnet metrics show values similar to load test
Screenshots
No response
Release version
v1.0.33
IDE
No response
OS version
Debian 11
Additional context
# cat /opt/garnet/etc/garnet.conf
{
	"Port" : 6379,
	"Address" : null,
	"MemorySize" : "100g",
	"PageSize" : "32m",
	"SegmentSize" : "1g",
	"IndexSize" : "128m",
	"IndexMaxSize": "",
	"MutablePercent" : 90,
	"ObjectStoreHeapMemorySize" : "",
	"ObjectStoreLogMemorySize" : "32m",
	"ObjectStorePageSize" : "1m",
	"ObjectStoreSegmentSize" : "32m",
	"ObjectStoreIndexSize" : "16m",
	"ObjectStoreIndexMaxSize": "",
	"ObjectStoreMutablePercent" : 90,
	"EnableStorageTier" : true,
	"CopyReadsToTail" : false,
	"ObjectStoreCopyReadsToTail" : false,
	"LogDir" : "/data",
	"CheckpointDir" : null,
	"Recover" : true,
	"DisablePubSub" : false,
	"EnableIncrementalSnapshots" : false,
	"PubSubPageSize" : "4k",
	"DisableObjects" : true,
	"EnableCluster" : false,
	"CleanClusterConfig" : false,
	"AuthenticationMode" : "NoAuth",
	"Password" : null,
	"ClusterUsername" : null,
	"ClusterPassword" : null,
	"AclFile" : null,
	"AadAuthority" : "https://login.host.com",
	"AadAudiences" : null,
	"AadIssuers" : null,
	"AuthorizedAadApplicationIds" : null,
	"AadValidateUsername": false,
	"EnableAOF" : false,
	"AofMemorySize" : "64m",
	"AofPageSize" : "4m",
	"AofReplicationRefreshFrequencyMs": 10,
	"SubscriberRefreshFrequencyMs": 0,
	"CommitFrequencyMs" : 0,
	"WaitForCommit" : false,
	"AofSizeLimit" : "",
	"CompactionFrequencySecs" : 300,
	"CompactionType" : "Lookup",
	"CompactionForceDelete": false,
	"CompactionMaxSegments" : 32,
	"ObjectStoreCompactionMaxSegments" : 32,
	"EnableLua" : false,
	"LuaTransactionMode" : false,
	"GossipSamplePercent" : 100,
	"GossipDelay" : 5,
	"ClusterTimeout" : 60,
	"ClusterTlsClientTargetHost" : "GarnetTest",
	"EnableTLS" : false,
	"CertFileName" : null,
	"CertPassword" : null,
	"CertSubjectName" : null,
	"CertificateRefreshFrequency" : 0,
	"ClientCertificateRequired" : true,
	"ServerCertificateRequired" : true,
	"CertificateRevocationCheckMode" : "NoCheck",
	"IssuerCertificatePath" : "",
	"LatencyMonitor" : true,
	"MetricsSamplingFrequency" : 1,
	"QuietMode" : false,
	"LogLevel" : "Trace",
	"LoggingFrequency" : "10",
	"DisableConsoleLogger" : false,
	"FileLogger" : null,
	"ThreadPoolMinThreads" : 0,
	"ThreadPoolMaxThreads" : 0,
	"UseAzureStorage" : false,
	"AzureStorageConnectionString" : null,
	"CheckpointThrottleFlushDelayMs" : 0,
	"EnableFastCommit" : true,
	"FastCommitThrottleFreq" : 1000,
	"NetworkSendThrottleMax" : 8,
	"EnableScatterGatherGet" : false,
	"ReplicaSyncDelayMs" : 5,
	"MainMemoryReplication" : false,
	"OnDemandCheckpoint" : false,
	"UseAofNullDevice" : false,
	"UseNativeDeviceLinux" : true,
	"RevivBinRecordSizes" : null,
	"RevivBinRecordCounts" : null,
	"RevivifiableFraction" : 1.0,
	"EnableRevivification" : false,
	"RevivNumberOfBinsToSearch" : 0,
	"RevivBinBestFitScanLimit" : 0,
	"RevivInChainOnly" : false,
	"RevivObjBinRecordCount" : 256,
	"ObjectScanCountLimit" : 1000,
	"ExtensionBinPaths": null,
	"ExtensionAllowUnsignedAssemblies": false,
	"IndexResizeFrequencySecs": 60,
	"IndexResizeThreshold": 50,
	"LoadModuleCS": null
}

It might be the case that the sampling window is too small to capture the actual number of ops per sec. Try using a MetricsSamplingFrequency=5.
In general, the accuracy of this metric is not that great because it depends on the sampling frequency.
We made that choice to avoid putting a heavy burden on the server while it gathers telemetry for debugging purposes.

@vazois
This particular metric is not only not very accurate, it is completely incorrect.
Unfortunately, varying MetricsSamplingFrequency (I was set it to 5, 10, 20, 30, 60) has no effect.
In general, there is a problem with metrics, because they do not repeat the metrics of Redis - all previously created monitoring templates (whether Zabbix, Grafana, Icinga, etc.) for Redis are almost completely useless with Garnet. And this is sad.

Can you check the attached pull request to see if that fixes your issue?

@vazois
I checked - it work:

pipeline:

@yrajas
Currently I am testing garnet to replace keydb both in standalone and cluster cases, the main requirements are reliability and speed. The loads are large - 1.5M pubsub channels, 60-100M keys (25% hashes, 52% strings, 21% sets), over 700k ops (w/o pipelines) and grow up.
This is used on multi-core servers, I partially described the case here: https://github.com/microsoft/garnet/issues/415#issuecomment-2303247240

authors: rnz vazois

repo: microsoft/garnet


text:
Fixed #93: servername was not supplied to TLS cfg

A change in fd44003 moved the code for setting p.Host down in the parse function, resulting in an empty p.Host value at the time of the creation of p.TLSConfig. I extracted the TLS parsing into a separate function and moved the TLSConfig creation to the end of the parsing function.
closes #93

@microsoft-github-policy-service agree

thx for opening the issue and PR!

pls update CHANGELOG.md . I will include this fix in the next tag

addressed the comments

Codecov Report

Merging #94 (f02b5ef) into main (f481f92) will decrease coverage by 0.56%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #94      +/-   ##
==========================================
- Coverage   74.64%   74.08%   -0.56%
==========================================
  Files          29       29
  Lines        5592     5592
==========================================
- Hits         4174     4143      -31
- Misses       1175     1204      +29
- Partials      243      245       +2

Impacted Files
Coverage Δ

token_string.go
86.36% <0.00%> (-13.64%)
:arrow_down:

tds.go
71.03% <0.00%> (-1.94%)
:arrow_down:

token.go
64.31% <0.00%> (-1.73%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

repo: microsoft/go-mssqldb


text:
Feature request: support ADO.NET connection strings

It would be very useful if there was a way to pass in an ADO.NET-style connection string instead of separate server/user/pw/etc params.
This is already supported by go-mssqldb, so I'm hoping that the amount of work required to add this feature wouldn't be excessive :)
If this sounds like a good idea, and we can get agreement on how the UX should look (a new flag --connection-string/-S  perhaps), then I can probably work on creating a PR for this feature :)
Thanks,
Eric

thx for opening an issue!
I agree that having switches per parameter isn't a sustainable model. It's a problem across the entire SQL toolset, from powershell cmdlets to bcp to SSMS.
How about following the same pattern as sqlcmd -v where we have a new flag that provides an arbitrary named parameter to append to the connection string?
sqlcmd -p applicationclientid=myid -p "clientcertpath=c:\certfolder\file.cer"

-p isn't a good candidate because it'll be used for print statistics but any available letter would work.

It would be really interesting to be able to pass a JSON string or even the path or URL to a JSON file kind of like a .net app.config file.

the context file sqlcmd creates for "modern" callers is that JSON file.  sqlcmd in back-compat mode will use the default context if SQLCMDSERVER env var is not set and you don't pass -S on the command line. The new flag would be useful in modern mode to configure the context too.

It's a problem across the entire SQL toolset, from powershell cmdlets to bcp to SSMS.

The interesting thing is that the powershell invoke-sqlcmd cmdlet already supports a -ConnectionString argument just like I'm suggesting... 😄
https://learn.microsoft.com/en-us/powershell/module/sqlserver/invoke-sqlcmd?view=sqlserver-ps#-connectionstring

How about following the same pattern as sqlcmd -v where we have a new flag that provides an arbitrary named parameter to append to the connection string?

That could potentially work, assuming that it would be fine to pass only the entire ado.net style connection string without having to specify anything else...
How would you propose handling conflicts between explicit params and the ones contained in this "additional KV string" ?
Having said that, I was hoping for a simple solution that would let the user pass in an ado.net connection string and then the sqlcmd code could hand it off unparsed to go-mssql, rather than something like the above that seems significantly more complicated in terms of having to map the keys into connection params, check for invalid entries, etc etc.
Unless I'm overthinking things, and it would be simple to take the results of StringToStringVarP and splat them into the required connection struct fields ?

That sounds good to me David!
Do you have suggested short-form flag names for both of these? Finding good somewhat-meaningful short names is a challenge given the ones that are already taken lol.
I think that -S for --connection-string seems reasonable, if that sounds ok to you.
I'll probably take a swipe at a PR for the "whole connection string" feature, and then the other one can be a followup. I'm not sure if I have time for both, and my personal usecase is for the former :D
Cheers

@shueybubbles I have the feature nearly done locally :)
I'm running into one issue regarding a unit test, what's a good way for me to ask someone for a little guidance?
I could open up a draft PR with the current work, and add a comment regarding the issue I'm having.

a draft PR is probably the best option

authors: dlevy-msft ericsampson shueybubbles

repo: microsoft/go-sqlcmd


text:
cbl-mariner images don't include useradd

I'm looking to use the cbl-mariner based images as part of an AzDO pipeline to build a Go client library, but AzDO pipelines run useradd as part of initializing a custom build container. The cbl-mariner images don't include useradd by default, so running the container fails.
I've been falling back to using the Debian based images, but would prefer using the cbl-mariner image if possible. According to https://github.com/microsoft/azurelinux/issues/3239, useradd comes from the shadow-utils package. Would it be possible to update the build scripts for the mariner images to include shadow-utils along with the other package dependencies that are installed currently?
On a related note, adding the sudo package would also be helpful in order to make it easier to install other custom dependencies in the container for my build, but I have workarounds I can use if sudo is missing, while there is currently no way to workaround useradd not being present.

I think adding useradd (shadow-utils) is a good spot to improve parity, making it more possible to drop CBL-Mariner into an existing build:
$ docker run -it --rm golang:1.22.1 useradd
Usage: useradd [options] LOGIN [...]
$ docker run -it --rm mcr.microsoft.com/oss/go/microsoft/golang:1.22.1-2 useradd
Usage: useradd [options] LOGIN [...]
$ docker run -it --rm mcr.microsoft.com/oss/go/microsoft/golang:1.22.1-2-cbl-mariner2.0 useradd
[...] exec: "useradd": executable file not found in $PATH: unknown.

I'm not so sure about sudo, at least it can't be justified in the same way:
$ docker run -it --rm golang:1.22.1 sudo
[...] exec: "sudo": executable file not found in $PATH: unknown.
$ docker run -it --rm mcr.microsoft.com/oss/go/microsoft/golang:1.22.1-2 sudo
[...] exec: "sudo": executable file not found in $PATH: unknown.
$ docker run -it --rm mcr.microsoft.com/oss/go/microsoft/golang:1.22.1-2-cbl-mariner2.0 sudo
[...] exec: "sudo": executable file not found in $PATH: unknown.

Our primary objective with these images is that you should be able to switch from upstream/official images to ours without surprises. The other way around matters too, though: if someone uses our images to build forked OSS code and make changes to the OSS code that only work with our images, they wouldn't realize it, and would have problems when they try to contribute the improvements to the OSS code upstream where the official Go images are still used.
I don't think we've formally thought out that sequence of events for our images in particular, but it's important to us when we consider making various Go compiler/toolset changes, so I think it makes sense here too. /cc @microsoft/golang-compiler

We just got a separate report that AzDO assumes groupadd is installed, another util provided by this package.

The images on MAR have been updated to include this package--tested on these tags:
mcr.microsoft.com/oss/go/microsoft/golang:1.21-cbl-mariner2.0
mcr.microsoft.com/oss/go/microsoft/golang:1.21-fips-cbl-mariner2.0
mcr.microsoft.com/oss/go/microsoft/golang:1.22-cbl-mariner2.0
mcr.microsoft.com/oss/go/microsoft/golang:1.22-fips-cbl-mariner2.0

Thanks for pointing this out. 👍

I'm also adding su for the same reasons (upstream has it and AzDO tries to use it), provided by util-linux.

authors: dagood danegsta

repo: microsoft/go


text:
Now using pyparsing and supporting infix notation!

This is a complete rewrite of the underlying parser of the Guidance language. It now uses pyparsing which is more feature rich and widely used compared to parsimonious. It is fully backward compatible with Handlebars, but it also now supports more pythonic syntax as well, like infix operators :)
Hopefully this will set up the Guidance language with a good stable and rich syntax going forward.

can't wait to see a new release!

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@d6b855a). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #230   +/-   ##
=======================================
  Coverage        ?   70.10%
=======================================
  Files           ?       49
  Lines           ?     2853
  Branches        ?        0
=======================================
  Hits            ?     2000
  Misses          ?      853
  Partials        ?        0

authors: codecov-commenter shamy1997 slundberg

repo: microsoft/guidance


text:
ENH: Add normalized confusion matrix key to metrics keys

Add normalized confusion matrix key to metrics keys

Add normalized confusion matrix key to metrics keys

repo: microsoft/hi-ml


text:
tscTask() does not support mutually exclusive build option

For better or worse, TypeScript chose to make the --build CLI parameter mutually exclusive from all other parameters, including the project parameter. Further, TypeScript requires the use of the --build parameter in order to respect the composite: true and/or references options in the tsconfig.json.
This conflicts with Just's current implementation of tscTask(), which always sets the project if it can successfully find the tsconfig.json.
Proposal: If build is true in the options argument to tscTask(), then skip setting the options.project value itself and instead provide the path to the discovered tsConfigFile as documented in https://www.typescriptlang.org/docs/handbook/project-references.html and party on.

Good catch!

Congrats on making the bug #123... It's being addressed in #128

This has been released on just-scripts 0.21!

authors: ScottHysom kenotron

repo: microsoft/just


text:
fixes a bug where empty query parameters would be passed

I have stopped the release and caught this minor bug in my integration testing.

Looks good.
Just realized we also missed the version bump in #11 so the release won't go through.

authors: andrueastman baywet

repo: microsoft/kiota-abstractions-dotnet


text:
Update Kiota logo with the new logo image

We need to update the Kiota logo replacing the old "Graph logo". The new logo will be shared internally.
Adding this issue here to track the work. Once the logo is ready, I'll update the issue so we can start working on it.

I've repurpose this issue to be only about updating the Kiota logo since we will no longer generate manifest.json so we won't need the icons.

Hey @sebastienlevert , once we have the final version (color and outline), could you please share with the team? Thanks!

Logos are available here.
https://microsoft.sharepoint-df.com/:u:/t/GraphTooling/ESfZW1-XKYZEuDVIKpLsO04BYLavjCsBMbmuATDUkIiaCA?e=czkyRB
https://microsoft.sharepoint-df.com/:i:/t/GraphTooling/ESKj08jSJpNFiX1Zmf8Xj0cBU06mkEwfYFtY0n0372Fe7w?e=bECbp9

@sebastienlevert, can this be closed since the new logos were updated or should we wait until the new extension is out?

Nah it's ok to close ! They were duplicates.

authors: calebkiage maisarissi sebastienlevert

repo: microsoft/kiota


text:
Add note about Node 16 requirement to release notes

Lage v2 uses AbortController which requires either Node 16 or the --experimental-abortcontroller flag in Node 14. This should be mentioned in the changelog and release info.
Other updates:

Mention all breaking changes in the changelog, release notes, and migration page
Link to the release notes and migration guide from the changelog
Add info about how to semi-automatically pass the flag in Node 14
I don't think it's necessary to have release notes include getting started info (presumably someone reading the notes is already using lage), so I replaced that section with breaking change info and a link to the migration guide

Come to think of it, abort-controller the package can be used to polyfill. Perhaps we can do that once again?

repo: microsoft/lage


text:
Add RHEL missing package fail, add warning for nonfatal error code

yum error code 1 means DNF handled an error with installation. Not checking for missing packages could result in the package installation failing silently, since missing packages halt the installation.
This adds a check for missing packages and fails if they are present. Other handled errors result in a warning.

@mcgov I provided all comments already.

derp, forgot to push

authors: mcgov squirrelsc

repo: microsoft/lisa


text:
Changes for KernelCI

Update make to return ExecutableResult
Add function to generate sas token for blob instead of using container wide sas token
Update ansi filter
Example:
Text: "\x1b[?1h\x1b=\rAdd linux-next specific files for 20230221\x1b[m\r\n\r\x1b[K\x1b[?1l\x1b>"
Filter Result: '\x1b=\rAdd linux-next specific files for 20230221\r\n\r\x1b>'
After change: '\rAdd linux-next specific files for 20230221\r\n\r'

@LiliDeng LGTM

authors: adityagesh squirrelsc


text:
Maximise window options not displayed when using ExtendsContentIntoTitleBar

Describe the bug
The maximise control doesn't display the standard Windows 11 maximise target options when hovering over the button when ExtendsContentIntoTitleBar is set to true
Steps to reproduce the bug
Set ExtendsContentIntoTitleBar to true, hover over the maximise button, no options are presented, just the tooltip text "Maximize"
Expected behavior
Should provide the standard Windows 11 maximize target selection tool
Screenshots
https://imgur.com/a/FuLUncT
NuGet package version
WinUI 3 - Windows App SDK 0.8: 0.8.0 (If you're seeing your issue in older previews of WinUI 3, please try this release)
Windows app type

[x] UWP
[ ] Win32

Device form factor
Desktop
Windows version
Insider Build (xxxxx)
Additional context
No response

@pratikone FYI, do you know whats going on here?

Fixed in Windows App SDK 1.0.1, check out the release notes for more info.

authors: StephenLPeters gabbybilka ispysoftware

repo: microsoft/microsoft-ui-xaml


text:
Added gitignored file to control inner loop solution

This should make the inner loop solution easier to use. Instead of changing FeatureAreas.props and then making sure not to check those changes in, we can all just change our personal copy of InnerLoopAreas.props and git should just ignore it automatically.

/azp run


text:
Update TabViewItem to properly support some properties

TabViewItem and its overlay corners are updated to work properly with the following properties: CornerRadius, Background, BorderBrush, BorderThickness.
*Bold = new resources
Background
TabViewItemHeaderBackgroundSelected
BorderBrush
TabView: TabViewBorderBrush, BorderBrush property
*TabViewItemRadiusRenderBorderBrush
TabViewSelectedItemBorderBrush
BorderThickness
TabView: BorderThickness property
*TabViewItemRightRenderThickness
*TabViewItemLeftRenderThickness
TabViewSelectedItemBorderThickness
The two new resources are currently separated, but I think it would be better to have a single one.
There are currently some temporary resources:
*TabViewItemLeftBorderSpacing
*TabViewItemRightBorderSpacing
They should be equal to the left/right border thickness you set. I'll remove them in a future commit and create a thickness to double converter (or does it exist already?)
CornerRadius
TabViewItem: CornerRadius property
*TabViewItemRadiusRenderCornerRadius
The tab corner radius now works properly. If you set 8,8,8,8 all corners will be rounded, and if you set 8,8,0,0 you'll get sharp bottom corners. By default it uses a converted OverlayCornerRadius.
This is only for v2 styles currently.
I'll create a spec for this PR soon™.
Some examples of what can be done now:

In with the obligitory, will these Tabs and TabBar work when using AcrylicBrushes, InApp and HostBackdrop? @chingucoding

I found another solution that works with transparency.
It basically adds another grid for the border with a bottom margin.

@mdtauk it works really well with acrylic

I have a larger commit that changes almost everything in this PR and adds support for more properties. Should I scrap this PR and do a new one?

What do you mean with support for more properties?

What do you mean with support for more properties?

I got the corners to also support BorderThickness and CornerRadius. This required a slightly larger change, but I think it still fits in this PR.

I got the corners to also support BorderThickness and CornerRadius. This required a slightly larger change, but I think it still fits in this PR.

I think it might be good to have that in a single PR, especially if they touch the same file.

It's quite a big change, but I think it's worth it. I still have to work on some things though.
The default look should be the same.

@gabrielconl For larger changes can you please open an issue/discuss go through the contribution guidelines/process outlined here - https://github.com/microsoft/microsoft-ui-xaml/blob/main/docs/contribution_workflow.md#contribution-workflow

Ah, sorry for that. It started as a small change and I kept updating it to work with more things, it didn't really cross my mind to create an issue.

Since this needs more discussion and the PR will possibly be scrapped, I'll close it for now.

authors: chingucoding gabrielconl mdtauk ranjeshj


text:
Strange white mask in diff editor

monaco-editor version: 0.17.1
Browser: Chromium
OS: Windows 10
Steps or JS usage snippet reproducing the issue:
Recently I developed a tool for developers named SmartCommit based on Electron and Vue (https://github.com/Symbolk/SmartCommit ), which takes over the job of git-status, git-diff, git-add, git-commit and git-push commands. I provide two kinds of diff views with monaco diff editor in the two tabs of a modal r: side by side and inline diff.
However, I found a strange bug with it: the default diff view is ok (Figure 1), but when switching to the other diff view, there will be a strange white mask with a close button  (Figure 2) at the top-right corner, which must be clicked to see the diff view  (Figure 3).
Figure 1

Figure 2

Figure 3

Related code are open sourced at https://github.com/Symbolk/SmartCommit, which is easy to compile and inspect. Thanks for any help!

Hi @Symbolk, did you ever figure this out? I am getting the same thing.

Hi @Symbolk, did you ever figure this out? I am getting the same thing.

Unfortunately not.

I just figured it out on my side. By adding a layout() on the editor, I was able to get it to render properly. Hopefully, that will get you working as well.

@mattduffield Seems like it does not work for me. I put two diff editors in two tabs of a Vue componment called SweetModal (https://sweet-modal-vue.adepto.as/). When the modal is open, the editor in the default tab is always ok, but the editor in the other tab is always covered by the white mask.

Hmm, I also have a regular editor and then I allow the user to launch a diff editor. By calling layout(), it now works fine. Previously, it was covered like you are describing. I hope you figure it out.

Do you have a standalone reproducible case? That looks like the diff review which is available when pressing F7 (it is an accessible friendly way to navigate a diff).

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!

authors: Symbolk alexdima mattduffield

repo: microsoft/monaco-editor


text:
Embedded autocomplete (nextEmbedded + registerCompletionItemProvider)

This might be somewhat similar to #732, but it's a much more general issue. I've implemented syntax highlighting for http using setMonarchTokensProvider. It uses nextEmbedded for syntax highlighting of bodies based on the Content-Type header. This works great.
Now I've implemented autocomplete for HTTP headers using registerCompletionItemProvider, which works great as well. However, I was confused why I get header-suggestions inside a JavaScript body. It seems like the autocomplete just ignores the fact that there's a different language embedded.
So maybe this should even be considered a bug? For me it seems like Monaco uses the wrong CompletionItemProvider when it should be using the one of the embedded language.
So am I doing something wrong or is this a feature that just doesn't exist right now? How does VS Code handle that? In the screenshots you can see that the tokens are definitely JavaScript but I still get suggestions from my http completion provider.

Here is our philosophy around embedded languages, let's take HTML as an example since it is so well known and understood.
<html>
<head>
	<style>
		:root { --main-bg-color: brown; }
 	</style>
	<style>
		body { background-color: var(--main-bg-color); }
 	</style>
</head>
<body></body>
</html>

The coloring story is pretty straight-forward, basically because it is non-semantic. We have built infrastructure into Monarch such that parsers can be embedded in other parsers and they have a pop condition, etc. So each content of each <style> tag can be colorized independently. It does not matter what other <style> tags contain, each <style> tag is self-contained from a coloring point of view.
But things get trickier if you begin to consider a simple feature like suggestions. As you can see, it would be expected that in the second <style> tag you get suggestions from the first <style> tag. This indicates that an embedded language is not self-contained in the embedded section. It could even be that there is a <link> tag somewhere above and that would define the CSS variable, etc.
It is therefore our philosophy that rich language features, such as completions, highlights, etc must be provided by the outer language. For the HTML case, there is an HTML completion provider that should figure out where the suggest request is positioned in the HTML document and either return HTML completions or compute CSS completions. This is a complicated story that works only in VS Code because it involves the HTML language server pulling in the typescript language services and the CSS language services in order to offer good quality completions.
But the idea is that the outer language knows a lot about its domain and it should figure out what to do at each position. The editor cannot do anything language agnostic.

authors: Prinzhorn alexdima


text:
[Bug] Bracket Pair colorization is not working

I used @monaco-editor/react
                                        <Editor 			height="100vh" 			defaultLanguage={languageId} 			value={values} 			options={{readOnly: true, minimap: { enabled: false },overviewRulerBorder: false,bracketPairColorization:                          {enabled:true}, "semanticHighlighting.enabled" : true}} 			/>
Everything works fine except bracket color.

Duplicate of #3237

Duplicate of #3449.

authors: IT-CASADO hediet narkreeta


text:
[Feature Request] Request for Editor.focus method optional params support

Context

[X] This issue is not a bug report. (please use a different template for reporting a bug)
[X] This issue is not a duplicate of an existing issue. (please use the search to find existing issues)

Description
It would be great if Editor.focus method supported optional params (preventScroll, focusVisible  - https://developer.mozilla.org/en-US/docs/Web/API/HTMLElement/focus#parameters) and used these params when focusing in editor textarea
The reason:
We are using Editor inside nodes within some canvas and have the feature of auto focusing Editor within newly added node.
As by default preventScroll is false this causes unintentional browser scrolling the document to bring the newly-focused element into view and this is the reason for some UI issues. If Editor.focus supports preventScroll parameter we can easily control browser behavior on focus
Monaco Editor Playground Link
No response
Monaco Editor Playground Code
No response

Related: #4042

Can you provide monaco editor playground example that demonstrates the problem? Thanks!

authors: hediet ivanksh


text:
Copy sources from Versions.props to NuGet.config

TL;DR
We need that all restore sources are located in NuGet.config since internal feeds can only be restored from there. More details including next steps here

@livarcocc @rainersigwald PTAL

Thanks @rainersigwald, looks like I don't have permissions to merge, could you merge once CIs complete (this change shouldn't cause any problems but just to make sure)

/azp run

/azp run

Intermittent test hangs are going to be fixed by #4374 as soon as I can get it working. I think I'll disable the tests for now; they're really annoying: microsoft/msbuild#4501.

Ok, so once #4501 is merged could you re-run the checks again so this PR can get merged please?

Yup, I'm baby-sitting things this morning. Don't let this block further progress--if it becomes the long pole we can merge as is.

/azp run

:shipit: :)

authors: AndyGerlicher jcagme rainersigwald

repo: microsoft/msbuild


text:
Ubuntu 19.10 Updates Crashes

This is what I get ANYTIME I try to update my system (Ubuntu 19.10). Please I need a solution from the community...
OUTPUT:
natus@etApi-CCXC:~$ sudo apt update
Get:1 http://us.archive.ubuntu.com/ubuntu eoan InRelease [2,821 B]
Err:1 http://us.archive.ubuntu.com/ubuntu eoan InRelease
Clearsigned file isn't valid, got 'NOSPLIT' (does the network require authentication?)
Get:2 http://us.archive.ubuntu.com/ubuntu eoan-updates InRelease [2,829 B]
Err:2 http://us.archive.ubuntu.com/ubuntu eoan-updates InRelease
Clearsigned file isn't valid, got 'NOSPLIT' (does the network require authentication?)
Get:3 http://ppa.launchpad.net/gezakovacs/ppa/ubuntu eoan InRelease [2,832 B]
Err:3 http://ppa.launchpad.net/gezakovacs/ppa/ubuntu eoan InRelease
Clearsigned file isn't valid, got 'NOSPLIT' (does the network require authentication?)
Get:4 http://us.archive.ubuntu.com/ubuntu eoan-backports InRelease [2,831 B]
Err:4 http://us.archive.ubuntu.com/ubuntu eoan-backports InRelease
Clearsigned file isn't valid, got 'NOSPLIT' (does the network require authentication?)
Get:5 http://ppa.launchpad.net/linuxgndu/sqlitebrowser/ubuntu eoan InRelease [2,841 B]
Err:5 http://ppa.launchpad.net/linuxgndu/sqlitebrowser/ubuntu eoan InRelease
Clearsigned file isn't valid, got 'NOSPLIT' (does the network require authentication?)
Get:7 http://us.archive.ubuntu.com/ubuntu eoan-security InRelease [2,830 B]
Err:7 http://us.archive.ubuntu.com/ubuntu eoan-security InRelease
Clearsigned file isn't valid, got 'NOSPLIT' (does the network require authentication?)
Hit:8 https://brave-browser-apt-release.s3.brave.com stable InRelease
Get:9 http://us.archive.ubuntu.com/ubuntu eoan-proposed InRelease [2,830 B]
Err:9 http://us.archive.ubuntu.com/ubuntu eoan-proposed InRelease
Clearsigned file isn't valid, got 'NOSPLIT' (does the network require authentication?)
Hit:6 https://packagecloud.io/AtomEditor/atom/any any InRelease
Reading package lists... Done
N: See apt-secure(8) manpage for repository creation and user configuration details.
N: Updating from such a repository can't be done securely, and is therefore disabled by default.
E: The repository 'http://us.archive.ubuntu.com/ubuntu eoan InRelease' is not signed.
E: Failed to fetch http://us.archive.ubuntu.com/ubuntu/dists/eoan/InRelease  Clearsigned file isn't valid, got 'NOSPLIT' (does the network require authentication?)
N: See apt-secure(8) manpage for repository creation and user configuration details.
N: Updating from such a repository can't be done securely, and is therefore disabled by default.
E: The repository 'http://us.archive.ubuntu.com/ubuntu eoan-updates InRelease' is not signed.
E: Failed to fetch http://us.archive.ubuntu.com/ubuntu/dists/eoan-updates/InRelease  Clearsigned file isn't valid, got 'NOSPLIT' (does the network require authentication?)
N: See apt-secure(8) manpage for repository creation and user configuration details.
N: Updating from such a repository can't be done securely, and is therefore disabled by default.
E: The repository 'http://ppa.launchpad.net/gezakovacs/ppa/ubuntu eoan InRelease' is not signed.
E: Failed to fetch http://ppa.launchpad.net/gezakovacs/ppa/ubuntu/dists/eoan/InRelease  Clearsigned file isn't valid, got 'NOSPLIT' (does the network require authentication?)
N: See apt-secure(8) manpage for repository creation and user configuration details.
N: Updating from such a repository can't be done securely, and is therefore disabled by default.
E: The repository 'http://us.archive.ubuntu.com/ubuntu eoan-backports InRelease' is not signed.
E: Failed to fetch http://us.archive.ubuntu.com/ubuntu/dists/eoan-backports/InRelease  Clearsigned file isn't valid, got 'NOSPLIT' (does the network require authentication?)
N: See apt-secure(8) manpage for repository creation and user configuration details.
N: Updating from such a repository can't be done securely, and is therefore disabled by default.
E: The repository 'http://ppa.launchpad.net/linuxgndu/sqlitebrowser/ubuntu eoan InRelease' is not signed.
E: Failed to fetch http://ppa.launchpad.net/linuxgndu/sqlitebrowser/ubuntu/dists/eoan/InRelease  Clearsigned file isn't valid, got 'NOSPLIT' (does the network require authentication?)
E: Failed to fetch http://us.archive.ubuntu.com/ubuntu/dists/eoan-security/InRelease  Clearsigned file isn't valid, got 'NOSPLIT' (does the network require authentication?)
E: The repository 'http://us.archive.ubuntu.com/ubuntu eoan-security InRelease' is not signed.
N: Updating from such a repository can't be done securely, and is therefore disabled by default.
N: See apt-secure(8) manpage for repository creation and user configuration details.
E: Failed to fetch http://us.archive.ubuntu.com/ubuntu/dists/eoan-proposed/InRelease  Clearsigned file isn't valid, got 'NOSPLIT' (does the network require authentication?)
E: The repository 'http://us.archive.ubuntu.com/ubuntu eoan-proposed InRelease' is not signed.
N: Updating from such a repository can't be done securely, and is therefore disabled by default.
N: See apt-secure(8) manpage for repository creation and user configuration details.

Hello there,
I don't think Ubuntu 19.0 is supported for SQL Server on Linux. According to Microsoft docs, only 16.04 is supported. You can see the details here.
Ubuntu 18.04 is supported after making these modifications.
Cheers.

Closing this issue as SQL Server is not supported on Ubuntu 19.0.

Alright thanks very much for the insights

authors: Bignatus amitmsft dbamaster

repo: microsoft/mssql-docker


text:
Sign-BcContainerApp not signing .app properly

Describe the issue
After running Sign-BcContainerApp, I get an output saying that it successfully signed my .app in a docker shared local folder. But when I check to verify that it is digitally signed, the 'Digital Signatures' tab is missing, so it isn't actually signed yet I believe. I want to properly digitally sign my .app package and verify.
Scripts used to create container and cause the issue
$MyAppFile = "C:\ProgramData\BcContainerHelper\Extensions\26233f2adb05\my\AppSource\ContractManagement.app"
$MyPfx = "C:\ProgramData\BcContainerHelper\Extensions\26233f2adb05\my\AppSource\MyCertName.pfx"
$MyPassword = ConvertTo-SecureString "MyPassWord" -AsPlainText -Force
$MyContainerName = "26233f2adb05"

Sign-BcContainerApp -appFile $MyAppFile -pfxFile $MyPfx -pfxPassword $MyPassword -containerName $MyContainerName

Full output of scripts
Signing C:\ProgramData\BcContainerHelper\Extensions\26233f2adb05\my\AppSource\ContractManagement.app
Done Adding Additional Store
Successfully signed: C:\ProgramData\BcContainerHelper\Extensions\26233f2adb05\my\AppSource\ContractManagement.app

Screenshots

Additional context
As you can see from the screenshot, it does not show the 'Digital Signatures' tab as a property.

I don't think you can see the Digital Signatures tab on a signed app unless you have the navsip crypto provider installed on the machine testing this on.
You can use Install-NAVSipCryptoProviderFromBcContainer to grab the crypto provider from a container and install it.

You can test whether it is signed by creating a container and publish the app without the -skipverification flag.

Install-NAVSipCryptoProviderFromBcContainer

I get this error when running that install command:  Container bcserver does not exist
Do I need to have Business Central On-Premise installed for this?

Install-NAVSipCryptoProviderFromBcContainer -containerName

You can test whether it is signed by creating a container and publish the app without the -skipverification flag.

May I get some resources on how to publish a .app to a BC sandbox running on a container, please? (kind of new to the Docker game).
Also, if the signature verification fails (via the method above), what are the likely causes of the failure?

$artifactUrl = Get-BCArtifactUrl -country us
$licenseFile = $LicenseFileSecret.SecretValue | Get-PlainText
$auth = "UserPassword"
$containerName = "bcserver"
$credential = New-Object pscredential -ArgumentList 'admin', $PasswordSecret.SecretValue
New-BCContainer -containerName $containerName `
                -accept_eula `
                -Auth UserPassword `
                -artifactUrl $artifactUrl `
                -Credential $credential `
                -licenseFile $licenseFile
Publish-BcContainerApp -containerName $containerName -appFile $appfile -sync -install

The artifacts above are set to the latest available US sandbox artifacts.
Licensefile and password are taken from a keyvault - but can be hardcoded.

As to likely causes - bad certificate or timestamp server down are the things I have seen - the latter very in-frequent.

As to likely causes - bad certificate or timestamp server down are the things I have seen - the latter very in-frequent.

Thank you Freddy, it was due to a bad cert.

authors: freddydk jdrco

repo: microsoft/navcontainerhelper


text:
Dev compression speedup

TODO:

replace print with logging

@Cjkkkk your comments are very helpful, will fix them in the next pr soon.

repo: microsoft/nni


text:
the augmented_model_path for quantization script needs to be update

Regarding the recent update in quantize.py of onnxruntime,  the augment_model_path has been changed to /tmp dir.

the corresponding augment_model_path in example script should also be updated accordingly.

https://github.com/microsoft/onnxruntime-inference-examples/blob/2b42b442526b9454d1e2d08caeb403e28a71da5f/quantization/image_classification/cpu/run.py#L29
Thanks

what i did to fix it is to update the get_next() function with argument of inputs information:

the data_reader looks don't need to keep the info of augment_model_path, and session/model input information is enough. not sure if anyone has better solution, please update it here
thanks


text:
[Bug]: HAR file not being saved

Playwright version
1.17.0
Operating system
Windows
What browsers are you seeing the problem on?
Chromium, Firefox
Other information
No response
What happened? / Describe the bug
I am trying to record a HAR file with Playwright.
According to the docs, all you need to do is pass RecordHarPath in the BrowserContextOptions and call CloseAsync on the BrowserContext.
With Firefox, this is not creating any HAR file.
With Chromium, it hangs on CloseAsync and never completes.
Code snippet to reproduce your bug
static async Task Main()
{
    var playwright = await Playwright.CreateAsync();

    var currentDir = Directory.GetCurrentDirectory();

    Console.WriteLine("Trying Firefox");
    var firefox = await playwright.Firefox.LaunchAsync(new BrowserTypeLaunchOptions { Headless = false});
    await TryWith(firefox, Path.Combine(currentDir, "firefoxHar.har"));

    Console.WriteLine("Trying Chromium");
    var chromium = await playwright.Chromium.LaunchAsync(new BrowserTypeLaunchOptions { Headless = false });
    await TryWith(chromium, Path.Combine(currentDir, "chromium.har"));
}

static async Task TryWith(IBrowser browser, string harPath)
{
    var context = await browser.NewContextAsync(new BrowserNewContextOptions()
    {
        RecordHarPath = harPath
    });

    var page = await context.NewPageAsync();
    await page.GotoAsync("https://www.microsoft.com");
    var link = await page.QuerySelectorAsync("a#uhf-shopping-cart");
    await link.ClickAsync();

    Console.WriteLine("Closing Context");
    await context.CloseAsync();
    Console.WriteLine("Closing Browser");
    await browser.CloseAsync();
}

Relevant log output
No response

Came to report the same Issue, also not seeing any HAR file, Tried multiple browsers including Chrome and Chromium and tried with and without video recording which does work and saves on CloseAsync. also tried with Dev tools open. would be nice to solve this.
Thanks

Was able to repro. Seems like the "HAR saving logic" is missing when the browser context gets closed.

authors: StevenDenman infinitejest69 mxschmitt

repo: microsoft/playwright-dotnet