git repository packfiles kept open

Atom uses libgit2 to read git repository.
However, if the repository has packfiles, the packfiles are kept open.
This makes git gc fail to delete the packfiles.
$ git gc
Counting objects: 53635, done.
Delta compression using up to 2 threads.
Compressing objects: 100% (8149/8149), done.
Writing objects: 100% (53635/53635), done.
Total 53635 (delta 30114), reused 53218 (delta 29697)
Unlink of file '.git/objects/pack/pack-0edf695bcfc5349ac8940012a80521b3cba76437.pack' failed. Should I try again? (y/n)

To reproduce,
On Win 7,
Drag a directory with .git folder into Atom
Then run git gc
If you look at Process Explorer, one of the atom.exe is opening files that are git packfiles of that repository.

Not only that - it also slows down console interaction with the git repository to the point where I have to close atom, run git gc, reopen it to continue working.

On Atom 1.3.2 and this is still happening. Same reproduce steps as @linquize.

Be on Windows 7
Open a folder containing a Git repository in Atom.
In a separate terminal window, run git gc.
During the cleanup process, I get "Unlink of file '.git/objects/pack/pack-abcdef' failed."
Click yes to try again, it fails with the same popup.
Close Atom, then click Yes. It completes successfully.

It looks like libgit2 intentionally keeps many packfiles open at once presumably for performance. More investigation is needed.


az ad app permission admin-consent --id   fails in CloudShell

Describe the bug
Issuing the command az ad app permission admin-consent --id  fails with
msrestazure.azure_active_directory : MSI: Failed to retrieve a token from 'http://localhost:50342/oauth2/token' with an error of '400 Client Error: Bad Request for url: http://localhost:50342/oauth2/token'. This could be caused by the MSI extension not yet fully provisioned.
To Reproduce
You need to be Global Admin in AAD
Run the following command in bash cloudshell
serverApplicationSecret="$(openssl rand -base64 30)"
serverApplicationId="$(az ad app create --display-name "Server" --identifier-uris "https://SomeTestServer" --query appId -o tsv)"
az ad app update --id $serverApplicationId --set groupMembershipClaims=All
az ad sp create --id $serverApplicationId
az ad sp credential reset --name $serverApplicationId --password $serverApplicationSecret --credential-description "Password"
az ad app permission add --id $serverApplicationId --api 00000003-0000-0000-c000-000000000000 --api-permissions e1fe6dd8-ba31-4d61-89e7-88639da4683d=Scope 06da0dbc-49e2-44d2-8312-53f166ab848a=Scope 7ab1d382-f21e-4acd-a863-ba3e13f7da61=Role
az ad app permission grant --id $serverApplicationId --api 00000003-0000-0000-c000-000000000000
az ad app permission admin-consent --id  $serverApplicationId
Expected behavior
Command should succeed the exact same command works outside of cloudshell logged in as the same user
Environment summary
CloudShell bash az version 2.0.61

This is by design, because portal doesn't have the token for related AAD resources

Thanks , IMO we should fix this, having things like this not work in CloudShell is not a great experience, if we cannot fix it then at the least we should produce an error message so the user knows what the issue is.

//CC: @jluk

@maertendMSFT who is now cloud shell owner

@maertendMSFT @yugangw-msft
Is this issue expected to be fixed anytime soon? Or is there a work around granting Directory.Read.All to sp?

@josh-sonrai
Workaround is to grant the permission in portal UI.
We will discuss within cloudshell team for this request.

Given the high level of permissions required and the low frequency that these operations are performed the team Cloud Shell is not going to support this scenario.  We recommend that users run this on a local CLI experience.

I started to experience exactly the same issue from local Azure CLI in a WSL\bash session on my machine.  I'm a global administrator of the Azure AD tenant, still I get the following error when running:
az ad app permission admin-consent --id $aksServerApplicationAppId --debug --verbose
it was working fine before, so I think you introduced a regression bug in a recent version of Azure CLI.
Here is the log produced by the command:
Command arguments: ['ad', 'app', 'permission', 'admin-consent', '--id', '09987941-58a8-4bf0-8bad-3a4a4eef17a0', '--debug', '--verbose']
Event: Cli.PreExecute []
Event: CommandParser.OnGlobalArgumentsCreate [<function CLILogging.on_global_arguments at 0x7f7c057078c8>, <function OutputProducer.on_global_arguments at 0x7f7c0523a620>, <function CLIQuery.on_global_arguments at 0x7f7c052510d0>]
Event: CommandInvoker.OnPreCommandTableCreate []
Installed command modules ['acr', 'acs', 'advisor', 'ams', 'appservice', 'backup', 'batch', 'batchai', 'billing', 'botservice', 'cdn', 'cloud', 'cognitiveservices', 'configure', 'consumption', 'container', 'cosmosdb',
'dla', 'dls', 'dms', 'eventgrid', 'eventhubs', 'extension', 'feedback', 'find', 'hdinsight', 'interactive', 'iot', 'iotcentral', 'keyvault', 'kusto', 'lab', 'maps', 'monitor', 'network', 'policyinsights', 'privatedns', 'profile', 'rdbms', 'redis', 'relay', 'reservations', 'resource', 'role', 'search', 'security', 'servicebus', 'servicefabric', 'signalr', 'sql', 'sqlvm', 'storage', 'vm']
Loaded module 'acr' in 0.009 seconds.
Loaded module 'acs' in 0.006 seconds.
Loaded module 'advisor' in 0.005 seconds.
Event: CommandLoader.OnLoadCommandTable []
Loaded module 'ams' in 0.026 seconds.
Loaded module 'appservice' in 0.009 seconds.
Loaded module 'backup' in 0.003 seconds.
Event: CommandLoader.OnLoadCommandTable []
Loaded module 'batch' in 0.018 seconds.
Loaded module 'batchai' in 0.006 seconds.
Loaded module 'billing' in 0.004 seconds.
Loaded module 'botservice' in 0.004 seconds.
Event: CommandLoader.OnLoadCommandTable []
Loaded module 'cdn' in 0.012 seconds.
Loaded module 'cloud' in 0.003 seconds.
Loaded module 'cognitiveservices' in 0.002 seconds.
Loaded module 'configure' in 0.002 seconds.
Loaded module 'consumption' in 0.005 seconds.
Loaded module 'container' in 0.003 seconds.
Loaded module 'cosmosdb' in 0.006 seconds.
Loaded module 'dla' in 0.007 seconds.
Loaded module 'dls' in 0.004 seconds.
Loaded module 'dms' in 0.003 seconds.
Loaded module 'eventgrid' in 0.004 seconds.
Loaded module 'eventhubs' in 0.005 seconds.
Loaded module 'extension' in 0.004 seconds.
Loaded module 'feedback' in 0.003 seconds.
Loaded module 'find' in 0.002 seconds.
Loaded module 'hdinsight' in 0.003 seconds.
Loaded module 'interactive' in 0.001 seconds.
Loaded module 'iot' in 0.008 seconds.
Loaded module 'iotcentral' in 0.004 seconds.
Loaded module 'keyvault' in 0.012 seconds.
Loaded module 'kusto' in 0.005 seconds.
Loaded module 'lab' in 0.005 seconds.
Loaded module 'maps' in 0.004 seconds.
Loaded module 'monitor' in 0.013 seconds.
Loaded module 'network' in 0.039 seconds.
Loaded module 'policyinsights' in 0.003 seconds.
Loaded module 'privatedns' in 0.007 seconds.
Loaded module 'profile' in 0.002 seconds.
Loaded module 'rdbms' in 0.010 seconds.
Loaded module 'redis' in 0.004 seconds.
Loaded module 'relay' in 0.005 seconds.
Loaded module 'reservations' in 0.003 seconds.
Loaded module 'resource' in 0.009 seconds.
Loaded module 'role' in 0.007 seconds.
Loaded module 'search' in 0.003 seconds.
Loaded module 'security' in 0.004 seconds.
Loaded module 'servicebus' in 0.006 seconds.
Loaded module 'servicefabric' in 0.003 seconds.
Loaded module 'signalr' in 0.002 seconds.
Loaded module 'sql' in 0.010 seconds.
Loaded module 'sqlvm' in 0.005 seconds.
Event: CommandLoader.OnLoadCommandTable []
Loaded module 'storage' in 0.039 seconds.
Loaded module 'vm' in 0.025 seconds.
Loaded all modules in 0.396 seconds. (note: there's always an overhead with the first module loaded)
Extensions directory: '/root/.azure/cliextensions'
Found 5 extensions: ['aks-preview', 'dev-spaces-preview', 'eventgrid', 'mesh', 'resource-graph']
Event: CommandLoader.OnLoadCommandTable []
Extensions directory: '/root/.azure/cliextensions'
Loaded extension 'aks-preview' in 0.008 seconds.
Event: CommandLoader.OnLoadCommandTable []
Loaded extension 'dev-spaces-preview' in 0.005 seconds.
Extensions directory: '/root/.azure/cliextensions'
Loaded extension 'eventgrid' in 0.007 seconds.
Extensions directory: '/root/.azure/cliextensions'
Loaded extension 'mesh' in 0.010 seconds.
Extensions directory: '/root/.azure/cliextensions'
Loaded extension 'resource-graph' in 0.005 seconds.
Event: CommandInvoker.OnPreCommandTableTruncate [<function AzCliLogging.init_command_file_logging at 0x7f7c04fc1598>]
az_command_data_logger : command args: ad app permission admin-consent --id {} --debug --verbose
metadata file logging enabled - writing logs to '/root/.azure/commands'.
Event: CommandInvoker.OnPostCommandTableCreate [<function register_global_subscription_argument..add_subscription_parameter at 0x7f7c04f66598>, <function register_ids_argument..add_ids_arguments at 0x7f7c04f28048>]
Event: CommandInvoker.OnCommandTableLoaded []
Event: CommandInvoker.OnPreParseArgs [<function _documentdb_deprecate at 0x7f7c02bfc8c8>]
Event: CommandInvoker.OnPostParseArgs [<function OutputProducer.handle_output_argument at 0x7f7c0523a6a8>, <function CLIQuery.handle_query_parameter at 0x7f7c05251158>, <function register_ids_argument..parse_ids_arguments at 0x7f7c04f280d0>, <function handler at 0x7f7c02ae4a60>]
msrest.universal_http.requests : Configuring retry: max_retries=4, backoff_factor=0.8, max_backoff=90
msrest.universal_http.requests : Configuring retry: max_retries=4, backoff_factor=0.8, max_backoff=90
msrest.async_paging : Paging async iterator protocol is not available for ApplicationPaged
attempting to read file /root/.azure/accessTokens.json as utf-8-sig
adal-python : c6a4c3b6-c453-4df4-899c-9d6b4f2ff475 - Authority:Performing instance discovery: ...
adal-python : c6a4c3b6-c453-4df4-899c-9d6b4f2ff475 - Authority:Performing static instance discovery
adal-python : c6a4c3b6-c453-4df4-899c-9d6b4f2ff475 - Authority:Authority validated via static instance discovery
adal-python : c6a4c3b6-c453-4df4-899c-9d6b4f2ff475 - TokenRequest:Getting token from cache with refresh if necessary.
adal-python : c6a4c3b6-c453-4df4-899c-9d6b4f2ff475 - CacheDriver:finding with query keys: {'_clientId': '...', 'userId': '...'}
adal-python : c6a4c3b6-c453-4df4-899c-9d6b4f2ff475 - CacheDriver:Looking for potential cache entries: {'_clientId': '...', 'userId': '...'}
adal-python : c6a4c3b6-c453-4df4-899c-9d6b4f2ff475 - CacheDriver:Found 14 potential entries.
adal-python : c6a4c3b6-c453-4df4-899c-9d6b4f2ff475 - CacheDriver:Resource specific token found.
adal-python : c6a4c3b6-c453-4df4-899c-9d6b4f2ff475 - CacheDriver:Returning token from cache lookup, AccessTokenId: b'/1FQomTIal2eW70tH0qxARFI5Gd/wseeeiWtJaDzems=', RefreshTokenId: b'0n1XVOlTG0A48+MvrCqKGpVTux+qneM5T/FBdnIMAaA='
msrest.http_logger : Request URL: 'https://graph.windows.net/769ea97c-d6f6-48f3-a452-7d6ef2e3c8cb/applications?$filter=identifierUris%2Fany(s%3As eq '09987941-58a8-4bf0-8bad-3a4a4eef17a0')&api-version=1.6'
msrest.http_logger : Request method: 'GET'
msrest.http_logger : Request headers:
msrest.http_logger :     'Accept': 'application/json'
msrest.http_logger :     'accept-language': 'en-US'
msrest.http_logger :     'User-Agent': 'python/3.6.5 (Linux-4.4.0-18362-Microsoft-x86_64-with-debian-stretch-sid) msrest/0.6.6 msrest_azure/0.6.0 azure-graphrbac/0.60.0 Azure-SDK-For-Python AZURECLI/2.0.62'
msrest.http_logger : Request body:
msrest.http_logger : None
msrest.universal_http : Configuring redirects: allow=True, max=30
msrest.universal_http : Configuring request: timeout=100, verify=True, cert=None
msrest.universal_http : Configuring proxies: ''
msrest.universal_http : Evaluate proxies against ENV settings: True
urllib3.connectionpool : Starting new HTTPS connection (1): graph.windows.net:443
urllib3.connectionpool : https://graph.windows.net:443 "GET /769ea97c-d6f6-48f3-a452-7d6ef2e3c8cb/applications?$filter=identifierUris%2Fany%28s%3As%20eq%20%2709987941-58a8-4bf0-8bad-3a4a4eef17a0%27%29&api-version=1.6 HTTP/1.1" 200 121
msrest.http_logger : Response status: 200
msrest.http_logger : Response headers:
msrest.http_logger :     'Cache-Control': 'no-cache'
msrest.http_logger :     'Pragma': 'no-cache'
msrest.http_logger :     'Content-Type': 'application/json; odata=minimalmetadata; streaming=true; charset=utf-8'
msrest.http_logger :     'Expires': '-1'
msrest.http_logger :     'ocp-aad-diagnostics-server-name': 'gufVld34TPvxFz0+D5Vne/M2hMFOtE9I0Z1tpyyYzJo='
msrest.http_logger :     'request-id': '0d8f49cb-5458-4ba1-854a-c2c5b52f9d73'
msrest.http_logger :     'client-request-id': '48150a7c-87d6-11e9-931e-44850064eb13'
msrest.http_logger :     'x-ms-dirapi-data-contract-version': '1.6'
msrest.http_logger :     'ocp-aad-session-key': 'nxLlqdMFEXCVs0ghruNzg2P6UJABzAaoDo1ClZR5xWIninrZlP9Gt021i8Pmw85TLTKdKERToaEIAfVwsxeBagCocwmMvXKgl-oqa14Ee1HHUeGe4kB2E3mKeYr-igaKyx_xE2oHb4e6wgzp9flhlA.BcAgARAscFd6j8CYBY9aDDq8U1orOcLsYaNThK1kjjs'
msrest.http_logger :     'DataServiceVersion': '3.0;'
msrest.http_logger :     'Strict-Transport-Security': 'max-age=31536000; includeSubDomains'
msrest.http_logger :     'Access-Control-Allow-Origin': ''
msrest.http_logger :     'X-AspNet-Version': '4.0.30319'
msrest.http_logger :     'X-Powered-By': 'ASP.NET'
msrest.http_logger :     'Duration': '380375'
msrest.http_logger :     'Date': 'Wed, 05 Jun 2019 21:10:02 GMT'
msrest.http_logger :     'Content-Length': '121'
msrest.http_logger : Response content:
msrest.http_logger : {"odata.metadata":"https://graph.windows.net/769ea97c-d6f6-48f3-a452-7d6ef2e3c8cb/$metadata#directoryObjects","value":[]}
msrest.async_paging : Paging async iterator protocol is not available for ApplicationPaged
adal-python : 0982e0e0-b827-46f0-93a2-d1cb5d7bce9d - Authority:Performing instance discovery: ...
adal-python : 0982e0e0-b827-46f0-93a2-d1cb5d7bce9d - Authority:Performing static instance discovery
adal-python : 0982e0e0-b827-46f0-93a2-d1cb5d7bce9d - Authority:Authority validated via static instance discovery
adal-python : 0982e0e0-b827-46f0-93a2-d1cb5d7bce9d - TokenRequest:Getting token from cache with refresh if necessary.
adal-python : 0982e0e0-b827-46f0-93a2-d1cb5d7bce9d - CacheDriver:finding with query keys: {'_clientId': '...', 'userId': '...'}
adal-python : 0982e0e0-b827-46f0-93a2-d1cb5d7bce9d - CacheDriver:Looking for potential cache entries: {'_clientId': '...', 'userId': '...'}
adal-python : 0982e0e0-b827-46f0-93a2-d1cb5d7bce9d - CacheDriver:Found 14 potential entries.
adal-python : 0982e0e0-b827-46f0-93a2-d1cb5d7bce9d - CacheDriver:Resource specific token found.
adal-python : 0982e0e0-b827-46f0-93a2-d1cb5d7bce9d - CacheDriver:Returning token from cache lookup, AccessTokenId: b'/1FQomTIal2eW70tH0qxARFI5Gd/wseeeiWtJaDzems=', RefreshTokenId: b'0n1XVOlTG0A48+MvrCqKGpVTux+qneM5T/FBdnIMAaA='
msrest.http_logger : Request URL: 'https://graph.windows.net/769ea97c-d6f6-48f3-a452-7d6ef2e3c8cb/applications?$filter=appId eq '09987941-58a8-4bf0-8bad-3a4a4eef17a0'&api-version=1.6'
msrest.http_logger : Request method: 'GET'
msrest.http_logger : Request headers:
msrest.http_logger :     'Accept': 'application/json'
msrest.http_logger :     'accept-language': 'en-US'
msrest.http_logger :     'User-Agent': 'python/3.6.5 (Linux-4.4.0-18362-Microsoft-x86_64-with-debian-stretch-sid) msrest/0.6.6 msrest_azure/0.6.0 azure-graphrbac/0.60.0 Azure-SDK-For-Python AZURECLI/2.0.62'
msrest.http_logger : Request body:
msrest.http_logger : None
msrest.universal_http : Configuring redirects: allow=True, max=30
msrest.universal_http : Configuring request: timeout=100, verify=True, cert=None
msrest.universal_http : Configuring proxies: ''
msrest.universal_http : Evaluate proxies against ENV settings: True
urllib3.connectionpool : Starting new HTTPS connection (1): graph.windows.net:443
urllib3.connectionpool : https://graph.windows.net:443 "GET /769ea97c-d6f6-48f3-a452-7d6ef2e3c8cb/applications?$filter=appId%20eq%20%2709987941-58a8-4bf0-8bad-3a4a4eef17a0%27&api-version=1.6 HTTP/1.1" 200 2596
msrest.http_logger : Response status: 200
msrest.http_logger : Response headers:
msrest.http_logger :     'Cache-Control': 'no-cache'
msrest.http_logger :     'Pragma': 'no-cache'
msrest.http_logger :     'Content-Type': 'application/json; odata=minimalmetadata; streaming=true; charset=utf-8'
msrest.http_logger :     'Expires': '-1'
msrest.http_logger :     'ocp-aad-diagnostics-server-name': 'NbQQVYWBv5Gk6t5Wd/et+OeyEdbn9/+/KkWVa677TPU='
msrest.http_logger :     'request-id': 'd1b1aaa6-0302-4f04-bfd0-18cebdb5c10e'
msrest.http_logger :     'client-request-id': '48150a7c-87d6-11e9-931e-44850064eb13'
msrest.http_logger :     'x-ms-dirapi-data-contract-version': '1.6'
msrest.http_logger :     'ocp-aad-session-key': 'BwHVg1ghe-CARbULMP-9z2_rfsRAwJsRSw2HcFtLRCTcsZ2Z0e5vZKmF-n3V7tsQD5v-XEBAt2FZ8X2dwCqW1J4_8F2SCBVkmG3uucrWnKfDkOmiIxVm9mr_RCx5EHUtyB_FQ5Qk0cqa47e79Wyz9A.b0Ce-gBG1mFfF8rmePvS8VOzh1vnoGEEni46JQKis1s'
msrest.http_logger :     'DataServiceVersion': '3.0;'
msrest.http_logger :     'Strict-Transport-Security': 'max-age=31536000; includeSubDomains'
msrest.http_logger :     'Access-Control-Allow-Origin': ''
msrest.http_logger :     'X-AspNet-Version': '4.0.30319'
msrest.http_logger :     'X-Powered-By': 'ASP.NET'
msrest.http_logger :     'Duration': '391841'
msrest.http_logger :     'Date': 'Wed, 05 Jun 2019 21:10:02 GMT'
msrest.http_logger :     'Content-Length': '2596'
msrest.http_logger : Response content:
msrest.http_logger : {"odata.metadata":"https://graph.windows.net/769ea97c-d6f6-48f3-a452-7d6ef2e3c8cb/$metadata#directoryObjects","value":[{"odata.type":"Microsoft.DirectoryServices.Application","objectType":"Application","objectId":"9ac26b5a-5657-4a3f-ba20-e94bac376c7f","deletionTimestamp":null,"acceptMappedClaims":null,"addIns":[],"appId":"09987941-58a8-4bf0-8bad-3a4a4eef17a0","applicationTemplateId":null,"appRoles":[],"availableToOtherTenants":false,"displayName":"LeonardAksServerApplication","errorUrl":null,"groupMembershipClaims":"All","homepage":null,"identifierUris":["https://LeonardAksServerApplication.babo.onmicrosoft.com"],"informationalUrls":{"termsOfService":null,"support":null,"privacy":null,"marketing":null},"isDeviceOnlyAuthSupported":null,"keyCredentials":[],"knownClientApplications":[],"logoutUrl":null,"logo@odata.mediaEditLink":"directoryObjects/9ac26b5a-5657-4a3f-ba20-e94bac376c7f/Microsoft.DirectoryServices.Application/logo","logoUrl":null,"mainLogo@odata.mediaEditLink":"directoryObjects/9ac26b5a-5657-4a3f-ba20-e94bac376c7f/Microsoft.DirectoryServices.Application/mainLogo","oauth2AllowIdTokenImplicitFlow":true,"oauth2AllowImplicitFlow":false,"oauth2AllowUrlPathMatching":false,"oauth2Permissions":[{"adminConsentDescription":"Allow the application to access LeonardAksServerApplication on behalf of the signed-in user.","adminConsentDisplayName":"Access LeonardAksServerApplication","id":"652acdae-ba6c-4498-a256-62325f84bfeb","isEnabled":true,"type":"User","userConsentDescription":"Allow the application to access LeonardAksServerApplication on your behalf.","userConsentDisplayName":"Access LeonardAksServerApplication","value":"user_impersonation"}],"oauth2RequirePostResponse":false,"optionalClaims":null,"orgRestrictions":[],"parentalControlSettings":{"countriesBlockedForMinors":[],"legalAgeGroupRule":"Allow"},"passwordCredentials":[{"customKeyIdentifier":"//5BAEsAUwBQAGEAcwBzAHcAbwByAGQA","endDate":"2020-06-05T21:09:54.814205Z","keyId":"e80233d3-309a-47b0-adf7-12aef2efa6f5","startDate":"2019-06-05T21:09:54.814205Z","value":null}],"publicClient":null,"publisherDomain":"babo.onmicrosoft.com","recordConsentConditions":null,"replyUrls":["https://LeonardAksServerApplication.babo.onmicrosoft.com"],"requiredResourceAccess":[{"resourceAppId":"00000003-0000-0000-c000-000000000000","resourceAccess":[{"id":"e1fe6dd8-ba31-4d61-89e7-88639da4683d","type":"Scope"},{"id":"06da0dbc-49e2-44d2-8312-53f166ab848a","type":"Scope"},{"id":"7ab1d382-f21e-4acd-a863-ba3e13f7da61","type":"Role"}]}],"samlMetadataUrl":null,"signInAudience":"AzureADMyOrg","tokenEncryptionKeyId":null}]}
adal-python : 28f89c8a-55ba-414a-95c5-b1b1efedc1cc - Authority:Performing instance discovery: ...
adal-python : 28f89c8a-55ba-414a-95c5-b1b1efedc1cc - Authority:Performing static instance discovery
adal-python : 28f89c8a-55ba-414a-95c5-b1b1efedc1cc - Authority:Authority validated via static instance discovery
adal-python : 28f89c8a-55ba-414a-95c5-b1b1efedc1cc - TokenRequest:Getting token from cache with refresh if necessary.
adal-python : 28f89c8a-55ba-414a-95c5-b1b1efedc1cc - CacheDriver:finding with query keys: {'_clientId': '...', 'userId': '...'}
adal-python : 28f89c8a-55ba-414a-95c5-b1b1efedc1cc - CacheDriver:Looking for potential cache entries: {'_clientId': '...', 'userId': '...'}
adal-python : 28f89c8a-55ba-414a-95c5-b1b1efedc1cc - CacheDriver:Found 14 potential entries.
adal-python : 28f89c8a-55ba-414a-95c5-b1b1efedc1cc - CacheDriver:Resource specific token found.
adal-python : 28f89c8a-55ba-414a-95c5-b1b1efedc1cc - CacheDriver:Returning token from cache lookup, AccessTokenId: b'/1FQomTIal2eW70tH0qxARFI5Gd/wseeeiWtJaDzems=', RefreshTokenId: b'0n1XVOlTG0A48+MvrCqKGpVTux+qneM5T/FBdnIMAaA='
msrest.http_logger : Request URL: 'https://graph.windows.net/769ea97c-d6f6-48f3-a452-7d6ef2e3c8cb/applications/9ac26b5a-5657-4a3f-ba20-e94bac376c7f?api-version=1.6'
msrest.http_logger : Request method: 'GET'
msrest.http_logger : Request headers:
msrest.http_logger :     'Accept': 'application/json'
msrest.http_logger :     'accept-language': 'en-US'
msrest.http_logger :     'User-Agent': 'python/3.6.5 (Linux-4.4.0-18362-Microsoft-x86_64-with-debian-stretch-sid) msrest/0.6.6 msrest_azure/0.6.0 azure-graphrbac/0.60.0 Azure-SDK-For-Python AZURECLI/2.0.62'
msrest.http_logger : Request body:
msrest.http_logger : None
msrest.universal_http : Configuring redirects: allow=True, max=30
msrest.universal_http : Configuring request: timeout=100, verify=True, cert=None
msrest.universal_http : Configuring proxies: ''
msrest.universal_http : Evaluate proxies against ENV settings: True
urllib3.connectionpool : Starting new HTTPS connection (1): graph.windows.net:443
urllib3.connectionpool : https://graph.windows.net:443 "GET /769ea97c-d6f6-48f3-a452-7d6ef2e3c8cb/applications/9ac26b5a-5657-4a3f-ba20-e94bac376c7f?api-version=1.6 HTTP/1.1" 200 2593
msrest.http_logger : Response status: 200
msrest.http_logger : Response headers:
msrest.http_logger :     'Cache-Control': 'no-cache'
msrest.http_logger :     'Pragma': 'no-cache'
msrest.http_logger :     'Content-Type': 'application/json; odata=minimalmetadata; streaming=true; charset=utf-8'
msrest.http_logger :     'Expires': '-1'
msrest.http_logger :     'ocp-aad-diagnostics-server-name': 'wWkc+s8RiEzYhqgyU5nm0K0RUFWoXWB/UrDt0vQszas='
msrest.http_logger :     'request-id': '50c23475-b51a-4cd5-8069-e8f33cfbcba9'
msrest.http_logger :     'client-request-id': '48150a7c-87d6-11e9-931e-44850064eb13'
msrest.http_logger :     'x-ms-dirapi-data-contract-version': '1.6'
msrest.http_logger :     'ocp-aad-session-key': '_nZCrorcIC5kpnC2uiX1f06hhjOS0uMQxWg3rIK_YYnEK9nrJ1lX7OGGn_Yepm9tS-8IFaXs-eJ7zQS72sAh2DuuKyx6EToNPQMTC5fT31zz8ImfNC5SBieBq698tQBzm5xSV6qcjEwpltdvgSEUJA.aosZvrckLTMfvxp34eL3Y6ZEmKsNP2bvWCNzfSXaons'
msrest.http_logger :     'DataServiceVersion': '3.0;'
msrest.http_logger :     'Strict-Transport-Security': 'max-age=31536000; includeSubDomains'
msrest.http_logger :     'Access-Control-Allow-Origin': '*'
msrest.http_logger :     'X-AspNet-Version': '4.0.30319'
msrest.http_logger :     'X-Powered-By': 'ASP.NET'
msrest.http_logger :     'Duration': '418715'
msrest.http_logger :     'Date': 'Wed, 05 Jun 2019 21:10:03 GMT'
msrest.http_logger :     'Content-Length': '2593'
msrest.http_logger : Response content:
msrest.http_logger : {"odata.metadata":"https://graph.windows.net/769ea97c-d6f6-48f3-a452-7d6ef2e3c8cb/$metadata#directoryObjects/@Element","odata.type":"Microsoft.DirectoryServices.Application","objectType":"Application","objectId":"9ac26b5a-5657-4a3f-ba20-e94bac376c7f","deletionTimestamp":null,"acceptMappedClaims":null,"addIns":[],"appId":"09987941-58a8-4bf0-8bad-3a4a4eef17a0","applicationTemplateId":null,"appRoles":[],"availableToOtherTenants":false,"displayName":"LeonardAksServerApplication","errorUrl":null,"groupMembershipClaims":"All","homepage":null,"identifierUris":["https://LeonardAksServerApplication.babo.onmicrosoft.com"],"informationalUrls":{"termsOfService":null,"support":null,"privacy":null,"marketing":null},"isDeviceOnlyAuthSupported":null,"keyCredentials":[],"knownClientApplications":[],"logoutUrl":null,"logo@odata.mediaEditLink":"directoryObjects/9ac26b5a-5657-4a3f-ba20-e94bac376c7f/Microsoft.DirectoryServices.Application/logo","logoUrl":null,"mainLogo@odata.mediaEditLink":"directoryObjects/9ac26b5a-5657-4a3f-ba20-e94bac376c7f/Microsoft.DirectoryServices.Application/mainLogo","oauth2AllowIdTokenImplicitFlow":true,"oauth2AllowImplicitFlow":false,"oauth2AllowUrlPathMatching":false,"oauth2Permissions":[{"adminConsentDescription":"Allow the application to access LeonardAksServerApplication on behalf of the signed-in user.","adminConsentDisplayName":"Access LeonardAksServerApplication","id":"652acdae-ba6c-4498-a256-62325f84bfeb","isEnabled":true,"type":"User","userConsentDescription":"Allow the application to access LeonardAksServerApplication on your behalf.","userConsentDisplayName":"Access LeonardAksServerApplication","value":"user_impersonation"}],"oauth2RequirePostResponse":false,"optionalClaims":null,"orgRestrictions":[],"parentalControlSettings":{"countriesBlockedForMinors":[],"legalAgeGroupRule":"Allow"},"passwordCredentials":[{"customKeyIdentifier":"//5BAEsAUwBQAGEAcwBzAHcAbwByAGQA","endDate":"2020-06-05T21:09:54.814205Z","keyId":"e80233d3-309a-47b0-adf7-12aef2efa6f5","startDate":"2019-06-05T21:09:54.814205Z","value":null}],"publicClient":null,"publisherDomain":"babo.onmicrosoft.com","recordConsentConditions":null,"replyUrls":["https://LeonardAksServerApplication.babo.onmicrosoft.com"],"requiredResourceAccess":[{"resourceAppId":"00000003-0000-0000-c000-000000000000","resourceAccess":[{"id":"e1fe6dd8-ba31-4d61-89e7-88639da4683d","type":"Scope"},{"id":"06da0dbc-49e2-44d2-8312-53f166ab848a","type":"Scope"},{"id":"7ab1d382-f21e-4acd-a863-ba3e13f7da61","type":"Role"}]}],"samlMetadataUrl":null,"signInAudience":"AzureADMyOrg","tokenEncryptionKeyId":null}
Current cloud config:
AzureCloud
adal-python : 1ef94e01-2769-4103-b882-d5f7fee54ba9 - Authority:Performing instance discovery: ...
adal-python : 1ef94e01-2769-4103-b882-d5f7fee54ba9 - Authority:Performing static instance discovery
adal-python : 1ef94e01-2769-4103-b882-d5f7fee54ba9 - Authority:Authority validated via static instance discovery
adal-python : 1ef94e01-2769-4103-b882-d5f7fee54ba9 - TokenRequest:Getting token from cache with refresh if necessary.
adal-python : 1ef94e01-2769-4103-b882-d5f7fee54ba9 - CacheDriver:finding with query keys: {'_clientId': '...', 'userId': '...'}
adal-python : 1ef94e01-2769-4103-b882-d5f7fee54ba9 - CacheDriver:Looking for potential cache entries: {'_clientId': '...', 'userId': '...'}
adal-python : 1ef94e01-2769-4103-b882-d5f7fee54ba9 - CacheDriver:Found 14 potential entries.
adal-python : 1ef94e01-2769-4103-b882-d5f7fee54ba9 - CacheDriver:Resource specific token found.
adal-python : 1ef94e01-2769-4103-b882-d5f7fee54ba9 - CacheDriver:Returning token from cache lookup, AccessTokenId: b'ERXXrDi5W9oU0LL4AVylB7x7T/+1ej4QjekkLIyHCmY=', RefreshTokenId: b'0n1XVOlTG0A48+MvrCqKGpVTux+qneM5T/FBdnIMAaA='
urllib3.connectionpool : Starting new HTTPS connection (1): main.iam.ad.ext.azure.com:443
urllib3.connectionpool : https://main.iam.ad.ext.azure.com:443 "POST /api/RegisteredApplications/09987941-58a8-4bf0-8bad-3a4a4eef17a0/Consent?onBehalfOfAll=true HTTP/1.1" 400 1423
cli.azure.cli.core.util : Bad Request
Bad Request
az_command_data_logger : exit code: 1
telemetry.save : Save telemetry record of length 2521 in cache
telemetry.check : Negative: The /root/.azure/telemetry.txt was modified at 2019-06-05 23:08:51.437353, which in less than 600.000000 s
command ran in 2.379 seconds.

I have the same issue  when using a Service Principal to run az ad app permission admin-consent --id $serverApplicationId. The Service principal has all AAD and Graph Application and Delegate Permissions and still returns an error
az : ERROR: Bad Request
At line:1 char:1

az ad app permission admin-consent --id $serverApplicationId

  + CategoryInfo          : NotSpecified: (ERROR: Bad Request:String) [], RemoteException
  + FullyQualifiedErrorId : NativeCommandError

Whats missing?

Are you the admin in the the tenant, which has the privilege to consent to the permission required by the service principal?

@yugangw-msft  I am the Global Administrator of the tenant.

Can you please capture the trace and send it to me at yugangw at microsoft dot com?

Info sent, thanks!

Per our offline mail thread, this error is caused by you invoked this command under a service principal which is not supported by design

Per our offline mail thread, this error is caused by you invoked this command under a service principal which is not supported by design

Strange I am deploying number of applications per day using azure devops, azure devops service principal has rights to perform necessary registrations and can do almost everything, except this. Maybe we can automate this via ARM deployment? Do you have any plans to support app reg manifests in ARM deployment style? We could solve app reg challenge then... Automation is not complete... right now. For each QA test I need to go to portal and click the button.

The online documentation for this command makes no mention of the MSI requirement. Automation is hindered by this MSI requirement.

This hindrance is affecting my organization's use of AKS. We need to be able to automate cluster deployment from Azure DevOps.

@yugangw-msft can this issue please be re-opened or documented appropriately, we don't understand why we can't do this through a service principal which is assigned the global administrator role and access to it appropriately restricted

CC @jiasli the new owner of CLI/Graph support

move to S166.

@haroldrandom, can we remove the Cloud Shell tag as this been found outside of Cloud Shell per the comments above.

done. Added Graph label. Correct me if wrong.

Please check https://github.com/Azure/azure-cli/issues/12137 and see if you can get unblocked.

Please refer to the detailed description in #12137.   Close this first, if have other problems, please reopen it or submit a new one. Thanks


Backfill prod-stable to main

Release TEAMNADO-4699 to prod stable

Codecov Report
Base: 93.91% // Head: 93.88% // Decreases project coverage by -0.03% :warning:

Coverage data is based on head (93a1e57) compared to base (ea41ce1).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #135      +/-   ##
==========================================
- Coverage   93.91%   93.88%   -0.03%
==========================================
  Files          36       36
  Lines         460      458       -2
  Branches       98       95       -3
==========================================
- Hits          432      430       -2
  Misses         25       25
  Partials        3        3

Impacted Files
Coverage Δ

...components/SubscriptionTable/SubscriptionTable.tsx
87.65% <100.00%> (-0.30%)
:arrow_down:

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Task entity

Pull Request Test Coverage Report for Build 64

0 of 3 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-11.9%) to 83.333%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

keepchronos/src/main/java/com/garritano/keepchronos/model/Task.java
0
3
0.0%

Totals

Change from base Build 62:
-11.9%

Covered Lines:
20

Relevant Lines:
24

💛  - Coveralls


Add 0.15.1 binary

It's out! :smile_cat:

I've extracted the OS X binaries from the pkg and updated the Reactor assets, but I can't get the Linux binaries compiled because of a barrage of cabal errors. Not sure if there's a stackage LTS or something I can build this against, because the hackage head is super broken at the moment. :/

Okay, figured it out, I tried to compile it with musl. Some dependency wasn't happy about that. There doesn't seem to be an elm-doc anymore. So we'll need to update the scripts to reflect that.

That leaves us with Windows binaries.

I've added the Windows binaries in PR #24.

So we can close this one?

Yes, I think this issue can be closed, and a new release of the installer been made with the new binaries.

I've tagged this as v2.0, since we break backwards-comp with the new binary names. I just need @kevva to publish this to npm. :)

@passy, I think you have ownership, no? Anyway, it's published!


docs: add developer documentation

Codecov Report
All modified lines are covered by tests :white_check_mark:

Comparison is base (02cd682) 87.85% compared to head (ddf66b1) 88.06%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #274      +/-   ##
==========================================
+ Coverage   87.85%   88.06%   +0.21%
==========================================
  Files          19       19
  Lines         947      947
==========================================
+ Hits          832      834       +2
+ Misses        115      113       -2

see 2 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


NEW FUNCTION: Send_Alerts.py - Query users ST_Dwithin from a sensor

See workflow 4 - Alerts if len(new_spike_sensors) > 0: Number 2
This function should take in pg_connection_dict and a sensor_id then return a list of record_ids (from "Sign Up Information") that are subscribed to the service and within 1 kilometer of the sensor

Done with f9c68bb


Socks5 support

Is it possible to add Socks5 proxy support ?

The quiche library is agnostic to sockets, so really its up to applications to support SOCKS if they want.
The apps are intended mainly as demos or simple test tools. Adding SOCKS isupport s feasible but probably low on the priority list. A simpler workaround is to use something like socksify that can inject that behaviour rather than require explicit changes - https://linux.die.net/man/1/socksify. Does that solve your use case?

Is it possible to specify my own ssl library then and custom transport protocol?
For example I would use boring-ssl and somehow my own socks library.
I need to add SOCKS support from rust code itself, socksify wouldn't work.
Is there any libraries such as reqwest supporting quiche, allowing to do GET/POST requests?

Swapping tls is feasible but might be complicated depending on how the alternative works.
Quiche parses or create QUIC packets, that apps can read or write over any carriage. You can straightforwardly substitute UDP socket read/write for Rust SOCKS library equivalent and it'll work, I've seen that.
We don't offer integration with reqwest or hyper here. But aware of various work that made that happening by gluing it all together. The low-level of quiche makes it very flexible to different purposes, but requires a little more glue to fit the desired environment.

Thank you!


Xubuntu: InvalidOperationException when debugging with .NET Core Launch (web)

VSCode Version: 1.5.3
OS Version: Xubuntu 16.04

Steps to Reproduce:

Create ASP.NET Core app
Start debugging with ".NET Core Launch (web)".

Every time a request hits the server, the exception is thrown. The browser gets no response (blank screen). Note: There is no exception when attaching to the server (dotnet) process with ".NET Core Attach".
Exception::

Hosting environment: Development
Content root path: /home/osboxes/Coding
Launching browser (xdg-open http://localhost:5000)
Now listening on: http://localhost:5000
Application started. Press Ctrl+C to shut down.
Loaded '/usr/share/dotnet/shared/Microsoft.NETCore.App/1.0.1/System.Threading.Tasks.Extensions.dll'. Cannot find or open the symbol file.
[...]
Loaded '/usr/share/dotnet/shared/Microsoft.NETCore.App/1.0.1/System.Resources.ResourceManager.dll'. Cannot find or open the symbol file.
Exception thrown: 'System.InvalidOperationException' in System.Private.CoreLib.ni.dll
Exception thrown: 'System.InvalidOperationException' in System.Private.CoreLib.ni.dll

Project: https://github.com/blackcity/backload
web: https://www.backload.org

This issue was moved to OmniSharp/omnisharp-vscode#781

@chrmarti this is a problem of the C# debugger. Moving to their repo.


Documents with missing sortBy fields don't show up in the list

Description
Documents with missing sortBy fields don't show up in the collection list
https://www.loom.com/share/244853da315e4f959ebf371d662192dd
Expected result
They should appear at the start, or end of the list (depending on sort order)
Versions
Tinacms Package Versions
├──<EMAIL_ADDRESS>├──<EMAIL_ADDRESS>

@jamespohalloran This was the behaviour we made on purpose because we didn't want to place a default value on every field and wanted to leave that up to the user.

Why do the documents need a default? I don't think most editors will think to update the sort to see all the documents as-is.

The sorted lists are based on an index. The index is built based on a field
(or fields) in a document. If you have a document that has a field foo,
only the documents with that field present will be included when sorting by
that field. The only way for documents to get included, is to set a default
value for that field when building the index, which we explicitly decided
not to do.
On Wed, Nov 2, 2022 at 9:33 AM jamespohalloran @.***>
wrote:

@jamespohalloran https://github.com/jamespohalloran This was the
behaviour we made on purpose because we didn't want to place a default
value on every field and wanted to leave that up to the user.
Why do the documents need a default? I don't think most editors will think
to update the sort to see all the documents as-is.
—
Reply to this email directly, view it on GitHub
https://github.com/tinacms/tinacms/issues/3363#issuecomment-1300413202,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAC2SC3FIB7QNYEWB6HC5U3WGJURXANCNFSM6AAAAAARVB27E4
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Gotcha, even if we could display a label somewhere that certain documents with missing {{sortBy}} may be excluded from this list, that would help.

I don't think you could display that labelly unconditionally unless you
query the full list (with default filename sort) to get the length, and
then compare the full sortedBy list. IMO the correct solution here is to
provide a default value so you always get your full document list
On Wed, Nov 2, 2022 at 9:57 AM jamespohalloran @.***>
wrote:

Gotcha, even if we could display a label somewhere that certain documents
with missing {{sortBy}} may be excluded from this list, that would help.
—
Reply to this email directly, view it on GitHub
https://github.com/tinacms/tinacms/issues/3363#issuecomment-1300454089,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAC2SC6VER7ZNHRRWHQFNGDWGJXLFANCNFSM6AAAAAARVB27E4
.
You are receiving this because you commented.Message ID:
@.***>

I think it would probably have to be a developer configurable default,
since it would need to vary depending on the type of the field
On Wed, Nov 2, 2022 at 10:05 AM jamespohalloran @.***>
wrote:

IMO the correct solution here is to provide a default value so you always
get your full document list
like a default of null or undefined you mean? Just want to make sure we
don't start walking back this change
https://linear.app/tina/issue/ENG-591/change-behaviour-of-defaults,
where field defaults were injected at runtime.
—
Reply to this email directly, view it on GitHub
https://github.com/tinacms/tinacms/issues/3363#issuecomment-1300473769,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAC2SCZ2JTMUIZOA6CGJIXLWGJYJRANCNFSM6AAAAAARVB27E4
.
You are receiving this because you commented.Message ID:
@.***>

@jamespohalloran For some background context I believe we chose this behaviour because other solutions. For example firebase also filter out of the order by field is null.

I think there's a difference between an API behaviour (which has reference documentation) and the editor UI/UX.
I don't want to have to go out of our way too much to fix this medium UX issue, but what about something like this?


Add go.mod for rueidisaside and om packages

Solves https://github.com/redis/rueidis/issues/452

Made the change, go mod tidy added github.com/redis/rueidis v1.0.27 anyways

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (eb3729b) 96.02% compared to head (d26b0eb) 95.98%.
Report is 1 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #456      +/-   ##
==========================================
- Coverage   96.02%   95.98%   -0.05%
==========================================
  Files          75       67       -8
  Lines       32104    31549     -555
==========================================
- Hits        30829    30282     -547
+ Misses       1090     1085       -5
+ Partials      185      182       -3

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Simplify integration test

Turns out the test can run without a mock for the task launcher (I had a hard time understanding why it wouldn't). The inner @Configuration class (even if empty) is needed though.

This is a sanity test to check that autoconfig is working, not an integration test, so we have to use fake creds.  The constructor for tasklauncher makes a connection to CF in order to determine which version of the task launcher to create, we want to circumvent this behavior.


KVS Audio Only Streaming

Hello,
I'm looking to use KVS to stream audio only. Currently able to stream audio using gstreamer plugin on my system with the below command.
gst-launch-1.0 -v kvssink name=sink stream-name="teststream" access-key="" secret-key="" aws-region="us-east-1"  osxaudiosrc ! audioconvert ! avenc_aac ! queue ! sink.
However i'm unable to playback the stream using HLS or DASH because the API expects the stream to have a video encoded in H264 to generate a HLS or DASH streaming url. AWS Enterprise support recommended a work around to send a static image along with the audio stream to playback successfully. However i'm unable to figure out how to do that from the gstreamer docs. Appreciate if someone can help on how to stream audio with a static image ( or a sample video? ) to enable HLS or DASH playback ? Also if this is not possible at all, please let me know, so i don't waste my time in trial and error.

We are facing the same issue. We couldn't find any documentation on how to send audio only.

You have the pipeline elements reversed so the source element needs to come first and then you can send it to kvssink.  Perhaps you ca use autovideosrc or some other dummy video h264 source, then you can merge them together (in your GStreamer pipeline) and then take that result and send it to kvssink.
Alternatively you can use our C SDK here are samples:
https://github.com/awslabs/amazon-kinesis-video-streams-producer-c/blob/master/samples/KvsAacAudioVideoStreamingSample.c
https://github.com/awslabs/amazon-kinesis-video-streams-producer-c/blob/master/samples/KvsVideoOnlyStreamingSample.c
But you need to modify this to be audio only.
But your issue again is for HLS there needs to be a video track, so you would need to use the audio/video sample, see how in our sample we just read the frames from disk, they are in the repo here:
https://github.com/awslabs/amazon-kinesis-video-streams-producer-c/tree/master/samples
You can do something like that just have the same frame you send over and over.  Keep in mind there are costs associated with the bandwidth so adding video will mean higher bandwidth, even if its very compressible it will be non zero.
In the mean time I'll see if there's a way to get an HLS or DASH stream with audio only.

Yeah I've confirmed that is the case today we do not support audio only for HLS/DASH, it is in our roadmap but we have no ETA.
If you'd like to use GStreamer then I'd do what I recommended above (you can even use filesrc to read h264 encoded static frames, you can keep 1 GOP of low bandwidth frame, low quality that you replay over and over again) there are many ways to go about it.  If you need help with that I would reach out to the GStreamer forums basically your goal is to first construct a pipeline which generates the dummy h264 mixed with your aac audio and try to get it to work with filesink.  Once you're able to validate on your end that your pipeline works and you can successfully stream it to a file and play it back with a player like VLC then just substitute filesink with kvssink and you should be good to go.

thanks @hassanctech i will try out steps mentioned above and let u know how it works out. Appreciate the detailed response.

Thanks, closing as there is no Kinesis Video Streams, issue.  If you have a specific bug / question about the KVS SDKs, please open a new issue with those details.

I've confirmed that is the case today we do not support audio only for HLS/DASH, it is in our roadmap but we have no ETA.

@mudassir-mukhtar-ahmed  Did you find a working solution for this? Would be grateful if you can share if so, or share what didn't work.


Add campaign names support for stepchain workflows

Fixes #11706
Status
Ready
Description
Appends all campaigns available in a stepChain workflow job to the WMTask campaign name.
Is it backward compatible (if not, which system it affects?)
YES
Related PRs
<If it's a follow up work; or porting a fix from a different branch, please mention them here.>
External dependencies / deployment changes
<Does it require deployment changes? Does it rely on third-party libraries?>

Here are my test cases:
Structure:
workload Campaign = DMWM_Test
Step1 Campaign = CMSSW_12_4_0_pre2__fullsim_noPU_2021_14TeV-1649399635Step1
Step2 Campaign = CMSSW_12_4_0_pre2__fullsim_noPU_2021_14TeV-1649399635Step2
Step3 Campaign = CMSSW_12_4_0_pre2__fullsim_noPU_2021_14TeV-1649399635Step3

Case 1:
wmagent_SC_EL8_khurtado_sc8v13_noStep2Campaign_230922_201854_9245

Workload campaign = yes
Step 1 campaign= yes
Step 2 campaign= no
Step 3 campaign= yes
Result:
campaignName = 'CMSSW_12_4_0_pre2__fullsim_noPU_2021_14TeV-1649399635Step1,DMWM_Test,CMSSW_12_4_0_pre2__fullsim_noPU_2021_14TeV-1649399635Step3'

Case 2:
wmagent_SC_EL8_khurtado_sc8v14_noStepCampaigns_230922_202131_1605

Workload campaign = yes
Step 1 campaign= no
Step 2 campaign= no
Step 3 campaign= no
Result:
campaignName = 'DMWM_Test'

Case 3:
wmagent_SC_EL8_khurtado_sc8v17_noca_230922_202457_7564

Workload campaign = no
Step 1 campaign= no
Step 2 campaign= no
Step 3 campaign= no
Result:
campaignName = 'DEFAULT_AcqEra'

Case 4:
wmagent_SC_EL8_khurtado_sc8v16_230922_202424_131
Workload campaign = yes
Step 1 campaign= yes
Step 2 campaign= yes
Step 3 campaign= yes

Result:
campaignName = 'CMSSW_12_4_0_pre2__fullsim_noPU_2021_14TeV-1649399635Step1,CMSSW_12_4_0_pre2__fullsim_noPU_2021_14TeV-1649399635Step2,CMSSW_12_4_0_pre2__fullsim_noPU_2021_14TeV-1649399635Step3'

@amaltaro This is ready for review.

@amaltaro Yes! I will add the unit tests

@amaltaro The unit tests have been added.


Game crashes when rendering Create blocks

My game will crash when i attempt to load any blocks from create on ships

my modlist

Downgrade to 0.5e

on it

which mod?

Create 0.5g is not supported, use 0.5e


Deleteing child node checking parent's checkboxes

Hello, I've run into a small issue and wanted to see what you think. I was trying to delete the last child node from a parent while using the checkbox plugin. When I deleted the child node from the parent it caused the parent's checkbox to be checked. This seems to happen when you delete the last child from a parent.
Example: http://jsfiddle.net/jakeyankovic/rvLL10h6/
Thanks,
Jake

Thanks a lot for fixing this. Just got the latest and it works perfectly.
Jake

Hi Vekata, I'm using v3.3.5 and it seems that this issue still exists. If the last node is selected, as soon as I delete it, the tree checked the parent.  Any fix?


user store domain is not set in the challenge question path

Even though we passed the username with correct tenant domain and user store domain (ex<EMAIL_ADDRESS>in change password in challenge question path
It displays the message as  "user does not exist" in the dashboard UI. even though the user exists in the secondary user store.
And following info log in the wso2carbon.log
INFO {org.wso2.carbon.identity.recovery.password.SecurityQuestionPasswordRecoveryManager} -  initiateUserChallengeQuestion :User store domain is not in the request. set to default for user<EMAIL_ADDRESS>Applicable version IS 5.5.0

Thank you for your contribution!
We are closing this issue since it has not been prioritized for a long time. Chances are that it has already been solved in more recent versions. If not, we will be re-evaluating this when it becomes a priority.


Container does not (re-)start after host reboot

Hi,
first of all - thanks for this great work!
I used your quick start guide with a postgres and a docker container. Everything works fine - except that the "redmine" container does not start after a reboot of the host system. The postgres container runs smoothly.
Since I'm very new to Docker, I have no clue where to look first. I checked the Docker reference, but I don't think that the "restart policies" are the right way to fix this.
Thanks.
-Sebastian
BTW: Starting the container manually by "docker start redmine" works fine!

@arnoldse You can use the --restart argument or setup a init.d or systemd startup script. You may refer to https://docs.docker.com/engine/articles/host_integration/ for the available options.

Hi,
I fixed it with a daily cron job which starts the container. The --restart argument did not work for me.
But thanks again ...


Make date handling more flexible for non "post" content

Currently, dates are only show on article pages for content in the post directory. Some people don't want to use that file structure for posts, but still consider them posts. We need to make this more flexible.
As brought up in #85

Fixed in fa3dcbf5358c14fd8f8dcf5a39e43ae0062944ea


Hello, when will all the code be released? Can it be released in December this year? Thanks you!

Hi, it will be released this week with support for GPT-4V.


Add KeePassDX icon

Trickier one! 😅 Addresses #374

[x] New file which is under 1,024 bytes
[x] Edit Readme
[x] Add reference image
[x] Edit reference page
[x] Android Image

Amazing use of feDropShadow! I don't think we've had one of those before.
  This is per the advice of The Pull Request Hack.
Please use this privileged wisely.

Thank you @edent! I'll absolutely take the responsibility seriously – and I admire the philosophy of the linked post.
I must admit I had no cause to use feDropShadow before either 😁 Thanks again


Add custom template name field to form

I've added a custom template name field that gives the site administrator the option to override the default "forms/form_detail.html" template in the Form admin.

Sorry this has been rejected previously, see #132


Reformat reportalEditPage.vm file with save plugin

Reformat change only

Pull Request Test Coverage Report for Build 3988

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

Files with Coverage Reduction
New Missed Lines
%

azkaban-exec-server/src/main/java/azkaban/execapp/jmx/JmxJobMBeanManager.java
1
72.73%

azkaban-exec-server/src/main/java/azkaban/execapp/JobRunner.java
1
77.11%

Totals

Change from base Build 3982:
0.07%

Covered Lines:
10436

Relevant Lines:
28924

💛  - Coveralls


Revert "Add support for Oracle JDK"

Reverts actions/setup-java#401

What's the reason for reverting #401?

@fniephaus unfortunately, we're not able to support Oracle JDK in setup-java.  I'm sorry for the confusion.  We should have caught it sooner on the PR and relevant issue.

If you'd like to send me an email at brcrista at github.com, I'd be happy to discuss the blockers with you in more detail.

@brcrista @fniephaus i'd like to join this conversation. Will follow up by email.

Hello @Gakk. Thank you for your report. We've released a new version v3.8.0. The major tag will be updated later.

We've released a new version v3.8.0

Thank you for responding so quickly 🚀
After a friendly nudge Dependabot produced a PR bumping to 3.8.0, and all checks passed 👍

#421

@AlenaSviridenko


Repo sync for protected CLA branch

The pull request is created from master637721642035489620 to master to fix git push error for protected CLA branch

@ghogen : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Stage 4 Cross-Correlation Fails for MIRI

The Stage 4 cross-correlation step (used to correct for drift/jitter in the 1D spectra) fails on every frame when run on the MIRI simulated data. While the MIRI data are blackbody simulations (compared to the NIRCam simulated data which have spectral features), I think there should still be plenty of curvature and features in the 1-D spectrum for the cross-correlation step to work. Because the step is run inside a try-catch statement, I don't yet understand why it is failing, but the output cross-correlation strength plot looks like an exponential decay, so it seems like it wants to shift the spectrum unreasonably to the left. I have tried setting sub_mean to True and False, but neither option works. The code does work for NIRCam data (haven't been able to test on NIRSpec yet), so the issue seems to be MIRI specific for some reason.
If others want to try and tackle this, please feel free - otherwise I'll try to get to this in a day or few.

The MIRI spectra are quite featureless and have a strong variation in flux with wavelength. As a result, the cross-correlation just wants to shift the spectra as far to the left as it can since the star is brighter at shorter wavelengths. I've yet to work out if/how we can still do cross-correlation with these mock observations.

For reference, the NIRCam simulated observations are much more feature dense, so the cross-correlation step works well enough for them with the exception of a slight bias which could have a similar origin to the bias from the MIRI data.

Figure 1. Extracted NIRCam spectra

Figure 2. Fitted spectral drifts from NIRCam spectra showing a slight bias

The only thing I've come up with that at least partially works here is subtracting a boxcar filtered version of the spectrum which removes most of the continuum variations but mostly leaves the spectral lines. This was done using the following:
from astropy.convolution import convolve, Box1DKernel
def highpassfilt(signal, highpassWidth):
    g = Box1DKernel(highpassWidth)
    return convolve(signal, g, boundary='extend')
ref_spec -= highpassfilt(ref_spec, highpassWidth)

where highpassWidth is several spectral elements (3-30 as a ballpark). Note that this is done instead of subtracting the mean of ref_spec.
There is a fair bit of scatter in the determined drift though. With the current method, the standard deviation in the determined drift for NIRCam simulated observations (which doesn't have any real spectral drift/scatter) is 0.0048 pixels. For my slightly tweaked method, I get standard deviations of 0.0076 pixels (highpassWidth=3), 0.0160 pixels (highpassWidth=5), 0.0732 pixels (highpassWidth=10), 0.0825 pixels (highpassWidth=20), and 0.1823 pixels (highpassWidth=30). This may be driven by the fact that the cross-correlation values look nothing at all like a Gaussian, so trying to fit them with a Gaussian is likely only adding scatter - the same is true, however, for the current cross-correlation method. For both methods, I tried using np.roll to introduce a 1 pixel shift in some of the spectra, and both methods were able to approximately recover that value. While the current method has a slight bias (-0.0401 pixels), my method shows no statistically significant biases for the NIRCam simulations (for 3-30 pixel boxcars), but there is a significant bias in for boxcars 10 pixels wide or larger for the MIRI simulations.
I've also found a cross-correlation method in PyAstronomy (pyasl.crosscorrRV) which at least in principle seems to work. If I first subtract the spectra's continua with the highpassfilt() function, then there appears to be minimal bias or scatter in the retrieved drift/jitter for the NIRCam data I tested it on. Specifically, I get 0 pix scatter with a -9.9e-15 bias for highpassWidth = 10 and no oversampling on the reference spectrum. For 3x oversampling on the reference spectrum (using kind="quadratic"), I get 0.0025 pix scatter and -0.0022 pix bias and for 10x oversampling (using kind="quadratic"), I get 0 scatter with a -9.9e-15 bias. I was successfully able to recover integer shifts in the spectrum (made using np.roll), but I haven't got a interpolater working yet to test if the method can recover fractional pixel shifts. For future reference, I've included below the code I wrote based on the example on the PyAstronomy documentation page.
from PyAstronomy import pyasl
from scipy.interpolate import interp1d
from tqdm import tqdm
import numpy as np
import matplotlib.pyplot as plt

# wave_1d and optspec are output from S3 and need to be loaded before running the following. Must also define the highpassfilt as written above

induce_jitter = False
highpassWidth = 10
interp_scale = 3
if interp_scale==1:
    wave_high = wave_1d
    tf_high = optspec[-1]
else:
    wave_high = np.arange(wave_1d[0], wave_1d[-1], (wave_1d[1]-wave_1d[0])/interp_scale)
    fn = interp1d(wave_1d, optspec[-1], kind='quadratic')
    tf_high = fn(wave_high)

for n in tqdm(range(meta.n_int)):
    tf = np.copy(tf_high)
    tw = np.copy(wave_high)
    tf /= np.max(tf)

    # Create data, which are not that well sampled
    dw = np.copy(wave_1d)
    df = np.copy(optspec[n])
    df /= np.max(df)

    if n%400==0 and induce_jitter:
        # Introdude a shift in the spectra on occasion to see if we can recover it. Hacky but works okayish
        df = np.roll(df, 1)

    df -= highpassfilt(df, highpassWidth)
    tf -= highpassfilt(tf, highpassWidth*interp_scale)

    vpix = (dw[1]-dw[0])/dw[0]*astropy.constants.c.value/1e3
    vmax = vpix*5
    dv = vpix/50

    # Carry out the cross-correlation.
    rv, cc = pyasl.crosscorrRV(dw, df, tw, tf, -vmax, vmax, dv, skipedge=50)

    # Find the index of maximum cross-correlation function
    maxind = np.argmax(cc)

    # Fit a Gaussian to the cross-correlation outputs
    params, err = g.fitgaussian(cc/cc.max(), rv, guess=[vmax/5., rv[maxind], 1])

    #meta.drift1d[n] = rv[maxind]/vpix*4/3
    meta.drift1d[n] = params[1]/vpix*4/3

    if n%200==0:
        # Plot the occasional cross-correlation result to make sure things look reasonable
        plt.plot(rv, cc, 'bp-')
        plt.plot(rv[maxind], cc[maxind], 'ro')
        model  = g.gaussian(rv, params[0], params[1], params[2]*cc.max())
        plt.plot(rv, model)
        plt.show()

Given that the modelled observations have no drift/jitter, anything we do with this step is only going to add bias or noise though, so I think I will stop looking into this question for now and return to this issue once we have simulations with drift/jitter or we get some real data. For now, I recommend all future S4 runs set correctDrift equal to False.

I like the idea of applying a high-pass filter, before calling the cross correlation (CC) routine. When it comes time to reducing real data where we don't know the answer, it may be that we'll want to apply several options and see which one produces the smoothest drift and the smallest jitter.
I see this as a nice task for S4... to implement a suite of CC options, iterate through them with different settings, compare drift/jitter values, and select the best option.

Okay, well I'll implement what I have so far then since it's functional and a viable option at least, and then we can add other methods as we think of them and test them later on real data


Extract project model to separate crate

I'm looking into creating a separate crate that would allow getting a HIR db for a project for 'batch' analyses, and this seems to be an obvious first step. We'd probably want to change the error handling to not rely on failure, though, right?

cc @RazrFalcon  this should help with using rust-analyzer as a stand-alone thing.

Nice! Will look into it.

bors r+


Have we experimented with QuietUninstallString yet?

I recently started using the chocolatey-uninstall.extension and Get-UninstallRegistryKey. In investigating another UninstallString, I noticed that this application also had a QuietUninstallString.
EngineVersion        : 3.7.1224.0
DisplayIcon          : C:\ProgramData\Package Cache\{69d72156-6582-4556-8637-06f40aa7f85b}\ImageResizerSetup.exe,0
DisplayName          : Image Resizer for Windows
DisplayVersion       : 3.0.4802.35565
Publisher            : Brice Lambson
ModifyPath           : "C:\ProgramData\Package Cache\{69d72156-6582-4556-8637-06f40aa7f85b}\ImageResizerSetup.exe" /modify
NoElevateOnModify    : 1
QuietUninstallString : "C:\ProgramData\Package Cache\{69d72156-6582-4556-8637-06f40aa7f85b}\ImageResizerSetup.exe" /uninstall /quiet
UninstallString      : "C:\ProgramData\Package Cache\{69d72156-6582-4556-8637-06f40aa7f85b}\ImageResizerSetup.exe" /uninstall
EstimatedSize        : 2598
Resume               : 3
Installed            : 1
PSPath               : Microsoft.PowerShell.Core\Registry::HKEY_LOCAL_MACHINE\SOFTWARE\Wow6432Node\Microsoft\Windows\CurrentVersion\Uninstall\{69d72156-6582-4556-8637-06f40aa7f85b}
PSParentPath         : Microsoft.PowerShell.Core\Registry::HKEY_LOCAL_MACHINE\SOFTWARE\Wow6432Node\Microsoft\Windows\CurrentVersion\Uninstall
PSChildName          : {69d72156-6582-4556-8637-06f40aa7f85b}
PSDrive              : HKLM
PSProvider           : Microsoft.PowerShell.Core\Registry

Is this standard for all installed applications? If so, should we start recommending this in the package description and other docs (examples, etc.)?

Have you heard of auto uninstaller? Have you looked at the source code?

Rob! I'm perpetually behind. I'll look :)

On Aug 17, 2016, at 12:39 PM, Rob Reynolds<EMAIL_ADDRESS>wrote:
Have you heard of auto uninstaller? Have you looked at the source code?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Most of that is written up at https://chocolatey.org/docs/commandsuninstall
And by the way, QuietUninstallString is not always available.


loosing dependencies with 0.7.0

Hi,
We are still using buildSrcVersions because we started with that and also have nit yet upgraded to Gradle 6.  Thus, this report is for version 0.7.0.
We have a multi-module project with a mixture of Java and Kotlin modules.  Most dependencies are used by both but we now have one dependency that is only used by a Kotlin module.  We manually added that dependency to Libs.kt and Versions.kt. Running gradle buildSrcVersions will remove this dependency (and thus breaks the build).
Concrete setup:

Libs.kt contains const val mock_web_server: String = "com.squareup.okhttp3:mockwebserver:" + Versions.mock_web_server
Versions.kt contains const val mock_web_server: String = "4.4.1"
modules/api/rest-client/build.gradle contains testImplementation Libs.mock_web_server
settings.gradle contains include "modules:api:rest-client"

Any idea what is going wrong here?  I currently manually add the dependency back after each buildSrcVersions run, but that is obviously tedious.

Hi, we are no longer working on buildSrcVersions and I'm not familiar with it personally.
That said, here's what I observed:
The constant name you have doesn't reflect the maven coordinates like I believe buildSrcVersions requires.
You are trying to do it manually while buildSrcVersions has a built-in mechanism to detect dependencies not yet managed. Please see the legacy documentation: https://github.com/jmfayard/refreshVersions/issues/88 under the "Replace your dependencies" heading.

I thought the new approach still does not work for multi-module setups, or did I misunderstand?
The tip about the constant name was spot on: I replaced the name with it's value in build.gradle, ran buildSrcVersions again (which picked a different name than the one we manually picked), and then replaced the value with the modified name.  Now everything is back in order. Thanks!

Glad it worked for you!
refreshVersions and dependencies work perfectly for multi-module setups. There's only one limitation, that it shares with buildSrcVersions: it doesn't support having different versions for different modules.

Hmm, I should try that then.  Is there anything special to know for upgrading from 0.7.0?

Apart from the current README, refreshVersions and dependencies are working, but have room for improvement, especially when it comes to the migration task. You can already use it, or you can wait for it to be improved.


schemagen and xjc should add 'if-exists' to bindings when generating an episode file

When schemagen or xjc generates an episode file (via the -episode command-line argument; or in the case of the schemagen ant task, the episode attribute), the resultant file should set the 'if-exists' attribute on the 2nd-level bindings element.
Rationale: When generating an episode file, you may be generating bindings for multiple namespaces at once (especially if you're using schemagen and adding the episode file to your META-INF section of a jar file). If you then try to consume that bindings file via xjc (as documented at http://weblogs.java.net/blog/2006/09/05/separate-compilation-jaxb-ri-21), it will fail if your input schema doesn't have references to all of the namespaces included in the episode file. At least, it will fail if you're using the Ant xjc task, I haven't tried from the command line.
However, if the bindings had 'if-exists' set on them, the xjc task would simply keep going (the code that checks for this attribute is in SCDBasedBindingSet::Target::apply).
Concise example:
Output is currently generated like this:

/**...**/

If it was instead this (note the if-exists on the bindings with scd attribute):

/**...**/

Then the xjc task would ignore any unused bindings when processing it with the -b or from a jar file's META-INF. I think this would be the desired action when using bindings generated as an episode file.
Affected Versions
[2.2.3u1]

Well, it seems like this issue was resolved in a 2.3.0 version:

    <bindings xmlns:tns="http://www.w3.org/2009/xmldsig11#" if-exists="true" scd="x-schema::tns">

Unfortunately this is no longer the case in v4.0.1:

    <bindings xmlns:tns="http://www.w3.org/2009/xmldsig11#" scd="x-schema::tns">

So I guess, this is a new error for JAXB 4.0.x?

It's been 12 years since this issue was open and still this option is not supported by XJC which makes automatic episode generation useless for many use cases. I also have a shared schema between many WSDLs but each one of them uses just a subset of the whole schema, which results in the awful SCD "<whatever>" didnt match any schema component and that's stupid. The xjb file has all the elements that the WSDL might need but it fails because the WSDL doesn't reference them all.
The highsource plugin adds the "if-exist" option but it only supports JAXB 2 yet and we need JAXB 3 and 4 for Spring Boot 3 projects. Having such an option in the base xjc tool would be really really helpful.

Hi @getaceres (and @phax 😄 )
I just made a PR in jaxb-ri to end with jaxb-maven-plugin hack which adds if-exists attributes by default in episode file, since it doesn't hurt anyone from many years now.
We are also working with Matt on JAXB3 support (which is just javax -> jakarta change) and then fastly move and release JAXB4 support plugin versions. This will come very soon, the hard part of the work were done in latest v2 releases.
Regards


some missing macros / convenience functions

Is your feature request related to a problem? Please describe.
It makes sense to have the macros col! and row! in order to be consistent with mat!. This also seems a good opportunity to include a related macro (or function) blockwise!:
blockwise![[A, B], [C, D]];

// ┌───────┬──┐
// │       │  │
// │   A   │ B│
// │       │  │
// ├───────┼──┤
// │   C   │ D│
// └───────┴──┘

solved by #100


Deprecation warning

Hello
I updated to 2022.7.0 today and in the logs, this warning was presented:
Entity None (<class 'custom_components.goodwe.number.InverterNumberEntity'>) is using deprecated NumberEntity features which will be unsupported from Home Assistant Core 2022.10, please report it to the custom component author.

Thanks for notification, update was committed and will get into next release (soon).


Water Dragon Healing Patch & RBM fixes

Added an option to turn on healing the water dragon, which requires sacred water to access Ancient Cistern
Fixed various layers and objects to allow for more instances of RBM

Bug notes:

2 Skyview Boss Keys in the item pool when boss keys are "Anywhere" or "Vanilla"
Water dragon disappears (after beating FS?)

Do not merge until these are fixed; currently working on fixing these


Access to the port listened by Android Application

Is your feature request related to a problem? Please describe
Maybe not? Because it is related to an update of VM IP address removed from Settings app. With Advanced Networking, now the IP address of the VM is the same as the host/computer IP of Build 2205.40000.14.0
Describe the solution you'd like
With enabled of Advanced Networking, the IP of VM is hidden, but I have a requirement of accessing port listened by my developing app using Ktor server, which for debug, I need to access that from external, i.e. host PC browser. I tried using http://localhost:<PORT_LISTENED>/<API>, but responsed ERR_CONNECTION_REFUSED.
However, if I obtain the IP of VM using NetworkInterface, i.e.
NetworkInterface.getNetworkInterfaces().toList().forEach { networkInterface ->
    networkInterface.inetAddresses.toList().forEach { inetAddress ->
        if (!inetAddress.isLoopbackAddress) {
            if (inetAddress is Inet4Address) {
                Log.d("Android-ip", inetAddress.hostAddress.toString())
            }
        }
    }
 }

The output Android-ip: <IP_ADDRESS> is workable for accessing my API, i.e. http://<IP_ADDRESS>:<PORT_LISTENED>/<API>
Describe alternatives you've considered
Show IP address of VM again, or allow accessing port using localhost of host PC
Please specify the version of Windows Subsystem for Android
2303.40000.5.0

We're working on networking improvements that should allow connecting from Windows to your Android app's port using localhost (or the Windows IP address).  Right now, that's only supported for ADB's port.  The networking improvements are on our roadmap for this year, but we don't have a precise schedule to share.

We're working on networking improvements that should allow connecting from Windows to your Android app's port using localhost (or the Windows IP address). Right now, that's only supported for ADB's port. The networking improvements are on our roadmap for this year, but we don't have a precise schedule to share.

Maybe adding back the IP showing right now for transition before your team's completion of networking improvements for any requirement of network debugging for application developer? Because Google official AVD provides util for network debugging.

Tracking this under #213


rpm: always replace existing repo-configuration

The installation script's main purpose is to perform a fresh install, because it's not guaranteed to work correctly when updating an existing installation, and we print a warning if we find docker is already installed. However, it's known that the script is used to update existing installations.
The current script has some limitations for the update scenario;

When using the script for installing from the staging domain, the existing repo configuration is not overwritten. When using yum, or older dnf versions this resulted in 2 separate repo files to be created (docker-ce.repo and docker-ce-staging.repo), which resulted in errors due to duplicate repo definitions with the same name.

When using dnf5, we specified the filename to store the configuration, which resulted in an error;
  + sh -c 'dnf5 config-manager addrepo --save-filename=docker-ce.repo --from-repofile='\''https://download-stage.docker.com/linux/fedora/docker-ce-staging.repo'\'''
    File "/etc/yum.repos.d/docker-ce.repo" already exists and configures repositories with IDs "docker-ce-stable docker-ce-stable-debuginfo docker-ce-stable-source docker-ce-test docker-ce-test-debuginfo docker-ce-test-source docker-ce-nightly docker-ce-nightly-debuginfo docker-ce-nightly-source". Add "--overwrite" to overwrite.

On SLES, it would result in a similar error;
+ sh -c 'zypper addrepo https://download-stage.docker.com/linux/sles/docker-ce-staging.repo'
  Adding repository 'Docker CE Nightly - s390x' ......................................................................................[error]
  Repository named 'docker-ce-nightly' already exists. Please use another alias.
  Adding repository 'Docker CE Nightly - Debuginfo s390x' ............................................................................[error]
  Repository named 'docker-ce-nightly-debuginfo' already exists. Please use another alias.

This patch updates the script to overwrite / replace the existing repo config. This brings the rpm-based installations in line with the Debian approach, which unconditionally overwrites /etc/apt/sources.list.d/docker.list.
It's worth noting that this change will therefore overwrite any customisations made to the repo-configurations, so a warning is added for that.

- What I did
- How I did it
- How to verify it
- Description for the changelog

- A picture of a cute animal (not mandatory but encouraged)

/cc @neersighted


[REQUEST] [open-ce-builder] - --run_tests option should be made default

To test the package updated we should enable --run_tests by default. This will make sure every change we do is verified just after the build.

For this we need to create short tests in feedstocks' open-ce-tests.yaml that don't take more time to run, else the build time would increase. We need to brainstorm more on this how to have short/long tests in open-ce-tests.yaml.

Yes I agree, this just requires a minimal test to keep the build times unaffected.


Version is not incremented by commits within beta branches (release and hotfix branches)

We create nuget packages with the version we got from GitVersion.exe. The problem we have, is that none of the generated versions fits our needs. We used to use LegacySemVerPadded because it is the right format to create a valid nuspec and nuget package (1.8.0-beta0001). But now we noticed, that when we create release or hotfix branches (GitFlow branching model) and create commits within these branches, the version is not incremented by commits. This leads to deployment problems, because we cannot deploy the same version (which actually is not the same version) twice.
The strange thing is, that this behavior does not appear within the develop branch (unstable). It seems that in the develop branch the SemanticVersionPreReleaseTag.Number is incremented. Where as in release and hotfix branches this number always remains one.
We could now use ClassicVersionWithTag, but then we have the problem with the format when we want to create nuget packages (<IP_ADDRESS>-beta.1 is not allowed).

@FabianTrottmann just wanted to check...
What version of GitVersion are you currently using?

@gep13 GitVersion.Portable.2.0.1

try 3.0.3: choco GitVersion.Portable -pre -y

I would definitely suggest that you try out the current beta version, as @MeirionHughes suggests.  @JakeGinnivan has been putting in a ton of work to ensure that GitVersion is far more maintainable and support much more workflows than it did in the 2.0.x release.  It could (and hopefully will be the case) that the issue that you are seeing is already corrected.
If not, then I would suggest you raise a PR against the beta version, not directly against master.

@MeirionHughes @JakeGinnivan
The version remains the same (not incremented) as it is in 2.0.1. I'll raise a PR against the beta version.
Thanks for the quick responses

How about if you change the branch config to use continuous delivery? (not the one its set to by default). I think that is supposed to bump the version per commit.

Yeah, try using gitversion init it explains the config. also read https://github.com/ParticularLabs/GitVersion/blob/master/Docs/versionIncrements.md which explains why things are like they are.
In short if we incremented every commit then you would not be doing SemVer

Also see https://github.com/ParticularLabs/GitVersion/tree/master/Docs#the-semantic-version-does-not-increment-every-commit

This leads to deployment problems, because we cannot deploy the same version (which actually is not the same version) twice

If you tag each deployment with a git tag, then the version will increment as well.

I think I have the same problem:
http://i.imgur.com/LSXkGOf.png
Develop might've been at <IP_ADDRESS>-unstable before the hotfix, but once the hotfix is merged, develop is again only one commit ahead of the hotfixed master and GitVersion resets the number to <IP_ADDRESS>-unstable. We build NuGet packages off of our develop branch (e.g. "1.3.0-unstable0001") and if the counter resets, people do not get any updates anymore.

Hi, I am experiencing the same issue as @FabianTrottmann and am not quite sure what the conclusion is. Will this be fixed in a future release, so that release branches will have updated NuGet version on each commit?

Great. Any thoughts on making this more discoverable/obvious?

I think it is obvious enough in v3 - if you plan to make your own configuration file, I guess you would start with running GitVersion init, or running GitVersion /showconfig which gives much more usable output than the output I got from v2.
My problem was basically that I was running v2, and could not figure out what to put in the configuration file. I'm not sure about what version that documentation is referring to - might be referring to v3 features, that I then tried to use in v2?

Yeah, this stuff didn't exist in v2. Doco was lagging and not clear. I hoped to have v3 out a long while ago, but live happens :P

Hi, I'm having the same issue as the original post described, and the links referenced lead to 404 pages so I can't figure out which documentation was being cited in the answers.
In the gitflow branching model, I think there's an expectation that there may be small additional commits on the release branch before the release is finalised and the branch is closed. While it fits my expectations that the SemVer would not change, I'm not sure why the metadata element of the GitVersion output would stay forever fixed at "beta0001".
I'm using Octopus deploy and TeamCity, and without incrementing the version metadata for additional commits, it makes it very difficult to push new packages out from the release branches because Octopus doesn't think anything has changed.

@mattsalmon: This issue was closed almost 1.5 years ago. There's not much in here of relevance anymore. I think the best thing you can do is try to find an open issue that discusses your problem, or create a new one. Continuing discussion on an issue that has been closed for so long isn't the best way forward, I think. You can always reference this issue from your new one, though.


WIP: Major enhancements

UI changed to the shiny dashboard.
Shiny isolated modular approach adopted.
files and functions further streamlined.

Codecov Report

Merging #34 into master will decrease coverage by 4.30%.
The diff coverage is 0.00%.

@@            Coverage Diff            @@
##           master     #34      +/-   ##
=========================================
- Coverage    7.69%   3.38%   -4.31%
=========================================
  Files          10      10
  Lines         130     295     +165
=========================================
  Hits           10      10
- Misses        120     285     +165

Impacted Files
Coverage Δ

R/app_getOrg.R
0.00% <0.00%> (ø)

R/app_utils.R
0.00% <0.00%> (ø)

R/module_enrichGO.R
0.00% <0.00%> (ø)

R/module_enrichGO_utils.R
0.00% <0.00%> (ø)

R/module_enrichKEGG.R
0.00% <0.00%> (ø)

R/module_enrichKEGG_utils.R
0.00% <ø> (ø)

R/module_mapids.R
0.00% <0.00%> (ø)

R/module_mapids_utils.R
0.00% <ø> (ø)

... and 2 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a709125...ebbb355. Read the comment docs.

Codecov Report

Merging #34 into master will decrease coverage by 4.78%.
The diff coverage is 3.80%.

@@            Coverage Diff            @@
##           master     #34      +/-   ##
=========================================
- Coverage    7.69%   2.90%   -4.79%
=========================================
  Files          10      10
  Lines         130     344     +214
=========================================
  Hits           10      10
- Misses        120     334     +214

Impacted Files
Coverage Δ

R/app_utils.R
0.00% <0.00%> (ø)

R/module_app_input_validation.R
0.00% <0.00%> (ø)

R/module_enrichGO.R
0.00% <0.00%> (ø)

R/module_enrichGO_utils.R
0.00% <0.00%> (ø)

R/module_enrichKEGG.R
0.00% <0.00%> (ø)

R/module_enrichKEGG_utils.R
0.00% <ø> (ø)

R/module_mapids.R
0.00% <0.00%> (ø)

R/module_mapids_utils.R
0.00% <ø> (ø)

R/module_app_input_utils.R
20.83% <20.83%> (ø)

... and 5 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8b0c0f1...af907fd. Read the comment docs.


1.5 enhancement

Coverage increased (+0.2%) to 79.98% when pulling 15e29ddd93d0413b1dbccfad8a950f0bb3b50e3a on Pengyuz:master into e8377eab798b0a74232f05656cf2237f602712f3 on CS2103AUG2017-W09-B3:master.

Coverage increased (+0.5%) to 80.353% when pulling ddc860a086959518299ee3207104f7baa8341f77 on Pengyuz:master into e8377eab798b0a74232f05656cf2237f602712f3 on CS2103AUG2017-W09-B3:master.


Sound interesting but lacks screenshots

Hi Guys!
This thing sound very promising, but it would be extremely useful to see some screenshots of a demo example. Can you link to some screens shots or at least some easy to implement (clone and run) example code?
Thanks and Cheers!
E:V:A

Just found it! Sorry to bother you! :+1:


Do the files in the output directory completed?

Hi fmriprep gurus,
I have crushes while running fmriprep, but the output directory containing all the files I need.
I wonder if I can use them although the run as a whole is corrupted. Are they valid to use?
I am talking about this kind of output files:
“sub-1_ses-01_task-seker_run-01_space-MNI152NLin2009cAsym_desc-preproc_bold.nii.gz”
“...space-MNI152NLin2009cAsym_desc-preproc_T1w”
Thank you very much,
Sharon

Hi Sharon,
Yes, the files in the output directory have been fully processed. No further changes are made after they are placed there.
Could you describe the crashes you're seeing more?
Best,
Chris

Thank you!
I have started a dedicated new issue regarding the crushes.

Thanks.


java version conflict

Hello, bro
There is a problem that I restarted cerebro on linux when I changed the "auth" by using "basic" and set "username", "password".
The following message is exception:
! @7dmfpal90 - Internal server error, for (GET) [/] ->
play.api.UnexpectedException: Unexpected exception[RuntimeException: java.lang.UnsupportedClassVersionError: controllers/routes has been compiled by a more
recent version of the Java Runtime (class file version 55.0), this version of the Java Runtime only recognizes class file versions up to 52.0]
at play.api.http.HttpErrorHandlerExceptions$.throwableToUsefulException(HttpErrorHandler.scala:319)
at play.api.http.DefaultHttpErrorHandler.onServerError(HttpErrorHandler.scala:243)
at play.core.server.AkkaHttpServer$$anonfun$1.applyOrElse(AkkaHttpServer.scala:382)
at play.core.server.AkkaHttpServer$$anonfun$1.applyOrElse(AkkaHttpServer.scala:380)
at scala.concurrent.Future.$anonfun$recoverWith$1(Future.scala:417)

This is my configuration
auth = {
type: basic
settings: {
username = "2ccm-admin"
password = "1q2w3e4r-zxcv"
}
}

If I use default configuration, it's successful startup. Could you help me to solve this problem. Thank you.

Hi ChesterZheng,
I had the same issue after upgrading and updating the java version to a higher level solved the problem for me. I did not read the minimum spec but I am now running cerebro on openjdk 1.11.

@ChesterZheng Hi. Sorry, this was my mistake. One of the releases published used jdk 11 to compile. Upgrading to the latest version should work no problems, although I will soon increase the minimum java requirements to use java 11.


Project X Tidy Lovelace 🕵♂🎨🖼

Up until now I have added basic cards to Lovelace panels and have created a range of dashboards. I've structured panels for each room and have mostly dropped every entity related to that room while I have developed a boatload of automation and integrated every possible thing in my home. While the UI is not my primary driver for my smart home it is still important it is functional and reflective of my home. It's now time for me to start improving my Lovelace dashboards, panels and cards.
I'm unsure if I will keep this within one issue or many to constrain the context of screenshots etc nor will I post every card in every panel but where I want to preserve and share the code I will.

Utility

A nice resource for polygons backgrounds - https://halftonepro.com/polygons

Another useful background resource - https://trianglify.io

Illustrations https://www.freepik.com/


docs: Add Lucide-Svelte icon import instructions and type definitions

What is the purpose of this pull request?

[ ] New Icon
[ ] Bug fix
[ ] New Feature
[x] Documentation update
[ ] Other:

Description
Following the conversations in #1761 & #1585, this improves the docs around the areas of type definitions and import statements.
If there are any modifications you wish for me to carry out, please let me know. 🙂
Icon use case

Alternative icon designs

Icon Design Checklist
Concept

[ ] I have provided valid use cases for each icon.
[ ] I have not added any a brand or logo icon.
[ ] I have not used any hate symbols.
[ ] I have not included any religious or political imagery.

Author, credits & license

[ ] The icons are solely my own creation.
[ ] The icons were originally created in # by @
[ ] I've based them on the following Lucide icons:
[ ] I've based them on the following design:

Naming

[ ] I've read and followed the naming conventions
[ ] I've named icons by what they are rather than their use case.
[ ] I've provided meta JSON files in icons/[iconName].json.

Design

[ ] I've read and followed the icon design guidelines
[ ] I've made sure that the icons look sharp on low DPI displays.
[ ] I've made sure that the icons look consistent with the icon set in size, optical volume and density.
[ ] I've made sure that the icons are visually centered.
[ ] I've correctly optimized all icons to two points of precision.

Before Submitting

[x] I've read the Contribution Guidelines.
[ ] I've checked if there was an existing PR that solves the same issue.

Great! Maybe a link to the Svelte doc to the ComponentType would be nice? Apart from that, it seems perfect to me.
Link : https://svelte.dev/docs/typescript#types-componenttype


invalid client_id

Hi, I have followed your tutorial step by step and I keep getting invalid client_id in my response.
When I test the client secret in https://jwt.io, I get an invalid signature error. Any idea on what might be causing this ? thanks.

The same problem

I think it is the client_secret problem?

@jramini12 where exactly are you seeing "invalid client_id"?
When I test sending the user to Apple with a bad client_id I see this error from apple:

@aaronpk , when we call the http function wwith $url, $params.. in response we are getting the invalid_client.
the same client_id is actually working before hand, for creating of client_secret...

Still the same, after redirecting back to your script


Right click with day view and scheduler

Hello.
I was unable to use fullcalendar-rightclick.js with the day and scheduler views. The problem comes from this line (81) :
// FullCalendar >= 3.5.0:
cell = hit.component.getCellRange(hit.row, hit.col);

On these views the hit.row is undefined. It seems legit as there is only one day displayed.
But I was able to fix it this ways :
// FullCalendar >= 3.5.0:
// Update for day view
if(hit.row) {
	cell = hit.component.getCellRange(hit.row, hit.col);
} else {
    	var componentFootprint = hit.component.getSafeHitFootprint(hit);
	if (componentFootprint) {
		var dateProfile = that.calendar.footprintToDateProfile(componentFootprint);
		cell = { start: dateProfile.start, end: dateProfile.end };
	}
}

It uses the same base as the dayClick callback used by fullcalendar to get the exact instant target (hour/minutes included). As I'm not sure the hit.row is the right way to detect it, nor am I able to provide a unit test, I didn't send a pull request. ;)

Do you think this could be caused by your recent PR @duckbrain?

@jlannoy What version of FullCalendar and the scheduler plugin are you using? I'm assuming >= 3.5.0, but if not, there's a problem with version detection.
@mherrmann Almost certainly. I haven't reproduced it yet in testing, but I'm not going to have a lot of time to look into it. Hopefully it's a quick fix.

@duckbrain okay thanks! Looking forward to your findings.

Okay, I reproduced the issue. Happens in >= 3.5.0. Only happens in the agendaDay and agendaWeek` views (and probably the scheduler plugin, but I haven't tested for that). @jlannoy's fix seems to do it. PR to come.
TypeError: Cannot read property 'clone' of undefined
    at TimeGrid.DayTableMixin.getCellDate (https://cdnjs.cloudflare.com/ajax/libs/fullcalendar/3.10.0/fullcalendar.js:6702:62)
    at TimeGrid.DayTableMixin.getCellRange (https://cdnjs.cloudflare.com/ajax/libs/fullcalendar/3.10.0/fullcalendar.js:6706:26)
    at HTMLDivElement.<anonymous> (file:///home/jonathan/src/github.com/ec2-software/fullcalendar-rightclick/fullcalendar-rightclick.js:81:30)
    at HTMLDivElement.dispatch (https://code.jquery.com/jquery-2.1.3.js:4430:9)
    at HTMLDivElement.elemData.handle (https://code.jquery.com/jquery-2.1.3.js:4116:28)
    at Object.trigger (https://code.jquery.com/jquery-2.1.3.js:4345:12)
    at HTMLTableCellElement.<anonymous> (https://code.jquery.com/jquery-2.1.3.js:4896:17)
    at Function.each (https://code.jquery.com/jquery-2.1.3.js:374:23)
    at jQuery.fn.init.each (https://code.jquery.com/jquery-2.1.3.js:139:17)
    at jQuery.fn.init.trigger (https://code.jquery.com/jquery-2.1.3.js:4895:15)

Thank you guys!


util.isIE doesn't really work?

Hi,
It seems like isIE utility function doesn't always work.
Example: 'Mozilla/5.0 (compatible; MSIE 10.0; Windows NT 10.0; WOW64; Trident/7.0; .NET4.0C; .NET4.0E; .NET CLR 2.0.50727; .NET CLR 3.0.30729; .NET CLR 3.5.30729)'.match(/(MSIE|Trident.*rv[ :])([0-9]+)/)  will return null.
I had to change quantifier to  ? to have it work.

Hi,
This editor supports only version IE11.
It seems your userAgent string is IE10.
IE check will be modified to navigator.userAgent.indexOf('Trident') > -1.
Thank you!

Huh my bad, sorry and thank you.


Not Django 1.10 compatible

RequestContext user in views.form_sent has been removed in Django 1.10 - see also https://stackoverflow.com/questions/38996878/django-1-10-requestcontext-processors-not-working.
Monkey patching https://github.com/stephenmcd/django-forms-builder/blob/master/forms_builder/forms/views.py#L113-L119 to:
def form_sent(request, slug, template="forms/form_sent.html"):
    """
    Show the response message.
    """
    published = Form.objects.published(for_user=request.user)
    context = {"form": get_object_or_404(published, slug=slug)}
    return render(request, template, cont

seems to work.

after I add the name to settings.py I keep getting this error
Unhandled exception in thread started by <function wrapper at 0x7f5e2bb79a28> Traceback (most recent call last):   File "/home/jubanjan/.local/lib/python2.7/site-packages/django/utils/autoreload.py", line 226, in wrapper     fn(*args, **kwargs)   File "/home/jubanjan/.local/lib/python2.7/site-packages/django/core/management/commands/runserver.py", line 113, in inner_run     autoreload.raise_last_exception()   File "/home/jubanjan/.local/lib/python2.7/site-packages/django/utils/autoreload.py", line 249, in raise_last_exception     six.reraise(*_exception)   File "/home/jubanjan/.local/lib/python2.7/site-packages/django/utils/autoreload.py", line 226, in wrapper     fn(*args, **kwargs)   File "/home/jubanjan/.local/lib/python2.7/site-packages/django/__init__.py", line 27, in setup     apps.populate(settings.INSTALLED_APPS)   File "/home/jubanjan/.local/lib/python2.7/site-packages/django/apps/registry.py", line 85, in populate     app_config = AppConfig.create(entry)   File "/home/jubanjan/.local/lib/python2.7/site-packages/django/apps/config.py", line 116, in create     mod = import_module(mod_path)   File "/usr/lib/python2.7/importlib/__init__.py", line 37, in import_module     __import__(name)   File "/home/jubanjan/.local/lib/python2.7/site-packages/django/contrib/contenttypes/apps.py", line 7, in <module>     from .management import ( ImportError: cannot import name update_contenttypes
Now it won't run even after I've removed the name from INSTALLED APPS. Any help?
I'm using django 1.10.7

97ee3855204d9d6a33c097319e2135a817d5047f


[Feature Request] Quotes on Telegram

I am not sure if this is possible but it would be really nice if we could also see the quoted messages, because it's a little bit confusing some times if youre not seeing them.
For example: You can not see if someone answers one of your posts.

Could you give an example, maybe screenshot of what you mean ? I don't use telegram myself

In telegram, you can answer to someone specific by quoting. This is specially useful in bigger chats and gets used a lot. The problem is that when users in Telegram send a quote, users on the other side are just going to see the answer to the text, but not for which post this answer actually was.

In telegram, you can answer to someone specific by quoting. This is specially useful in bigger chats and gets used a lot. The problem is that when users in Telegram send a quote, users on the other side are just going to see the answer to the text, but not for which post this answer actually was.

@TomTheDragon I tried adding this, but I don't use telegram, so it's blind coding ;-)
Could you test it out ?
Binaries on https://bintray.com/42wim/nightly/Matterbridge/v1.2.0-11-g4bda29c#files

I tried it and it seems to work fine so far and its already super useful :)
This is actually the first time ever, i see a bot using this ;)
It outputs like this atm:
botname: [User1] hi
botname: [User2] hey there (re @User1:hi)
I think it would be better to have an output more like this maybe:
botname: [User1] hi
botname: [User2] hey there [@User1: hi]
But it would be "perfect", if we could customize the output of this with some simple tags i think :)

Ok, nice to see it works :)
I'll think about the customization


feat(list): input binding for dense mode

Replaces the attribute selector for dense mode with an Angular input binding.

Note: As mentioned in the issue, it's currently possible to change dense mode dynamically using [attr.dense] .. but the API is not very elegant and you have to use null to disable dense mode.
Fixes #3882

Wouldn't it be better to deprecate the dense attribute altogether and have a class for the dense mode instead? It will make it faster to toggle and will allow us to set up inheritance.

The dense attribute still works due to the boolean coercion.

I mean a CSS class for toggling the dense mode (e.g. .mat-dense), instead of an attribute. This will allow us to handle dense mode in a similar way to theming, when we start supporting it on more components.

Hmm not sure. The dense binding seems more convenient and a nicer API. Otherwise it ends up as in Material 1 where we have X classes on the list. Also the dense attribute sets a class already.

That's what I mean: the dense attribute is just a proxy for the class and it adds some unnecessary bloat. If we use a class, we get the toggling for free via ngClass and we can use it if we set up inheritance later on. Still, this is mostly a point for discussion.

@crisbeto That's probably a more generic discussion about API design.. Otherwise developers could also just do .md-accent, .md-primary etc.

I would rather avoid changing this until we have a more comprehensive approach for handling density. This would include a uniform API for all components, multiple levels of density, a way to set the default for all components, and possibly a way to set the density for just a portion of the app.
Is there a compelling reason to change just the list now?

Not really. Just a request through an issue. This change isn't really major anyways, it just changes the attribute selector to a @Input() binding (more convenient when dynamic toggling).
Feel free to close if necessary.

Ack- closing until we tackle the larger density issue