text:
Reaction Event - Context Requests

Would like to request one additional context tags:

[x] <context.group> - Returns the guild Group that the reaction is heard from

Would like to request some revisions:

[x] <context.user> returns the DiscordUser object; can we change this to <context.author> for consistency with other discord events?
[x] <context.emoji_id> returns the literal name of the emoji as opposed to the ID of the emoji; some emoji's from different groups can contain the same name where-as the ID is always directly associated with the correct emoji.

Would like to also add meta for this event to be called by our clone of the Meta bot if possible
Lastly, would it be difficult to add a DiscordEmoji object type that could return each of the emoji fields (Excluding the User, which is provided with the user context we have now) listed in the Emoji Object Structure?:
{
  "id": "41771983429993937",
  "name": "LUL",
  "roles": ["41771983429993000", "41771983429993111"],
  "user": {
    "username": "Luigi",
    "discriminator": "0002",
    "id": "96008815106887111",
    "avatar": "5500909a3274e1812beb4e8de6631111"
  },
  "require_colons": true,
  "managed": false,
  "animated": false
}

[x] ID | Emoji IDs would differentiate the Emoji from every other emoji regardless of name, animation or group it originated from
[x] Name | Great for formatted name of the emoji
[ ] Roles | This specifically would return the list of roles that the specified channel has whitelisted the roles to via the 'Allow Reactions' and 'Allow External Reactions' permissions
[ ] Require_colons | This may only be beneficial if we could utilize this as opposed to regex to coordinate easier message replacements of emojis
[ ] Managed | Returns if this emoji is implemented by this group or not
[x] Animated | Returns if this emoji is animated; may only be beneficial for the same reason as the require_colons field
[ ] Available | This would return true or false if the server loses access to this emoji due to Server Boost falling to a level that reduces the server's emoji count

Thanks :}

We have everything we need from this currently for the Ticket system; closing and leaving the unmarked for future potential features we may or may not need in the future.

repo: AuroraInteractive/dDiscordBot


text:
#28

Да говнокод, а что ты мне сделаешь? :d

Может ещё что-то нужно поправить, но хз
С телефона не очень удобно проверять

:d

😐

?

все настолько плохо?)

Не, но у меня есть пару мыслей на счёт данной фичи, через час сяду поковырять

source: gharchive/pull-request

authors: IsTopNick JoCat

repo: AuroraTeam/LauncherServer

license: MIT

license_type: permissive

license_source: github-api


text:
Add support for string based properties decorated with the RangeAttribute

Consider the following:
public class MyModel
{
    [Required]
    [Range(1, 65535)]
    public string Port { get; set; }
}
var fixture = new Fixture();
var model = fixture.Create<MyModel>();

In 3.51.0 the first call the Create() worked, however subsequent calls threw, now in 4.0.0-rc1 (the version I am using), the first call throws ObjectCreationException  whereas the attribute itself offers support for the use case.

Originated from https://stackoverflow.com/q/47184400/126014

@jcasale Thanks for raising the issue.
Well, as far as I noticed, AutoFixture wasn't designed to handle gracefully the Range attribute applied to non-numeric members. Therefore, I'm basically not surprised as we already have the same issue for enums. They should be fixed together and I've already started to investigate that.
I don't have too much experience with annotation attributes and at the first glance it looks a bit weird that you apply numeric range attribute to a property of string type. Could you please point to some documentation describing that such usage is perfectly valid (and what is the expected behavior)? Just want to ensure that we cover valid scenarios :blush:
P.S. :+1: for using AutoFixture v4 :wink:

To be honest, it's a convention I am used to from the ASP arena and I don't know if it's an abuse of the facility or intended. I will check out the docs and if not, the reference source for logic that indicates it's valid and expected and not simply a fragile coincidence.

Reference source  implies it will handle casting a string (which it does) and the docs illustrate an example where the the value being applied is coerced and in that case, boxed. I have a workaround and I admit the use case is rare, however it does appear to be valid but certainly low priority if any.

@jcasale Thanks for the confirmation. Well, it's indeed seems that string could be a valid type. Also it looks like that Range attribute could support even other types, like DateTime. The question is whether we are going to support that in AutoFixture out-of-the-box as we clearly cannot support all those types.
Probably, it makes sense to follow the way, when we support this feature partially - recognize the Range attribute and wrap that to the RangedRequest. Later clients could add customization to handle the RangedRequest for the custom types they have. The benefit is that you will not need to handle the Range attribute directly. Also we as a project will not need to support the variety of different possible types as they might be quite rare.
@jcasale What would you say about that plan?
@moodmosaic Your opinion is also appreciated as the topic is quite tricky.

The purpose of supporting data annotations in AutoFixture is to provide a more fine-grained control over the scope of generated values.
However, some of those data annotations have a totally weird API, where you can easily do the wrong thing, as with the RangeAttribute, which has 3 constructor overloads:

RangeAttribute(Double, Double)
RangeAttribute(Int32, Int32)
RangeAttribute(Type, String, String)

And because of that pesky 3rd constructor overload accepting a Type, it is possible to do this:
[Range(1, long.MaxValue)]
public long SomeProperty { get; set; }

1 gets casted into a Double
long.MaxValue gets casted into a Double resulting to an arithmetic overflow.

So, IMHO, and AFAICT, in this case:
[Range(1, 65535)]
public string Port { get; set; }

we should throw an error. In the error message we should probably tell to the user that the right way of controlling the scope of generated strings is by doing this:
[Range(typeof(string), "1", "65535")]
public string Port { get; set; }

And then, we'd have to make sure we support not only strings, but chars, dates, and so on:
[Range(typeof(DateTime),"14-Dec-1984", "14-Dec-2099")]

[Range(typeof(char), "n", "s")]

This is one of the reasons that F# Hedgehog makes this concept explicit, and easier, through the Range type and combinators.

In the error message we should probably tell to the user that the right way of controlling the scope of generated strings is by doing this:

[Range(typeof(string), "1", "65535")]
public string Port { get; set; }

That is incorrect, it works coincidentally for values where none of the place values exceed the leading place values of the max string. For example, 1 through 6 pass, however 7 through 9 fail, 10 through 65 pass, 66 to 99 fail, 100 through to 655 pass then 656 fails etc.
A contrived example:
using System;
using System.Collections.Generic;
using System.ComponentModel.DataAnnotations;
using System.Linq;
using System.Reflection;

internal class Program
{
    [Range(typeof(string), "1", "65535")]
    // [Range(1, 65535)]
    public string Port { get; set; }

    private static void Main()
    {
        for (int i = 1; i < 65536; i++)
        {
            Program program = new Program
            {
                Port = i.ToString()
            };

            foreach (var error in program.Validate())
            {
                Console.WriteLine($"{i}, {error.ErrorMessage}");
            }
        }
    }
}

public static class Extensions
{
    public static IEnumerable<ValidationResult> Validate<T>(this T model)
        where T : class
    {
        if (model == null)
        {
            throw new ArgumentNullException(nameof(model));
        }

        foreach (PropertyInfo propertyInfo in model.GetType().GetProperties())
        {
            object[] attributes = propertyInfo.GetCustomAttributes(typeof(ValidationAttribute), false);

            if (attributes.Length == 0)
            {
                continue;
            }

            ValidationContext validationContext = new ValidationContext(model)
            {
                DisplayName = propertyInfo.Name
            };

            if (attributes.OfType<RequiredAttribute>().FirstOrDefault() is RequiredAttribute required)
            {
                ValidationResult result = required.GetValidationResult(propertyInfo.GetValue(model), validationContext);

                if (result != null)
                {
                    yield return result;

                    yield break;
                }
            }

            foreach (ValidationAttribute attribute in attributes)
            {
                ValidationResult result = attribute.GetValidationResult(propertyInfo.GetValue(model), validationContext);

                if (result == null)
                {
                    continue;
                }

                yield return result;
            }
        }
    }
}

@jcasale Thanks for the sample.
That's why I'd suggest to not include this feature to the AutoFixture for now as there might be a whole set of different options depending on the OperandType/MemberType combinations.
In the #920 I've introduced the generic RangedRequest in a way that later any specimen builder could decide what to do. Request offers both the OperandType (this is how the Ranged attribute refers to the type you specified) and MemberData - type of the member you applied this property to.
Later @jcasale could register it's own RangedRequest builder to handle strings in a way he wants, without a need to deal with the RangedAttribute directly.
For me that looks like a good trade-off.

@zvirja No objections, I have a workaround and realized some good takeaways from this. In reference to registering my own RangedRequest, off the top of your head do you know of any code exercising the concept that I could review to see how this is accomplished?

any code exercising the concept that I could review

Probably, not for now, as #920 is still under the review by @moodmosaic. Only after we merge the PR, we'll know its shape, so I'll be able to show you a demo.
However, the best place would be to look at the NumericRangedRequestRelay implementation (if it doesn't changes during the review) as it's a sample of builder for numeric types.

@jcasale Feel free to use the NumericRangedRequestRelay or EnumRangedRequestRelay as a sample of such customization.
This API will be available since our next release that should happen in the nearby future.
Closing this one as no further action is required so far. Feel free to ask more questions if you have 😉

source: gharchive/issue

authors: jcasale moodmosaic ploeh zvirja

repo: AutoFixture/AutoFixture

license: mit

license_source: bigquery


text:
Add Support for FakeItEasy 2.1+ (#628)

Went with 2.1 plus, since it's the latest as of now.

I don't know why it fails to a missing assembly.

Thank you for your interest in contributing to AutoFixture! This looks promising :+1: The few comments I have are superficial, and are easy to address :smile:

Thank you for your contribution! It's now live as AutoFixture.AutoFakeItEasy2 3.48.0.

authors: caloggins ploeh


text:
Feat: general wrapper

Description
a general wrapper that accepts arguments to field mappings as arguments
Type of change

feat: A new feature

I like the look of this!
Would it be sensible to split up the wrapper into two – one which does the input name mapping, and one which does the output wrapping? Then we could use the two independently – perhaps your inner function needs to return two values (say conditions and experiment_js_code) and returns those as a Result/Delta object, but you still want a mapping. Conversely, perhaps you have a function which uses entirely standard naming for the variables, but you want to wrap the outputs.

hey @younesStrittmatter , do you think you could refactor this so that there's a wrapper function which just does the input field-name mapping? We could include that as an option in the on_state function from #33, but it needs to be an independent wrapper function first so that we can test it really extensively on its own.

I think this is covered by the newest version of the state object – closing this now.

authors: hollandjg younesStrittmatter

repo: AutoResearch/autora-core


text:
Check collection of input words

There should be an option to check a collection of input words, for better automaton validation.

Integrated in the next version

repo: AutoSimDevelopers/automata-simulation


text:
Tried to install in a jupyter docker container, failed

15:45 $ docker run -ti jupyter/jupyterhub /bin/bash
Unable to find image 'jupyter/jupyterhub:latest' locally
latest: Pulling from jupyter/jupyterhub
efd26ecc9548: Pull complete
a3ed95caeb02: Pull complete
298ffe4c3e52: Pull complete
758b472747c8: Pull complete
8b9809a68afc: Pull complete
93b253b5483d: Pull complete
ef8136abb53c: Pull complete
Digest: sha256:8f8cd2b62942b29e84bb99401ec9819d489b0cf0ebece99a42ba05946feeb72f
Status: Downloaded newer image for jupyter/jupyterhub:latest
root@766a9d01a6b4:/srv/jupyterhub# pip install moldesign
Collecting moldesign
  Downloading moldesign-0.7.3.tar.gz (20.0MB)
    100% |████████████████████████████████| 20.0MB 69kB/s
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/tmp/pip-build-jbo8ia1m/moldesign/setup.py", line 70
        print 'Thank you for installing the Molecular Design Toolkit!!!'
                                                                       ^
    SyntaxError: Missing parentheses in call to 'print'

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /tmp/pip-build-jbo8ia1m/moldesign/
You are using pip version 8.1.1, however version 8.1.2 is available.
You should consider upgrading via the 'pip install --upgrade pip' command.
111

It would be really great if there was a Dockerfile in the root directory that had the correct versions of everything to run and test, independent of the host setup.

Also tried to install and run on my machine. It installed ok with sudo, but could not run:
/usr/bin/python: No module named moldesign
✘-1 ~/autodesk/cloud-compute-cannon [stream_std_out_err_wip L|✚ 3⚑ 2]
19:26 $ which pip
/usr/local/bin/pip
✔ ~/autodesk/cloud-compute-cannon [stream_std_out_err_wip L|✚ 3⚑ 2]
19:27 $ which python
/usr/bin/python```

Thanks @dionjwa .

It doesn't install in the docker image because it's defaulting to Python 3, and MDT is still python 2 only. However, it should have printed out an error message stating that explicitly instead of dying with a syntax error - that's a bug.
Agreed, we need a root Dockerfile. Will add it.
Local installation doesn't work: :headdesk: Python + MacOS = sadness. Probably try switching to the Homebrew python install, or, per @hainm in #32, try pip install --user.

FYI @dionjwa - actually, the easiest way to deploy, as long as you're not doing development, is to pull the official docker image:
docker run -p 8888:8888 -it autodesk/moldesign:moldesign_notebook-0.7.3

Fixed with #108

authors: avirshup dionjwa

repo: Autodesk/molecular-design-toolkit


text:
Front-end Widget Catalogue

If the user can add front-end widgets to apps, where are those widgets stored, and how can they be referred to?
Deliverables:

Database of front-end widgets. (These could simply be tagged URLs for iframe widgets).
API to add/remove/edit widgets from the catalogue
Catalogue must support versioning
Policy for users to add their widgets (everything whitelisted or manually curated?)
Front-end Searching and viewing widgets.
Testing out widgets?

This ticket needs to be broken down when specced out more thoroughly.

Can this just be a publicly available github repo that we control final pull requests? This then specifies submission policies, potential license agreements, versioning.

@dionjwa - I really really like the idea of contributing workfows/apps via pull request to a master repository. This is exactly how Conda Forge manages its contributions. In addition to having a clear, well-understood contribution policy, the whole github infrastructure would also take care of the problems of scaling, community management, etc.

For right now, the priority is definitely on workflow development, so let's keep most of this stuff on the backlog for now.
We will definitely need a widget catalog for workflow/app developers; but as the widget themselves will just be MST components, this doesn't need to be anything more than a JSON document in the MST repo.

repo: Autodesk/molecular-simulation-tools


text:
Remove rendering when empty model data set on component

There was a bug where when you change the modelData to an empty instance (i.e. { atoms: [], bonds: [] }) the previously rendered model isn't removed. This can quickly be verified in npm run example.

Yup I was able to reproduce the bug.  I think this looks good.  I'm going to add another test or two just to prove to myself it works and then I think it's good to go.

authors: danielholmes justinmc


text:
Option to set visibility of answers to comment

What
I would like to be able to set the visibility of answers to „not listed“ in general and be able to set it for every comment before posting it on demand.
Why
Answering to many comments would clutter the timeline of folks just following the website.
The feature of sending answers to comments back to the fediverse is great but may be not usable in cases with a hugh ammount of interactions.
Sometimes if a comment from the fediverse gets big traction one might want to answer public in the timeline.
How
No response

I would like to be able to set the visibility of answers to „not listed“ in general and be able to set it for every comment before posting it on demand.

Ooh, this sounds like something I may want to add to my add-on plugin.

authors: janboddez jaschaurbach

repo: Automattic/wordpress-activitypub


text:
Parsing binding expression with type cast of attached property fails

Describe the bug
I'm attempting to bind to a property of the adorned element in a template set to the adorner property. To do this with compiled bindings a type cast is required but that fails to parse.
To Reproduce
  <TextBox>
    <TextBox.Styles>
      <Style Selector="TextBox">
        <Setter Property="(AdornerLayer.Adorner)">
          <Template>
            <TextBlock Margin="10 20 0 0" Foreground="Red" Text="{Binding $self.((TextBox)(AdornerLayer.AdornedElement)).Text}"/>
          </Template>
        </Setter>
      </Style>
    </TextBox.Styles>
  </TextBox>

This fails to compile.
Avalonia error AVLN:0004: Internal compiler error while transforming node XamlX.Ast.XamlAstObjectNode:
Avalonia error AVLN:0004: Avalonia.Data.Core.ExpressionParseException: Expected ')'.
Avalonia error AVLN:0004:    at Avalonia.Markup.Parsers.BindingExpressionGrammar.ParseTypeCast(CharacterReader& r, List`1 nodes) in /_/src/Markup/Avalonia.Markup/Markup/Parsers/BindingExpressionGrammar.cs:line 249
Avalonia error AVLN:0004:    at Avalonia.Markup.Parsers.BindingExpressionGrammar.Parse(CharacterReader& r) in /_/src/Markup/Avalonia.Markup/Markup/Parsers/BindingExpressionGrammar.cs:line 50
Avalonia error AVLN:0004:    at Avalonia.Markup.Xaml.XamlIl.CompilerExtensions.Transformers.AvaloniaXamlIlBindingPathParser.Transform(AstTransformationContext context, IXamlAstNode node) in /_/src/Markup/Avalonia.Markup.Xaml.Loader/CompilerExtensions/Transformers/AvaloniaXamlIlBindingPathParser.cs:line 28
Avalonia error AVLN:0004:    at XamlX.Transform.AstTransformationContext.Visitor.Visit(IXamlAstNode node) in /_/src/Markup/Avalonia.Markup.Xaml.Loader/xamlil.github/src/XamlX/Transform/AstTransformationContext.cs:line 58 Line 20, position 60.

With a ReflectionBinding it fails at runtime.
Avalonia.Data.Core.ExpressionParseException
  HResult=0x80131500
  Message=Expected ')'.
  Source=Avalonia.Markup
  StackTrace:
   at Avalonia.Markup.Parsers.BindingExpressionGrammar.ParseTypeCast(CharacterReader& r, List`1 nodes)
   at Avalonia.Markup.Parsers.BindingExpressionGrammar.Parse(CharacterReader& r)
   at Avalonia.Markup.Parsers.ExpressionParser.Parse(CharacterReader& r)
   at Avalonia.Markup.Parsers.ExpressionObserverBuilder.Parse(String expression, Boolean enableValidation, Func`3 typeResolver, INameScope nameScope)
   at Avalonia.Data.Binding.CreateExpressionObserver(AvaloniaObject target, AvaloniaProperty targetProperty, Object anchor, Boolean enableDataValidation)
   at Avalonia.Data.BindingBase.Initiate(AvaloniaObject target, AvaloniaProperty targetProperty, Object anchor, Boolean enableDataValidation)
   at Avalonia.AvaloniaObjectExtensions.Bind(AvaloniaObject target, AvaloniaProperty property, IBinding binding, Object anchor)
   at AvaloniaApplication3.MainWindow.XamlClosure_2.Build(IServiceProvider ) in MainWindow.axaml:line 20
   at Avalonia.Markup.Xaml.XamlIl.Runtime.XamlIlRuntimeHelpers.<>c__DisplayClass1_0`1.<DeferredTransformationFactoryV2>b__0(IServiceProvider sp)
   at Avalonia.Markup.Xaml.Templates.TemplateContent.Load(Object templateContent)
   at Avalonia.Markup.Xaml.Templates.Template.Build()
   at Avalonia.Markup.Xaml.Templates.Template.Avalonia.Styling.ITemplate.Build()
   at Avalonia.Styling.PropertySetterTemplateInstance.GetValue()
   at Avalonia.PropertyStore.EffectiveValue`1.GetValue(IValueEntry entry)
   at Avalonia.PropertyStore.EffectiveValue`1.SetAndRaise(ValueStore owner, IValueEntry value, BindingPriority priority)
   at Avalonia.PropertyStore.ValueStore.ReevaluateEffectiveValues(IValueEntry changedValueEntry)
   at Avalonia.PropertyStore.ValueStore.EndStyling()
   at Avalonia.StyledElement.ApplyStyling()
   at Avalonia.StyledElement.EndInit()
   at AvaloniaApplication3.MainWindow.!XamlIlPopulate(IServiceProvider , MainWindow ) in MainWindow.axaml:line 14
   at AvaloniaApplication3.MainWindow.!XamlIlPopulateTrampoline(MainWindow )
   at AvaloniaApplication3.MainWindow.InitializeComponent(Boolean loadXaml, Boolean attachDevTools) in Avalonia.Generators\Avalonia.Generators.NameGenerator.AvaloniaNameSourceGenerator\AvaloniaApplication3.MainWindow.g.cs:line 23
   at AvaloniaApplication3.MainWindow..ctor() in MainWindow.axaml.cs:line 14
   at AvaloniaApplication3.App.OnFrameworkInitializationCompleted() in App.axaml.cs:line 18
   at Avalonia.AppBuilder.SetupUnsafe()
   at Avalonia.AppBuilder.Setup()
   at Avalonia.AppBuilder.SetupWithLifetime(IApplicationLifetime lifetime)
   at Avalonia.ClassicDesktopStyleApplicationLifetimeExtensions.StartWithClassicDesktopLifetime(AppBuilder builder, String[] args, ShutdownMode shutdownMode)
   at AvaloniaApplication3.Program.Main(String[] args) in Program.cs:line 12

With ReflectionBinding you don't need the type cast so if you instead do {ReflectionBinding $self.(AdornerLayer.AdornedElement).Text} it works.
Expected behavior
No error and successful binding the same as when using the ReflectionBinding without type cast.
Environment:

OS: Windows
Avalonia-Version: 11.0.5

Additional context
It looks like BindingExpressionGrammar.ParseTypeCast expects ParseBeforeMember to parse the identifier but for attached properties it doesn't but instead returns State.AttachedProperty.
If I add parsing of an attached property after the call to ParseBeforeMember it looks like it works but I'm not well versed in the code base enough to know if that would be enough.
if (parseMemberBeforeAddCast)
{
	if (!ParseCloseBrace(ref r))
	{
		throw new ExpressionParseException(r.Position, "Expected ')'.");
	}

	result = ParseBeforeMember(ref r, nodes);

	// With this parsing a type cast of an attached property seems to work
	if (result == State.AttachedProperty)
		result = ParseAttachedProperty(ref r, nodes);

	if(r.Peek == '[')
	{
		result = ParseIndexer(ref r, nodes);
	}
}

        [Fact]
        public async Task Should_Get_Chained_Attached_Property_Value_With_TypeCast()
        {
            var expected = new Class1();

            var data = new Class1();
            data.SetValue(Owner.SomethingProperty, new Class1() { Next = expected });

            var target = ExpressionObserverBuilder.Build(data, "((Class1)(Owner.Something)).Next", typeResolver: (ns, name) => name == "Class1" ? typeof(Class1) : _typeResolver(ns, name));
            var result = await target.Take(1);

            Assert.Equal(expected, result);

            Assert.Null(((IAvaloniaObjectDebug)data).GetPropertyChangedSubscribers());
        }

        private static class Owner
        {
            public static readonly AttachedProperty<string> FooProperty =
                AvaloniaProperty.RegisterAttached<Class1, string>(
                    "Foo",
                    typeof(Owner),
                    defaultValue: "foo");

            public static readonly AttachedProperty<AvaloniaObject> SomethingProperty =
                AvaloniaProperty.RegisterAttached<Class1, AvaloniaObject>(
                    "Something",
                    typeof(Owner));
        }

@appel1 great findings and I like that you also directly added a unit test. If you want to, you may try to make a PR out of it.
This may help:

https://github.com/AvaloniaUI/Avalonia/wiki/Debugging-the-XAML-compiler
https://github.com/AvaloniaUI/Avalonia/blob/master/Documentation/build.md

repo: AvaloniaUI/Avalonia


text:
OpenGlControlBase enters OnOpenGlInit with OpenGL errors

Describe the bug
When implement OnOpenGlInit in an OpenGlControlBase and checking gl.GetError() for any error, there is one, presumably from something in Avalonia's code.
To Reproduce
Steps to reproduce the behavior:

Clone, build and run https://github.com/Dragorn421/DragoStuff/tree/ffe12b1003c3e6cd92419d7fdaab97b3c90c5dc4
Check the debug prints:

GL1.OnOpenGlInit
GL1.CheckError 1280
GL1.OnOpenGlRender
GL1.CheckError OK
GL1.CheckError OK
GL1.OnOpenGlRender
GL1.CheckError OK
GL1.CheckError OK

Notice GL1.CheckError 1280. It is printed by https://github.com/Dragorn421/DragoStuff/blob/ffe12b1003c3e6cd92419d7fdaab97b3c90c5dc4/MyOpenGLControl.cs#L29 , before the child control does any OpenGL call, so it seems to be an error originating from something in Avalonia.

Expected behavior
gl.GetError() should always return GL_NO_ERROR on entering child/user methods like OnOpenGlInit
Environment

OS: Kubuntu 23.10 (Linux, X11)
Avalonia-Version: 11.0.5

1280 is mean GL_INVALID_ENUM
Some operations use incorrect parameters, but this is related to the version of OpenGL

Still in Avalonia 11.0.10

@Dragorn421 if you wanted to check newer version, worth to try 11.1 betas

Still in Avalonia 11.1.1

authors: Coloryr Dragorn421 timunie


text:
Win32: Doesn't receive next key after Alt or F10

Describe the bug
When the Alt or F10 key is pressed, the next keypress is not received.
To Reproduce
Run the following program and press Alt or F10, followed by another key. The key subsequent to Alt or F10 is not registered.
public class MainWindow : Window
{
    public MainWindow()
    {
        this.InitializeComponent();
        this.AttachDevTools();
    }

    protected override void OnKeyDown(KeyEventArgs e)
    {
        System.Diagnostics.Debug.WriteLine("KeyDown " + e.Key);
        base.OnKeyDown(e);
    }
}

Desktop (please complete the following information):

OS: Windows
Version: master

Appears to be caused by our (not) handling of WM_ENTERIDLE?


text:
Correctly remove ContentPresenter's content from its parent host

Correctly remove ContentPresenter's content from its parent host when the content is updated while being detached from the logical tree.
I've updated the existing ContentControlTests.Should_Set_Child_LogicalParent_After_Removing_And_Adding_Back_To_Logical_Tree test with asserts that failed before, and now pass with this change. This seems acceptable to me as it already tested the right thing (but only from the child ⇒ parent side, now from both sides), tell me if you prefer a whole new test instead.
Fixes #11149

You can test this PR using the following package version. 11.0.999-cibuild0034002-beta. (feed url: https://pkgs.dev.azure.com/AvaloniaUI/AvaloniaUI/_packaging/avalonia-all/nuget/v3/index.json) [PRBUILDID]

Wow, thanks for finding that - looks like this bug's been in there forever!

Yay, this fixes https://github.com/AvaloniaUI/Avalonia/issues/9940 which was very annoying, thanks!

authors: BAndysc MrJul avaloniaui-team grokys


text:
Licencing and code usage

Hey !
I'm starting to write an addon aiming to help hunters with tranqshot rotation.
As I'm really new to addon, I used YaHT as a reference and inspiration for the base and announces part.
The addon should grow on his own way past a first basic release.
As our code is really similar and I found no licence there, I have to ask you if you are fine with this.
Repo at https://github.com/Slivo-fr/TranqRotate
Thanks for your time

Added a license.

Thanks <3

authors: Aviana Slivo-fr

repo: Aviana/YaHT

license: Unlicense


text:
为什么不添加一份答案呢？

我想给这些面试题添加一份答案，找答案的过程也是一种学习，并把它记录下来加深印象。你觉得如何？
https://github.com/Omooo/Android_QA

能做出来更好 前面的基础还可以做一下，后面有点高端的设计底层源码的问题就没那么容易整理了

的确是，然而时间也是能挤出来的，关键在于毅力吧。

authors: AweiLoveAndroid Omooo

repo: AweiLoveAndroid/CommonDevKnowledge


text:
Communication with Service Worker causes memory leak

Issue and Steps to Reproduce
Issue is reproducible on demo deployment https://icy-glacier-004ab4303.2.azurestaticapps.net/. When you log-in and app is opened for few minutes memory usage is increasing. It can be checked in Chrome Dev Tools/Memory panel.
Versions
7.22.8
Screenshots

Expected
When app is open (no actions), memory usage is not increasing over time.
Actual
I did few memory snapshots and noticed that most of new memory allocations are coming from MessagePort class. After digging into oidc-client code I found this util  used for Service Worker communication which creates MessageChannel but it's never closed after promise resolution.
Additional Details

Thank you very much @radk0s

hi @radk0s version 7.22.9 contains your fix!

Great, thanks @guillaume-chervet! Already checked and issue is gone.

repo: AxaFrance/oidc-client


text:
[BUG]: No Save/Download option with VSCode

Issue description
There is no option to save the rendered mapping with the VSCode Extension.  The Save and SaveAs do not seem to have any connection with the generated diagram
Media & Screenshots
No response
Operating system
Version: 1.75.1
Commit: 441438abd1ac652551dbe4d408dfcec8a499b8bf
Date: 2023-02-08T21:34:59.000Z
Electron: 19.1.9
Chromium: 102.0.5005.194
Node.js: 16.14.2
V8: 10.2.154.23-electron.0
OS: Darwin x64 21.6.0
Sandboxed: No
Priority this issue should have
Low (slightly annoying)

Merging under #29

repo: AykutSarac/jsoncrack-vscode


text:
Regular Expression -anshyyy

Description
Please include a summary of the change and which issue is fixed. List any dependencies that are required for this change.
Fixes #(issue_no)
Replace issue_no with the issue number which is fixed in this PR
Type of change
Please delete options that are not relevant.
Checklist:

[1 ] My code follows the style guidelines(Clean Code) of this project
[1 ] I have performed a self-review of my own code
[1 ] I have commented my code, particularly in hard-to-understand areas
[1] I have made corresponding changes to the documentation
[1] My changes generate no new warnings

@anshyyy issue number??

@anshyyy any update??


text:
Fix 31 issues

I have been studying this repository for a long time, and whenever I come across a typo, I fix it.
I discovered 31 typos in a total of 29 files. Then I merged them together.
The code quality is exceptionally high, so sincerely hope my PR can help make this repository more standardized.

We don't accept typo fixes directly, but thank you for your contribution.

authors: charlielye miles-six

repo: AztecProtocol/aztec-connect


text:
Temporary hack: remove public kernel checks that state updates are in correct order

Do #1616 (and maybe #1617) first
Followup: #1623

Will be closed by https://github.com/AztecProtocol/aztec-packages/pull/1685

repo: AztecProtocol/aztec-packages


text:
rollback FunctionAbi isTranspiled changes

Please read contributing guidelines and remove this line.

#5561  👈
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @fcarreiro and the rest of your teammates on  Graphite

Benchmark results
Metrics with a significant change:

note_trial_decrypting_time_in_ms (32): 112 (+218%)

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Values are compared against data from master at commit a581e80d and shown if the difference exceeds 1%.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
64 txs

l1_rollup_calldata_size_in_bytes
676
676
676

l1_rollup_calldata_gas
6,424
6,424
6,412

l1_rollup_execution_gas
585,757
585,757
585,745

l2_block_processing_time_in_ms
1,337 (+1%)
4,792 (-1%)
9,106 (+1%)

note_successful_decrypting_time_in_ms
248 (+5%)
601 (-4%)
1,036 (+3%)

note_trial_decrypting_time_in_ms
53.0 (-50%)
:warning: 112 (+218%)
32.7 (+27%)

l2_block_building_time_in_ms
11,985 (+1%)
43,796 (+1%)
86,804 (+1%)

l2_block_rollup_simulation_time_in_ms
7,182 (+1%)
24,860
47,857 (-1%)

l2_block_public_tx_process_time_in_ms
4,762 (+1%)
18,807 (+1%)
38,705 (+3%)

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
14,048 (-2%)
26,406

note_history_successful_decrypting_time_in_ms
1,272
2,470 (+4%)

note_history_trial_decrypting_time_in_ms
93.0 (+29%)
118 (+15%)

node_database_size_in_bytes
18,657,360
35,082,320 (+1%)

pxe_database_size_in_bytes
29,859
59,414

Circuits stats
Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit
circuit_simulation_time_in_ms
circuit_input_size_in_bytes
circuit_output_size_in_bytes

private-kernel-init
181
44,377
26,164

private-kernel-ordering
162
50,830
39,325

base-parity
4,366
128
311

root-parity
1,171 (+1%)
1,244
311

base-rollup
14,514
116,608
861

root-rollup
49.9
4,359
725

private-kernel-inner
221 (+1%)
71,744
26,164

public-kernel-app-logic
122 (+2%)
47,695
40,661

public-kernel-tail
165 (+1%)
53,372
13,269

merge-rollup
10.1
2,568
861

public-kernel-teardown
119 (+1%)
47,695
40,661

public-kernel-setup
118 (+1%)
47,695
40,661

Tree insertion stats
The duration to insert a fixed batch of leaves into each tree type.

Metric
1 leaves
16 leaves
64 leaves
128 leaves
512 leaves
1024 leaves
2048 leaves
4096 leaves
32 leaves

batch_insert_into_append_only_tree_16_depth_ms
10.1 (+1%)
16.1
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_count
16.8
31.6
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_16_depth_hash_ms
0.587
0.496
N/A
N/A
N/A
N/A
N/A
N/A
N/A

batch_insert_into_append_only_tree_32_depth_ms
N/A
N/A
46.5
72.7
231 (-1%)
448
873
1,759 (+1%)
N/A

batch_insert_into_append_only_tree_32_depth_hash_count
N/A
N/A
96.0
159
543
1,055
2,079
4,127
N/A

batch_insert_into_append_only_tree_32_depth_hash_ms
N/A
N/A
0.476
0.448
0.420 (-2%)
0.418 (+1%)
0.413 (-1%)
0.421 (+1%)
N/A

batch_insert_into_indexed_tree_20_depth_ms
N/A
N/A
54.4
107 (-1%)
339 (+1%)
663
1,309 (-2%)
2,614
N/A

batch_insert_into_indexed_tree_20_depth_hash_count
N/A
N/A
105
207
691
1,363
2,707
5,395
N/A

batch_insert_into_indexed_tree_20_depth_hash_ms
N/A
N/A
0.479
0.482
0.458
0.456
0.452 (-2%)
0.454
N/A

batch_insert_into_indexed_tree_40_depth_ms
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
61.6

batch_insert_into_indexed_tree_40_depth_hash_count
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
109

batch_insert_into_indexed_tree_40_depth_hash_ms
N/A
N/A
N/A
N/A
N/A
N/A
N/A
N/A
0.535

Miscellaneous
Transaction sizes based on how many contract classes are registered in the tx.

Metric
0 registered classes
1 registered classes

tx_size_in_bytes
40,548
501,142

Transaction size based on fee payment method

Metric
native fee payment method
fpc_public fee payment method
fpc_private fee payment method

tx_with_fee_size_in_bytes
905
1,161
1,377

Transaction processing duration by data writes.

Metric
0 new note hashes
1 new note hashes
2 new note hashes

tx_pxe_processing_time_ms
1,751
1,098
5,635 (+2%)

Metric
1 public data writes
2 public data writes
3 public data writes
4 public data writes
5 public data writes
8 public data writes

tx_sequencer_processing_time_ms
587 (+2%)
443
1,081
604 (+1%)
1,768
597 (+1%)

authors: AztecBot fcarreiro


text:
対応アバター追加

T.Garden様のももかちゃんのボーン構造をBoneNames.xmlに追記しました

こまいぬ通り様のネメシスちゃん、並びにリィちゃん(並びに共通素体のきなこちゃん)のボーン構造を追記しました

repo: Azukimochi/BoneRenamer


text:
Example pipeline (i.e HELLOWORLD pipeline) to use local kubernetes (external silo) PVC data for training

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Don't find the syntax to use local k8s data in external silo for FL training. Followed these steps to create PVC - https://github.com/Azure-Samples/azure-ml-federated-learning/blob/main/docs/tutorials/read-local-data-in-k8s-silo. How do I modify the HELLOWORLD pipeline to use this local PVC data for training?
Describe the solution you'd like
A clear and concise description of what you want to happen.
Provide example pipline config files to use local k8s silo data.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Searched for syntax both in this repo and Azure AML K8s repo. Only method for creating and exposing PVC is provided. Could not find yaml syntax for pointing to local mountpath,
Additional context
Add any other context or screenshots about the feature request here.

Hi @amahab !
First of all, thanks for filing an issue. We will address it and provide detailed instructions in the repo on what needs to be done to consume local data.
However, for the time being, please let me provide an answer here to unblock you as fast as possible. The key difference between reading local data vs consuming data from an Azure ML data asset is that for local data one cannot use uri_file or uri_folder input parameter like in this config file for instance. Instead, one needs to use a string input. To implement that, you will need to do 2 things:

have your "read local data" component accept a string input in its spec, and have the component code use this string as needed;
provide the value of the string parameter in the config.yml used to submit the job.

We did have a very basic example for that in the initial version of this PR, but at the last minute we decided to drop it. The good news is, the files are still available in GitHub.

The component spec can be found here; see that string input called local_data_path.
The component code can be found here; see how the input parameter value is being used .
The job config.yml is there, and the associated submission script is over there. They both use the parameter defined in the component spec.
The documentation, at that time, had a section  on how to configure and run a test job, that was leveraging the files above.

Hopefully this is enough to unblock you, but if not feel free to ask follow-up questions :)

Hi,
Thanks for the response and pointing me to a version of this repo that has this example. Upon inspection of the submit script, I notice that pre-processed data is written back to the Azure cloud datastore for the silo. Is there example where all data remains local on the pvc (r/w mount) and only trained model parameters are written to the silo Azure datastore?
The benefit of external k8s silo is avoiding costly data movement to the cloud. E.g. A scenarios where multiple edge geo sites are generating data. The ability to train the data locally without data movement to Azure and aggregating only model weights via FL is useful.
Appreciate the follow up :-)

No, we don't have that other example currently @amahab .

@amahab as @thomasp-ms mentioned, we don't have another example like that. Overall, you could apply the same kind of guidance for the preprocessing step, although there might be some complications.
One question: how would you expect the interactions between the preprocessing and training when using local mount points in k8s?
For instance:

do you expect the preprocessing to write the data in a unique path (ex: use a run id as subfolder) in the local, then give this unique path down to the training to locate the preprocessed data?
or do you expect the preprocessing to just write in some hardcoded location?

@jfomhover
Preprocessing can write data to a runid subfolder on local and give that part for training.
The whole use case i'm evaluating is that data is generated at the data center/edge locations. These locations train the data locally without moving data to the cloud. Only FL model weights are sent to Azure Workspace for updating the wights. New data will come into the locations at a later time and model gets re-trained locally with updated model sent from cloud.

authors: amahab jfomhover thomasp-ms


text:
Use create_all for index creation

This PR uses the already existing create_all call to create indexes, so no additional create call is needed.
Also added missing index to async example.

@nachoalonsoportillo I think you mentioned the missing index in async, this fixes that.


text:
ERROR: deployment failed: error deploying infrastructure: deploying to subscription:

Please provide us with the following information:

This issue is for a: (mark with an x)
- [x] bug report -> please search issues before submitting
- [ ] feature request
- [x] documentation issue or request
- [ ] regression (a behavior that used to work and stopped in a new release)

Minimal steps to reproduce

running azd up

Any log messages given by the failure

ERROR: deployment failed: error deploying infrastructure: deploying to subscription:

Deployment Error Details:
Conflict: No available instances to satisfy this request. App Service is attempting to increase capacity. Please retry your request later. If urgent, this can be mitigated by deploying this to a new resource group.

No available instances to satisfy this request. App Service is attempting to increase capacity. Please retry your request later. If urgent, this can be mitigated by deploying this to a new resource group.

Expected/desired behavior

the app service plan is expected to deploy with no error

OS and Version?

Windows 11

Versions

22H2

Mention any other details that might be useful
nothing to mention the error detail provides all the info,also i tried making a new resource group just as it recommended but i got the same error

Thanks! We'll be in touch soon.

I am receiving the same error when trying to deploy to East US. Any update on this issue?

authors: FunkyDialUpDude IhebGhazala

repo: Azure-Samples/azure-search-openai-demo


text:
Installtion Setup

Dear Team,
When I am planning to execute the setup getting the following error:
ERROR: deployment failed: failing invoking action 'provision', error deploying infrastructure: deploying to subscription:
Deployment Error Details:
InvalidTemplateDeployment: The template deployment 'openai' is not valid according to the validation procedure. The tracking id is '6777928d-1567-4630-b286-8f77cbf7cacf'. See inner errors for details.
InsufficientQuota: The specified capacity '120' of account deployment is bigger than available capacity '0' for UsageName 'Tokens Per Minute (thousands) - GPT-35-Turbo'.
TraceID: 09f9bd8b6cdac42e96b5848442b7a646
But here I am trying to use the existing the deployed gpt & chat model in spite of why it is going to create new deployment model?
AZURE_OPENAI_CHATGPT_DEPLOYMENT="davinci"
AZURE_OPENAI_GPT_DEPLOYMENT="chat"
AZURE_OPENAI_RESOURCE_GROUP="XXXX"
AZURE_OPENAI_SERVICE="XXXX"
am I missing anything can you please help on the same
Thanks,
Bhaskar

I am getting the same error and I was able to successfully deploy last Friday.

The same here.

same

same

I have the same issue since this week indeed.
If you need 120 for example, but it is currently taking 120/120 , it still demands another 120 units to be free, although no extra units would be needed.
Temporary fix for us is to go to quotas and set all current deployments to a very low number such that you have enough free space again. You'd have to do this for every deployment, so I am also waiting for a fix.

I have the same issue since this week indeed. If you need 120 for example and it is currently taking 120/120 , it still demands another 120 units to be free, although no extra units would be needed in practice. Temporary fix for us is to go to quotas and set all current deployments to a very low number such that you have enough free space again. You'd have to do this for every deployment, so I am also waiting for a fix.

I'm having trouble changing the quotas, where is it?

Thanks for advise I had updated the quota in model the deployment is going fine.. but eventually we need to wait for fix form MSFT.

so, temporary solution is to go to Azure AI studio, Deployments, and for each  Edit deployment, Advanced, set "Tokens per Minute Rate Limit" from 120 to e.g. 2. Then deploy your accelerator

This is very inconvenient - having to change quotas manually, especially if the infra is deployed frequently. I hope it's gonna get fixed soon.

How do you decrease the quota for the service you are deploying?
Let's presume that you are using 120/120.
You decrease it to 1/120.
Then it tries to deploy another 120 and complains that it is more than the free quota which is 119.

This https://github.com/Azure-Samples/azure-search-openai-demo/commit/f5a7095e44bd45e224c3200874be36e93e4e0579 should help.  Can you either get that patch into your current code or start from scratch and try again?

authors: GregLin0420 TimoRiikonenIF bhasampa clojosh jjczopek jongio paprocki-r silvio-machado tickx-cegeka


text:
Azure SQL -> Should be SQL Server, there's no Azure SQL used in this sample project.

Instances of Azure SQL mentioned in README and sample title should reflect that this sample uses SQL Server 2022 (Linux). I'm missing where Azure SQL is used. I supposed you'd need to update the Readme to indicate that you could publish the dacpac to Azure SQL if desired. Then document where and how to update the DB connection string in the sample to point to Azure SQL and not localhost.

Thanks for the feedback! This issue was partially fixed on this commit: 06ff3078ffe49284f0d95fdcc66fcd6b20f5e994
The SQL Database project is created for an Azure SQL Database target, thus the database schema is Azure SQL Database compatible. I added a note to explain this situation here.

authors: croblesm seantleonard


text:
Remove unnecessary semicolon

Purpose

This remove unnecessary semicolon in Swift.
Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[x] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

/azp run


text:
HackTogether: Fix feature listing for registered users

Are you accessing the project from the GitHub Codespaces?

[ ] No, I am running the project on my local machine (I will fill in the OS section below)
[x] Yes, I am using GitHub Codespaces.

Note: The project is optimized to run on Codespaces. We welcome contributions to enhance it for cross-platform local development, but it's not something we will prioritize right now. Thank you!

Describe the bug

The bookmark icon is too close to the image.
The Property description is overlapping the bookmark icon.
The short description for the listed property could be removed in order to create a minimalistic design. This description could be assessed when the user wants to know more about the property.

To Reproduce
Steps to reproduce the behavior

Go to home page after login

Expected behavior

I expect the bookmark icon to be properly aligned/placed
The property name/description should not overlap the bookmark icon
A more aesthetic property listing

Screenshots

OS (please complete the following information):

OS:
OS Version:
npm version:
Node.js version:
output of azd version:

Additional context
Having the long description removed from the featured listing will let each card have equal height.

Hi @mzbhewtee Thanks for reporting! We suggest you investigate this issue as related to Angular material design. Thank you for participating. We look forward to your PR!

Hi @anfibiacreativa Thank you, i will investigate the issue.

Suggestion to tackle with issue #242

Removing long description is not ideal recommendation. we can properly customize scss for having responsive cards with appropriate dimensions.

I am also willing to contribute and investigate together @mzbhewtee

Do we have any updates? Thank you!

repo: Azure-Samples/contoso-real-estate


text:
fix: updated command for package installs for the notification server (ReadMe)

Issue #451
Readme instruction

Current output

Target output

Hi @juliamuiruri4 your change LGTM. Can you change your commit message to follow these guidelines.

Done @manekinekko
Let me know if I should change anything else. Thanks


text:
Adding pop tokens sample

Purpose

Adds a new sample where React SPA acquiring PoP tokens to call Node.js web API which validates the token using jose

cc @hectormmg
Does this introduce a breaking change?

[ ] Yes
[ x ] No

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ x ] Other... Please describe: **new sample**

How to Test

Get the code

git clone https://github.com/Azure-Samples/ms-identity-javascript-react-tutorial.git
cd ms-identity-javascript-react-tutorial
git checkout pop-sample
cd 6-AdvancedScenarios/2-call-api-pop/API
npm install
cd ../SPA
npm install

What to Check
Verify that the following are valid

Test in Chrome, Edge, Firefox
Test in private mode

"dependencies": {

delete this file?

Refers to: 3-Authorization-II/1-call-api/SPA/package-lock.json:6 in 3e8d6a7. [](commit_id = 3e8d6a7d8c3a8f054648fc291b1fd33a202924dc, deletion_comment = False)


text:
Deployment fails when using Custom Networking and when using custom Address space

Describe the bug
When using AKS Construction helper, the deployment fails with the following error:
{"status":"Failed","error":{"code":"DeploymentFailed","message":"At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/DeployOperations for usage details.","details":[{"code":"Conflict","message":"{\r\n  \"status\": \"Failed\",\r\n  \"error\": {\r\n    \"code\": \"ResourceDeploymentFailure\",\r\n    \"message\": \"The resource operation completed with terminal provisioning state 'Failed'.\",\r\n    \"details\": [\r\n      {\r\n        \"code\": \"DeploymentFailed\",\r\n        \"message\": \"At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/DeployOperations for usage details.\",\r\n        \"details\": [\r\n          {\r\n            \"code\": \"BadRequest\",\r\n            \"message\": \"{\\r\\n  \\\"error\\\": {\\r\\n    \\\"code\\\": \\\"NetcfgInvalidSubnet\\\",\\r\\n    \\\"message\\\": \\\"Subnet 'AzureFirewallManagementSubnet' is not valid in virtual network 'vnet-aks-rv-test'.\\\",\\r\\n    \\\"details\\\": []\\r\\n  }\\r\\n}\"\r\n          }\r\n        ]\r\n      }\r\n    ]\r\n  }\r\n}"}]}}

It appears that even when no Azure Firewall is requested in the helper, the helper attempts to create an AzureFirewallManagementSubnet
To Reproduce
Steps to reproduce the behavior:

Go to Networking Details tab
Under Network plugin, select "Use CNI..."
Under Default or Custom VNET, select Custom Networking
VNET Address space, anything, but the default space. EX: 10.200.0.0/16
AKS Nodes Subnet, anything within the range. EX: 10.200.0.0/20

Alternatively, it can be recreated it by: URL for AKS-Construction helper config
# Create Resource Group
az group create -l EastUS2 -n aks-rv-test-rg

# Deploy template with in-line parameters
az deployment group create -g aks-rv-test-rg  --template-uri https://github.com/Azure/AKS-Construction/releases/download/0.9.5/main.json --parameters \
	resourceName=aks-rv-test \
	kubernetesVersion=1.25.2 \
	upgradeChannel=stable \
	SystemPoolType=Standard \
	nodePoolName=npsymuser01 \
	agentCountMax=20 \
	osDiskType=Managed \
	custom_vnet=true \
	vnetAddressPrefix=10.200.0.0/19 \
	vnetAksSubnetAddressPrefix=10.200.0.0/21 \
	enable_aad=true \
	AksDisableLocalAccounts=true \
	enableAzureRBAC=true \
	adminPrincipalId=$(az ad signed-in-user show --query id --out tsv) \
	registries_sku=Standard \
	acrPushRolePrincipalId=$(az ad signed-in-user show --query id --out tsv) \
	enableTelemetry=false \
	omsagent=true \
	retentionInDays=30 \
	fileCSIDriver=false \
	diskCSIDriver=false \
	keyVaultAksCSI=true \
	keyVaultCreate=true \
	keyVaultOfficerRolePrincipalId=$(az ad signed-in-user show --query id --out tsv) \
	keyVaultAksCSIPollInterval=5m \
	fluxGitOpsAddon=true

Expected behavior
Deployment succeeds, and provisioned VNet does not have an AzureFirewallManagementSubnet created without requesting it.

I think this could have been fixed in #480 and just needs released.
@MattLeach25 - thoughts?

I think this could have been fixed in #480 and just needs released. @MattLeach25 - thoughts?

Yeah this is fixed with the new release.

We'll aim to cut a release tomorrow.
@romerve - if you clone the repo and target bicep/main.bicep instead of the release - then you should be good.

@MattLeach25 created a release today, so this should be fixed.

authors: Gordonby MattLeach25 romerve

repo: Azure/AKS-Construction


text:
Setting codespaces memory requirement

PR Summary
Closes #289
PR Checklist

[x] PR has a meaningful title
[x] Summarized changes
[x] This PR is ready to merge and is not Work in Progress
[x] Link to a filed issue

approve, but no way react dev takes 8GB, only takes 500MB locally

authors: Gordonby khowling


text:
26808 - Feature - Add support to allow Private DNS Zone VNet Linking for Spoke VNets

Overview/Summary
Adding support to allow Private DNS Zone VNet Linking for Spoke VNets through HubPeeredSpoke orchestration module and a new Private Dns Zone Linking module.
AB#26808
This PR fixes/adds/changes/removes

Added feature to HubPeeredSpoke orchestration module to allow Private DNS Zone Linking to Spoke VNets.
Added new module for Private DNS Zone Linking.

Breaking Changes

N/A

Testing Evidence
Evidence to be posted.
As part of this Pull Request I have

[X] Read the Contribution Guide and ensured this PR is compliant with the guide
[X] Ensured the resource API versions in .bicep file/s I am adding/editing are using the latest API version possible
[X] Checked for duplicate Pull Requests
[X] Associated it with relevant GitHub Issues
[X] (ALZ Bicep Core Team Only) Associated it with relevant ADO Items
[X] Ensured my code/branch is up-to-date with the latest changes in the main branch
[X] Performed testing and provided evidence.
[X] Updated one or more of the following tests (if required)

Unit
Linting
E2E (End-To-End)
ValidateAzCloud (Base validation in Azure Cloud)
ValidateMcCloud (Base validation in Azure China Cloud)

[X] Updated relevant and associated documentation (e.g. Contribution Guide, Module READMEs, Wiki Docs etc.)
[X] If relevant, created or updated Code Tours here

New module working - orchestration module test to be done

/azp run validateazcloud

/azp run e2e

/azp run validateazcloud

repo: Azure/ALZ-Bicep


text:
Add NRMS/Hypernet tags to cluster resource group in CI mode

Which issue this PR addresses:
The RP fails cluster installs if NSGs are attached to subnets. Simply Secure v1 applies NSGs in our subscriptions when these tags aren't present, resulting in a race condition.

What this PR does / why we need it:
Adds the tags needed to stop SSv1 from applying NSGs to subnets: https://eng.ms/docs/security-compliance-identity-and-management-scim/security/azure-security/security-health-analytics/network-isolation/netiso-teamdocs/netiso-program-overview/manage/hypernet-and-nrms-simply-secure-v1-network-security-rules

Test plan for issue:
Deploy this commit to central us euap (where the issue is faced)

Is there any documentation that needs to be updated for this PR?
No

/approve

hold for now as tagging the resource group didn't have the desired effect. We'll need to update these tags with a temporary SSv1 exception number and re-test.

@ross-bryan we can open a new PR with exception tags later if needed. Closing this one.

authors: cadenmarchese rogbas

repo: Azure/ARO-RP


text:
Fix Purge MSFT CI Subscription

Which issue this PR addresses:

Fixes
What this PR does / why we need it:

Test plan for issue:

Is there any documentation that needs to be updated for this PR?

Also fixes Purge Red Hat Dev-Gratis: https://msazure.visualstudio.com/AzureRedHatOpenShift/_build/results?buildId=69797295&view=logs&j=a4f1910f-c367-5697-edcd-724d81cde23b&t=2152b647-0197-5277-e95c-5d58fca9c7d3


text:
Set maintenace task to "" after  PUCM pending operation

Which issue this PR addresses:

Fixes bug introduced by https://github.com/Azure/ARO-RP/pull/3194
What this PR does / why we need it:

We never unset PUCM pending maintenance task
If we do admin update with no maintenance task set, we're suppose to do a complete admin update, but instead we get PUCM pending task completed again

Test plan for issue:

Unit
INT

Is there any documentation that needs to be updated for this PR?

No - bug fix

/azp run e2e

authors: bennerv niontive


text:
refactor: lint errors; strings, undefinied nums, global vars

What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Requirements:

[ ] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

Pull Request Test Coverage Report for Build 4612982352

44 of 126 (34.92%)  changed or added relevant lines in 8 files are covered.
2 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-2.3%) to 77.079%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/agent/utils.go
9
13
69.23%

pkg/agent/datamodel/types.go
12
17
70.59%

pkg/agent/datamodel/sig_config.go
0
73
0.0%

Files with Coverage Reduction
New Missed Lines
%

pkg/agent/datamodel/sig_config.go
1
42.29%

pkg/agent/datamodel/types.go
1
80.43%

Totals

Change from base Build 4612232559:
-2.3%

Covered Lines:
2011

Relevant Lines:
2609

💛  - Coveralls

repo: Azure/AgentBaker


text:
docs: add Windows image release notes for 2024-3b

What type of PR is this?
/kind documentation

What this PR does / why we need it:
add Windows image release notes for 2024-3b. Reference: #4175
Which issue(s) this PR fixes:

Fixes #
Requirements:

[x] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

Pull Request Test Coverage Report for Build 8321780001
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 79.343%

Totals

Change from base Build 8304229022:
0.0%

Covered Lines:
2293

Relevant Lines:
2890

💛  - Coveralls


text:
Event grid subscriptions no longer working

I had created some EventGrid subscriptions to a preview app configuration instance in East US. Today I can no longer see the subscription and receive the following error when clicking on Events

Are event grid subscriptions really no longer available in East US or is this a UI bug?

@nhart12 This is a UI bug, working on resolving it. Thanks for notifying us!

This has been resolved.

Thanks @jimmyca15 the UI seems to work now, the events however are not firing consistently in our app configuration instance

@nhart12. Is this happening in a single region with a single store? Do you mean events are firing sometimes, and then not firing other times or that they're not firing at all?

@jimmyca15 So this is happening in our East US store. Just tried again this morning, created a couple settings, updated them several times and then deleted. I have an EventGrid subscription setup for key-value Modified and Deleted. Only the deleted events fired

There was a bug in event grid publishing identified last week. We are in the process of rolling out our fix for that. That may be causing your issue. When the fix is rolled out I will let you know.

Ok perfect thanks for the info @jimmyca15 !

@jimmyca15 Can you provide an update regarding when you anticipate the event grid publishing issues may be resolved?  Also, is a GA estimate available...the most recent discussion around this mentioned October timeframe, is that still a possibility?

@jmsalvo event grid publishing changes are getting rolled out region by region. Expected to all regions by thursday eod.
Estimated GA is still end of October. Thanks.

Yesterday we deployed a fix to event grid to improve some event publishing reliability issues we were facing. Please open another issue if you are still experiencing problems.

Hi @nhart12 and @jmsalvo, I want to give you an update. We just posted our GA and pricing plan. Please check it out.
https://github.com/Azure/AppConfiguration-Announcements/issues/2

repo: Azure/AppConfiguration


text:
🪲 Bug Report - Upgrade Fails for SF VMSS with KeyVault Extension

Describe the bug
Upgrade fails due to KV VM extension timeout:
2023-06-14T21:07:57+00 [Information]:############################## Initializing Start-AzBasicLoadBalancerUpgrade ##############################
2023-06-14T21:07:57+00 [Information]:[Start-AzBasicLoadBalancerUpgrade] PowerShell Version: **7.3.4**
2023-06-14T21:07:57+00 [Information]:[Start-AzBasicLoadBalancerUpgrade] AzureBasicLoadBalancerUpgrade **Version: 2.0.19**
...
2023-06-14T21:08:00+00 [Information]:[Test-SupportedMigrationScenario] Checking whether VMSS scale set 'quotavmssdevbn' is a Service Fabric cluster...
WARNING: 2023-06-14T21:08:00+00 [Warning]:[Test-SupportedMigrationScenario] **VMSS appears to be a Service Fabric** cluster based on extension profile. SF Clusters experienced potentically significant downtime during migration using this PowerShell module. In testing, a 5-node Bronze cluster was unavailable for about 30 minutes and a 5-node Silver cluster was unavailabile for about 45 minutes. Shutting down the cluster VMSS prior to initiating migration will result in a more consistent experience of about 5 minutes to complete the LB migration. For Service Fabric clusters that require minimal / no connectivity downtime, adding a new nodetype with standard load balancer and IP resources is a better solution.
Do you want to proceed with the migration of your Service Fabric Cluster's Load Balancer?
...
2023-06-14T21:08:00+00 [Information]:[PublicLBMigration] **Public Load Balancer Detected**. Initiating Public Load Balancer Migration
...
2023-06-14T21:24:34+00 [Information]:[NatRulesMigration] Waiting for saving standard load balancer LB-quota-cluster-dev-bn job to complete...
2023-06-14T21:24:34+00 [Information]:[NatRulesMigration] Nat Rules Migration Completed
2023-06-14T21:24:34+00 [Information]:[InboundNatPoolsMigration] Initiating Inbound NAT Pools Migration
2023-06-14T21:24:34+00 [Information]:[InboundNatPoolsMigration] Adding Inbound NAT Pool LoadBalancerBEAddressNatPool to Standard Load Balancer
2023-06-14T21:24:34+00 [Information]:[InboundNatPoolsMigration] Saving Standard Load Balancer LB-quota-cluster-dev-bn
2023-06-14T21:24:49+00 [Information]:[InboundNatPoolsMigration] Waiting for saving standard load balancer LB-quota-cluster-dev-bn job to complete...
2023-06-14T21:24:49+00 [Information]:[GetVmssFromBasicLoadBalancer] Initiating GetVmssFromBasicLoadBalancer
2023-06-14T21:24:49+00 [Information]:[GetVmssFromBasicLoadBalancer] Getting VMSS object '/subscriptions/.../resourcegroups/azure-quota-dev-eastus2/providers/microsoft.compute/virtualmachinescalesets/quotavmssdevbn' from Azure
2023-06-14T21:24:49+00 [Information]:[GetVmssFromBasicLoadBalancer] VMSS loaded Name quotavmssdevbn from RG azure-quota-dev-eastus2
2023-06-14T21:24:49+00 [Information]:[_MigrateNetworkInterfaceConfigurations] Adding InboundNATPool to VMSS quotavmssdevbn
2023-06-14T21:24:49+00 [Information]:[_MigrateNetworkInterfaceConfigurations] Checking if VMSS 'quotavmssdevbn' NIC 'NIC-azure-quota-dev-eastus2' IPConfig 'NIC-azure-quota-dev-eastus2' should be associated with NAT Pool 'LoadBalancerBEAddressNatPool'
2023-06-14T21:24:49+00 [Information]:[_MigrateNetworkInterfaceConfigurations] Adding NAT Pool 'LoadBalancerBEAddressNatPool' to IPConfig 'NIC-azure-quota-dev-eastus2'
2023-06-14T21:24:49+00 [Information]:[_MigrateNetworkInterfaceConfigurations] Migrate NetworkInterface Configurations completed
2023-06-14T21:24:49+00 [Information]:[InboundNatPoolsMigration] Saving VMSS quotavmssdevbn
2023-06-14T21:24:49+00 [Information]:[UpdateVmss] Updating configuration of VMSS 'quotavmssdevbn'
2023-06-14T21:25:04+00 [Information]:[UpdateVmss] Waiting for job (id: '5') updating VMSS 'quotavmssdevbn' to complete...
...
2023-06-14T23:10:50+00 [Information]:[UpdateVmss] Waiting for job (id: '5') updating VMSS 'quotavmssdevbn' to complete...
InvalidOperation: Long running operation failed with status 'Failed'. Additional Info:'Provisioning of VM extension **KvVmExtension** has timed out. Extension provisioning has taken too long to complete. The extension did not report a message. More information on troubleshooting is available at https://aka.ms/vmextensionwindowstroubleshoot'
ErrorCode: VMExtensionProvisioningTimeout
ErrorMessage: Provisioning of VM extension KvVmExtension has timed out. Extension provisioning has taken too long to complete. The extension did not report a message. More information on troubleshooting is available at https://aka.ms/vmextensionwindowstroubleshoot
ErrorTarget: 0
StartTime: 6/14/2023 9:24:52 PM
EndTime: 6/14/2023 11:10:27 PM
OperationID: 85ee53b5-9ce3-4458-9edd-f46e8c7baf02
Status: Failed
Write-Error: 2023-06-14T23:10:50+00 [Error]:[InboundNatPoolsMigration] An error occured when attempting to update VMSS network config on the new Standard LB backend pool membership. To recover address
the following error, and try again specifying the -FailedMigrationRetryFilePath parameter and Basic Load Balancer backup State file located either in this directory or the directory
specified with -RecoveryBackupPath

To Reproduce
Steps to reproduce the behavior:

VMSS
Public LB
KVVMExtension [this case, extension adds a cert to local store, auto upgrade disabled]
SF Cluster [?]

Additional context - please include:
See log

Trying to test this scenario, I can't get the LB migration to even start due to:
InvalidOperation: Long running operation failed with status 'Failed'. Additional Info:'VM has reported a failure when processing extension 'vmss-01'. Error message: "Failed to parse the configuration settings with: 'not a string'"

More information on troubleshooting is available at https://aka.ms/vmextensionwindowstroubleshoot '
ErrorCode: VMExtensionProvisioningError
ErrorMessage: VM has reported a failure when processing extension 'vmss-01'. Error message: "Failed to parse the configuration settings with: 'not a string'"

This issue is reportedly intermittent...still working to repro

Closing due to lack of activity and reproducibility

@mbrat2005 Im experiencing the same issue. Did you ever find a solution?

Hi Andrew,
I haven't made progress on this one, since I couldn't seem to repro it. Would you be able to share your upgrade log for details? Also, are you upgrading a basic LB for a Service Fabric Cluster?
Thanks!
Matthew

From: Andrew Smith @.>
Sent: Wednesday, July 3, 2024 13:29
To: Azure/AzLoadBalancerMigration @.>
Cc: Mention @.>; Author @.>; Comment @.>; Assign @.>; State change @.***>
Subject: Re: [Azure/AzLoadBalancerMigration] 🪲 Bug Report - Upgrade Fails for SF VMSS with KeyVault Extension (Issue #65)
@mbrat2005https://github.com/mbrat2005 Im experiencing the same issue. Did you ever find a solution?
—
Reply to this email directly, view it on GitHubhttps://github.com/Azure/AzLoadBalancerMigration/issues/65#issuecomment-2207038571 or unsubscribehttps://github.com/notifications/unsubscribe-auth/AGBW6WHTSMOHXES4YPWH4F3ZKRGLFBFKMF2HI4TJMJ2XIZLTSWBKK5TBNR2WLJDUOJ2WLJDOMFWWLO3UNBZGKYLEL5YGC4TUNFRWS4DBNZ2F6YLDORUXM2LUPGBKK5TBNR2WLJDUOJ2WLJDOMFWWLLTXMF2GG2C7MFRXI2LWNF2HTAVFOZQWY5LFUVUXG43VMWSG4YLNMWVXI2DSMVQWIX3UPFYGLAVFOZQWY5LFVI2DKNRTHEYTONZQHGSG4YLNMWUWQYLTL5WGCYTFNSBKK5TBNR2WLKRVGU2DSOJZGYZDQM5ENZQW2ZNJNBQXGX3MMFRGK3FMON2WE2TFMN2F65DZOBS2YSLTON2WKQ3PNVWWK3TUUZ2G64DJMNZZJAVEOR4XAZNKOJSXA33TNF2G64TZUV3GC3DVMWUTKMZZHE4TSOJYG6BKI5DZOBS2K2LTON2WLJLWMFWHKZNKGE3TKOJTGQZDEMZQQKSHI6LQMWSWYYLCMVWKK5TBNR2WLKRUGU3DGOJRG43TAOMCUR2HS4DFUVWGCYTFNSSXMYLMOVS2UNJVGQ4TSOJWGI4DHJ3UOJUWOZ3FOKTGG4TFMF2GK.
You are receiving this email because you were mentioned.
Triage notifications on the go with GitHub Mobile for iOShttps://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Androidhttps://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

@AndrewCS149 I haven't made progress on this one, since I couldn't seem to repro it. Would you be able to share your upgrade log for details? Also, are you upgrading a basic LB for a Service Fabric Cluster?

repo: Azure/AzLoadBalancerMigration


text:
Error while clicking Manage button from the already installed BitSight solution under the Content Hub

Installed the BitSight solution from the Content Hub from the azure portal. After successfully installed when I go to manage it by clicking on the Manage button from the installed solution page from the Content Hub getting the "not found" error as below:

To Reproduce
Steps to reproduce the behavior:

Login to Azure Portal
Go to Microsoft Sentinel Workspace
Go to the Content Hub from the side menu and search for the BitSight solution
Install that solution by providing the required information of Subscription, Resource Group etc...
Again go to the BitSight solution from the content hub and click on the Manage button

Expected behavior
When clicking on the Manage button from the BitSight solution under the content hub it should display BitSight Solution Manage screen which all the available component of that solution.

Desktop (please complete the following information):

OS: Windows
Browser: chrome
Version: 122.0.6261.129 (Official Build) (64-bit)

Hi @jayeshprajapaticrest,
Thanks for flagging this issue, we will investigate this issue and get back to you with some updates by 03 Apr 2024. Thanks!

@v-rusraut
Understood. Thanks

Hi @jayeshprajapaticrest,
Please check with below link , if you still facing issue,please let us know.
https://ms.portal.azure.com/
Thanks

@v-rusraut
Thanks
Still getting the same error.

@v-rusraut
I checked today and the issue get resolved.
Thanks

Just to know Can you please let me know the cause of this issue?

authors: jayeshprajapaticrest v-rusraut

repo: Azure/Azure-Sentinel


text:
Create Theom Solution

Required items, please complete
Change(s):

Adding Solution for Theom
Added 1 Workbook, 20 Analytics Rules

Reason for Change(s):

New

Version Updated:

Yes

Testing Completed:

Yes

Checked that the validations are passing and have addressed any issues that are present:

Yes

@theomsoc :
Logo is looking good

Workbook is looking good

Could you please create a solution package by using V2 tool
Please follow the below URL and do the package creation. Thanks!
Azure-Sentinel/README.md at master · Azure/Azure-Sentinel (github.com)

@v-spadarthi should I use my branch as input for Logo and Basepath in input file at https://github.com/Azure/Azure-Sentinel/blob/master/Tools/Create-Azure-Sentinel-Solution/V2/README.md#create-input-file

@v-spadarthi added solution package, please review

@v-spadarthi thanks for the review. Addressed comments and rebuilt package. Shortcut names and atypical are as expected, to match the wording on our application.

@theomsoc : Data connector is looking fine

CreateUI definition

Main Template

Arm-ttk also running fine

@aprakash13 : Please review the analytics and provide your signoff. Thanks!
Once Analytics rules reviewed by @aprakash13 we are going to approve this PR.

@aprakash13 addressed comments and re-published new package

@aprakash13 @v-spadarthi could you please review and merge PR soon, as we are waiting on publishing solution.

Thanks @aprakash13 !! @v-spadarthi please do the needful.

@theomsoc : Please add the solution input file for that create Data folder under Solutions/Theom/ and keep it here.

@theomsoc : Thanks for adding solution input file.
Plz remove the empty properties from solution input file like parsers and playbooks

@theomsoc : Please add the solution input file for that create Data folder under Solutions/Theom/ and keep it here.

@v-spadarthi Added input file, resolved conflicts and removed empty properties.
If all looks good, please merge PR, thanks!

@v-spadarthi could you please check and merge PR soon, so we are able to publish today ?

Create UI definition is looking good

Arm ttk also running fine

All are looking good

Approved and merged since Suresh's has approved the PR

authors: theomsoc v-dvedak v-spadarthi


text:
Add-GalleryItems from local folder; Get-GalleryItem; Remove-GalleryIt…

Add-GalleryItems from local folder; Get-GalleryItem; Remove-GalleryItem in 1 script

@larisaborodina,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

authors: larisaborodina msftclas

repo: Azure/AzureStack-Tools


text:
Update script ERROR

Describe the bug
Run the update script to update existing SaaS-Accelerator in Azure, but get error
To Reproduce
Steps to reproduce the behavior:

Fork the SaaS-Accelerator code with latest release 7.6.3

Logon to Azure Portal > Open Cloud Shell and run below update-script
wget https://dotnet.microsoft.com/download/dotnet/scripts/v1/dotnet-install.sh;                      chmod +x dotnet-install.sh;
./dotnet-install.sh -version 6.0.423;                      $ENV:PATH="$HOME/.dotnet:$ENV:PATH";
dotnet tool install --global dotnet-ef --version 6.0.1;                      git clone https://github.com/xxxx/Commercial-Marketplace-SaaS-Accelerator.git;                      cd ./Commercial-Marketplace-SaaS-Accelerator/deployment;
.\Upgrade.ps1                       -WebAppNamePrefix "xxxxx"
-ResourceGroupForDeployment "xxxx"

See the errors:

Line |
32 |  Param(
|  ~~~~~
| The term 'Param' is not recognized as a name of a cmdlet, function, script file, or executable program. Check the spelling of the name, or if a path was
| included, verify that the path is correct and try again.
Deploying new database
ERROR: argument --vault-name: expected one argument
String-Between: /home/martel/Commercial-Marketplace-SaaS-Accelerator/deployment/Upgrade.ps1:64
Line |
64 |  $Server = String-Between -source $ConnectionString -start "Data Sourc …
|                                   ~~~~~~~~~~~~~~~~~
| Cannot bind argument to parameter 'Source' because it is an empty string.
Expected behavior
Expect to see successful message
Environment (please complete the following information):

Are you using the CloudShell? YES
If not using the CloudShell:

OS: [e.g. iOS] Window
PowerShell version
Modules installed: Pre installed SaaS-Accelerator 6.5

PowerShell Version Used: 7.4.3

The latest PR should have fixed this. @code4clouds can you see if it’s still an issue please

I got the fix and rerun the Upgrade.ps1 script, but get error with MSI:
Deploying new database
ERROR: Failed to connect to MSI. Please make sure MSI is configured correctly.
Get Token request returned: <Response [400]>
Upgrade.ps1:
Line |
8 |  .\Upgrade.ps1 `
|  ~~~~~~~~~~~~~~~
| Cannot bind argument to parameter 'Source' because it is an empty string.

@marteltran if you have already installed latest any reason why you are upgrading?
But if you are anyway, please make sure to remove vnet protection for sql, keyvault and app services during the upgrade and add it back again after the upgrade

I have installed the old version (6.5.x) so I want to upgrade to the latest (7.4.3).
Just confirm:

Do I have to DELETE the key-vault and 2 App services (Admin & Portal) before running the upgrade script? (I do not use VNET).
How to add the key-vault after all?

@marteltran - lets take a step back, but the latest is 7.6.2 not 7.4.3. I dont even see 7.4.3 anywhere am I missing something?
Also there is no comment about deleting anything, so please dont delete anything unless you are aware of what you are doing.

I confuse. When I fork the main branch, What is current version? and How to make sure that my fork contain 7.6.2?

authors: marteltran santhoshb-msft


text:
'BadEncodingLimitsExceeded ' received when creating subscriptions

We have OPC Publisher 2.3.3 running on iotedge 1.0.6.1 (3fa6cbef8b7fc3c55a49a622735eb1021b8a5963)
On startup up of the Publisher we are getting a  'BadEncodingLimitsExceeded ' error.
How should we go about debugging/resolving this issue?
[04:34:04 INF] Publisher is running. Press CTRL-C to quit.
[04:34:04 INF] Connect and monitor session and nodes on endpoint 'opc.tcp://10.70.64.95:49320'.
[04:34:06 INF] Create unsecured session for endpoint URI 'opc.tcp://10.70.64.95:49320' with timeout of 10000 ms.
[04:34:06 INF] Session successfully created with Id ns=1;g=73c5b05f-456f-49ec-a62b-a555657aaa6d.
[04:34:06 INF] The session to endpoint 'opc.tcp://10.70.64.95:49320/' has 3 entries in its namespace array:
[04:34:06 INF] Namespace index 0: http://opcfoundation.org/UA/
[04:34:06 INF] Namespace index 1: urn:NZS070HSTOPC01X.prod01.inroot.net:PTC.ThingWorxIC.1:UA%20Server
[04:34:06 INF] Namespace index 2: ThingWorx Industrial Connectivity
[04:34:06 INF] Created subscription with id 8 on endpoint 'opc.tcp://10.70.64.95:49320'
[04:34:06 INF] Create subscription on endpoint 'opc.tcp://10.70.64.95:49320' requested OPC publishing interval is 30000 ms. (revised: 30000 ms)
[04:34:06 INF] Start monitoring items on endpoint 'opc.tcp://10.70.64.95:49320'. Currently monitoring 0 items.
[04:34:06 INF] Now monitoring 10000 items in subscription with id '8'
[04:34:07 INF] Now monitoring 20000 items in subscription with id '8'
[04:34:08 INF] Now monitoring 30000 items in subscription with id '8'
[04:34:10 INF] Now monitoring 40000 items in subscription with id '8'
[04:34:14 INF] Now monitoring 50000 items in subscription with id '8'
[04:34:14 ERR] Unhandled OPC UA ServiceResultException 'BadEncodingLimitsExceeded 'Could not send request to server.'' when monitoring node 'ns=2;s=NZ070_Darfield_NZS070SRV01X.C270.NZ070_B05VA0506_AlmFault' on endpoint 'opc.tcp://10.70.64.95:49320'. Continue.
[04:34:14 INF] Now monitoring 60000 items in subscription with id '8'
[04:34:14 ERR] Unhandled OPC UA ServiceResultException 'BadEncodingLimitsExceeded 'Could not send request to server.'' when monitoring node 'ns=2;s=NZ070_Darfield_NZS070SRV01X.C801.NZ070_S1PW1P01_Msg1' on endpoint 'opc.tcp://10.70.64.95:49320'. Continue.
[04:34:15 ERR] Exception
Opc.Ua.ServiceResultException: Could not send request to server.
   at Opc.Ua.Bindings.ChannelAsyncOperation`1.End(Int32 timeout, Boolean throwOnError)
   at Opc.Ua.Bindings.UaSCUaBinaryClientChannel.EndSendRequest(IAsyncResult result)
   at Opc.Ua.Bindings.UaSCUaBinaryTransportChannel.EndSendRequest(IAsyncResult result)
   at Opc.Ua.Bindings.UaSCUaBinaryTransportChannel.SendRequest(IServiceRequest request)
   at Opc.Ua.SessionClient.CreateMonitoredItems(RequestHeader requestHeader, UInt32 subscriptionId, TimestampsToReturn timestampsToReturn, MonitoredItemCreateRequestCollection itemsToCreate, MonitoredItemCreateResultCollection& results, DiagnosticInfoCollection& diagnosticInfos)
   at Opc.Ua.Client.Subscription.CreateItems()
   at Opc.Ua.Client.Subscription.ApplyChanges()
   at OpcPublisher.OpcSession.MonitorNodesAsync(CancellationToken ct) in /app/opcpublisher/OpcSession.cs:line 561
[04:34:25 INF] Start monitoring items on endpoint 'opc.tcp://10.70.64.95:49320'. Currently monitoring 66541 items.
[04:34:25 ERR] Unhandled OPC UA ServiceResultException 'BadEncodingLimitsExceeded 'Could not send request to server.'' when monitoring node 'ns=2;s=NZ070_Darfield_NZS070SRV01X.C270.NZ070_B05VA0506_AlmFault' on endpoint 'opc.tcp://10.70.64.95:49320'. Continue.
[04:34:25 ERR] Exception
Opc.Ua.ServiceResultException: Could not send request to server.
   at Opc.Ua.Bindings.ChannelAsyncOperation`1.End(Int32 timeout, Boolean throwOnError)
   at Opc.Ua.Bindings.UaSCUaBinaryClientChannel.EndSendRequest(IAsyncResult result)
   at Opc.Ua.Bindings.UaSCUaBinaryTransportChannel.EndSendRequest(IAsyncResult result)
   at Opc.Ua.Bindings.UaSCUaBinaryTransportChannel.SendRequest(IServiceRequest request)
   at Opc.Ua.SessionClient.CreateMonitoredItems(RequestHeader requestHeader, UInt32 subscriptionId, TimestampsToReturn timestampsToReturn, MonitoredItemCreateRequestCollection itemsToCreate, MonitoredItemCreateResultCollection& results, DiagnosticInfoCollection& diagnosticInfos)
   at Opc.Ua.Client.Subscription.CreateItems()
   at Opc.Ua.Client.Subscription.ApplyChanges()
   at OpcPublisher.OpcSession.MonitorNodesAsync(CancellationToken ct) in /app/opcpublisher/OpcSession.cs:line 561

I have tried removing the tags that are reporting errors but it appears to just report the next tag as error'd after this.
This error is also mentioned on 2 other OPC github projects.
https://github.com/OPCFoundation/UA-.NET-Legacy/issues/107
https://github.com/open62541/open62541/issues/434
Neither is very clear to me why this might be happening, but sounds like it could be something going over the size limit for an OPCUA message?

@marcschier Yes, this use case is working now that we have the correct path.
We are seeing some performance challenges, I think it would be good to consider batching the OPC UA subscriptions if the resulting OPC UA group would result in a really large subscription.  Would be interesting to see if this would improve performance or reliability.

repo: Azure/Industrial-IoT


text:
WYSIWYG Formatting Issues

Describe the Bug with repro steps

Update any WYSIWYG text fields in new Logic App Editor

Save

On reload, line breaks of edited fields tend to have been removed

In my organization, this has been observed especially with Outlook Email "Message" field and Azure DevOps > Create a Work Item "Description" field

What type of Logic App Is this happening in?
Consumption (Portal)
Which operating system are you using?
Windows, Linux
Are you using new designer or old designer
New Designer
Did you refer to the TSG before filing this issue? https://aka.ms/lauxtsg
Yes
Workflow JSON
No response
Screenshots or Videos
Formatting before saving changes

Formatting after saving changes

Browser
Microsoft Edge, Chrome, Firefox
Additional context

This has occurred in 4 of our logic apps that I'm aware of
This has occurred under at least two different users on two different computers
This seems to generally occur on larger logic apps
Only occurs on New Designer, remediated by using Old Designer, which is slated for retirement

Also confirmed issue on Firefox from Fedora Linux

This has been fixed but not deployed, will start rolling out deployment for this on Monday https://github.com/Azure/LogicAppsUX/pull/5392

Right on, thanks a ton for the heads up, I'll look forward to it!

Actually Tuesday! Monday holiday in US

In case anyone finds this, deployment to prod took effect today 9/10/2024
Thanks a ton for all of your help!

authors: DanielleCogs DrLeeroyPhD

repo: Azure/LogicAppsUX


text:
Update Kubernetes Spark Install Demo

Update the Kubernetes + Spark install demo to work with the SimDem update. Adds cloning the Kubernetes GitHub repository to the home directory as part of the preparation as it takes a while.

@suhaildawood,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

authors: msftclas suhaildawood

repo: Azure/acs-demos


text:
Outbound internet does not work with CNI plugin

To repro:

deploy a cluster with CNI
kubectl exec to the container, and try to access the internet from a container

@ofiliz can you take a look?

I propose that we revise the issue title: "Kubernetes pods do not have outbound internet access with CNI plugin in l2tunnel mode".
Azure-vnet CNI/CNM plugins in regular IaaS VMs can be given outgoing access to internet with a simple NAT rule:
iptables -t nat -A POSTROUTING -m iprange ! --dst-range 168.63.129.16 -m addrtype ! --dst-type local ! -d {{{vnetCidr}}} -j MASQUERADE
The problem is the traffic policy enforced by CNI plugin and iptables rules set by kube-proxy are incompatible. We'll have to add this capability directly to Azure SDN stack, instead of SNAT'ing in the VM.

Discussed further with @seanknox and it is better to remove the examples until this can be fixed, otherwise customers will keep hitting this error.

Verified this is no longer an issue. On a cluster with azure CNI enabled:
$ ps auxfww | grep '/etc/cni/'
azureus+  19583  0.0  0.0  12948   936 pts/0    S+   00:06   0:00              \_ grep --color=auto /etc/cni/
root       6778  0.0  0.2 161524 17400 ?        Ssl  Nov21   0:00 /usr/bin/docker run --net=host --pid=host --privileged --rm --volume=/dev:/dev --volume=/sys:/sys:ro --volume=/var/run:/var/run:rw --volume=/var/lib/docker/:/var/lib/docker:rw --volume=/var/lib/kubelet/:/var/lib/kubelet:shared --volume=/var/log:/var/log:rw --volume=/etc/kubernetes/:/etc/kubernetes:ro --volume=/srv/kubernetes/:/srv/kubernetes:ro --volume=/etc/cni/:/etc/cni:ro --volume=/opt/cni/:/opt/cni:ro --volume=/var/lib/waagent/ManagedIdentity-Settings:/var/lib/waagent/ManagedIdentity-Settings:ro gcrio.azureedge.net/google_containers/hyperkube-amd64:v1.7.9 /hyperkube kubelet --kubeconfig=/var/lib/kubelet/kubeconfig --require-kubeconfig --pod-infra-container-image=gcrio.azureedge.net/google_containers/pause-amd64:3.0 --address=0.0.0.0 --allow-privileged=true   --enable-server --pod-manifest-path=/etc/kubernetes/manifests --cluster-dns=10.0.0.10 --cluster-domain=cluster.local --node-labels=kubernetes.io/role=master,kubernetes.azure.com/cluster=francis-1dot6 --cloud-provider=azure --cloud-config=/etc/kubernetes/azure.json --azure-container-registry-config=/etc/kubernetes/azure.json --network-plugin=cni --max-pods=30 --node-status-update-frequency=10s --image-gc-high-threshold=85 --image-gc-low-threshold=80 --v=2  --non-masquerade-cidr=10.0.0.0/8 --register-node=true --register-with-taints=node-role.kubernetes.io/master=true:NoSchedule

Was able to create a container, apt-get install curl, and curl to the internet:
$ kubectl exec -it shell-demo -- /bin/bash
root@shell-demo:/# curl google.com
<meta http-equiv="content-type" content="text/html;charset=utf-8">
<TITLE>301 Moved</TITLE>

301 Moved

The document has moved
here.

Awesome! Do you know what changed? Newer CNI image?

@seanknox I suspect it was the addition of a masquerade rule in iptables. See here for an example:
https://github.com/Azure/acs-engine/blob/master/parts/kubernetesmastercustomdata.yml#L283

@jackfrancis  @seanknox  Yes, it was fixed with the IPTables rule. Internet access should be working.

authors: anhowe jackfrancis ofiliz seanknox sharmasushant

repo: Azure/acs-engine


text:
Drop Azure Arc ESU Dashboard

Add in a drop referencing an Azure Arc Windows ESU Dashboard that people can use to view their Arc and ESU status.

Thanks for the feedback @fcabrera23, I've made the necessary changes.  Let me know your thoughts.  Thanks!


text:
Redeploying Bastion via template fails (Subnet AzureBastionSubnet is in use)

I'm not sure if this belongs here, but I'll give it a try.
I've created a bicep template to deploy a vnet resource with a Bastion resource. The first time deploying this template works as expected and all resources are created.
But when redeploying the template, I get the error "Subnet AzureBastionSubnet is in use by /subscriptions/..../resourceGroups/rg-test/providers/Microsoft.Network/bastionHosts/avd-vnet-bastion/bastionHostIp
Configurations/IpConf and cannot be deleted. In order to delete the subnet, delete all the resources within the subnet."
What I'm expecting is that when redeploying the template, it would detected that the resources exist and no changes are made to the resources. This is how other templates work when deploying other resources. I don't understand why it's trying to delete the AzureBastionSubnet.
See attached file that is the bicep template.
bastion.txt

@nielsvdc This is related to this behavior stated here: https://stackoverflow.com/questions/55901747/azure-arm-code-inusesubnetcannotbedeleted-when-trying-to-update-vnet
When the vnet gets deployed and no subnets are specified in the properties section, the vnet tries to delete the subnets. However, services are connected and hence, the deployment fails. Just the first deployment works fine, as there is no subnet that is in use.

Sorry people, I was unavailable for a moment. I tested with redeployment with CLI and also via Azure GUI. These also both failed with error message "Subnet AzureBastionSubnet is in use by /subscriptions/.../resourceGroups/test/providers/Microsoft.Network/bastionHosts/avd-vnet-bastion-eus/bastionHostIpConfigurations/IpConf and cannot be deleted. In order to delete the subnet, delete all the resources within the subnet."

Hi @nielsvdc, if we try to run az deployment group what-if ... we get the below outcome which clearly shows that they are trying to delete IpConf. So this looks like an expected behaviour.
This is a possibility when different resources doesnt have context about other resources as explained in this article
If you still feel this should not be an expected behaviour then as we use azure-cli internally yo make these calls, they will be able to help you out. Can you please raise an issue here? https://github.com/Azure/azure-cli
Note: The result may contain false positive predictions (noise).
You can help us improve the accuracy of the result by opening an issue here: https://aka.ms/WhatIfIssues.

Resource and property changes are indicated with these symbols:
  - Delete
  ~ Modify
  = Nochange
  * Ignore

The deployment will update the following scope:

Scope: /subscriptions/c00d16c7-6c1f-4c03-9be1-6934a4c49682/resourceGroups/deepak-rg

  ~ Microsoft.Network/bastionHosts/avd-vnet-bastion [2021-02-01]
    - properties.dnsName:    "bst-fffc8494-9771-467e-979f-389868569805.bastion.azure.com"
    - properties.scaleUnits: 2
    ~ properties.ipConfigurations: [
      ~ 0:

        - id: "/subscriptions/c00d16c7-6c1f-4c03-9be1-6934a4c49682/resourceGroups/deepak-rg/providers/Microsoft.Network/bastionHosts/avd-vnet-bastion/bastionHostIpConfigurations/IpConf"

      ]
    ~ sku.name:              "Basic" => "Standard"

  ~ Microsoft.Network/publicIPAddresses/avd-vnet-bastion-ip [2021-02-01]
    - sku.tier: "Regional"

  = Microsoft.Network/virtualNetworks/myVnet [2021-02-01]
  = Microsoft.Network/virtualNetworks/myVnet/subnets/AzureBastionSubnet [2021-02-01]
  * Microsoft.Network/networkSecurityGroups/NRMS-mqpdnhjdzis7mmyVnet
  * Microsoft.Network/networkSecurityGroups/NRMS-wlg2nxyjuzp7ymyVnet-AzureBastionSubnet
  * Microsoft.Storage/storageAccounts/deepak21
  * Microsoft.Storage/storageAccounts/deepak2121
  * Microsoft.Storage/storageAccounts/devstore4z32rb65cghuc
  * Microsoft.Web/serverFarms/AppServicePlan-GEN-UNIQUE
  * Microsoft.Web/serverFarms/devplan
  * Microsoft.Web/sites/devapp4z32rb65cghuc

Resource changes: 2 to modify, 2 no change, 8 to ignore.

Hi @nielsvdc are you still facing this issue? If no, then should we close this?

authors: marvinbuss nielsvdc t-dedah

repo: Azure/arm-deploy


text:
Ensure base class is absrtact in polymorphism

Description
Regen PR in sdk repo: https://github.com/Azure/azure-sdk-for-net/pull/30235
Checklist
To ensure a quick review and merge, please ensure:

[ ] The PR has a understandable title and description explaining the why and what.
[ ] The PR is opened in draft if not ready for review yet.

If opened in draft, please allocate sufficient time (24 hours) after moving out of draft for review

[ ] The branch is recent enough to not have merge conflicts upon creation.

Ready to Land?

[ ] Build is completely green

Submissions with test failures require tracking issue and approval of a CODEOWNER

[ ] At least one +1 review by a CODEOWNER
[ ] All -1 reviews are confirmed resolved by the reviewer

Override/Marking reviews stale must be discussed with CODEOWNERS first

We might need to make exceptions for this  in GA'd types as its a breaking change.  Technically if someone is using that ctor today it wouldn't work so it might be an acceptable break I have asked @KrzysztofCwalina to weigh in before we move forward.


text:
Reduce Number of Methods Generates in Clients

Currently, depending on the configuration setup, in ClientMethodMapper we can end up generating overloads of methods that'll never be used. For example, if Context is enabled we'll end up with api(params) and api(paramsPlusContext) and if this is being used with a hand-written SDK the non-Context overload will almost never be called as the written layer will handle either propagating a Context or use an empty Context. This has a few downsides, it increases the size of JARs, results in longer compilation times, and skews test code coverage, given that we should look into the following ideas/improvements:

Only generate a Context overload if the generated client will be in implementation. The service client being in implementation is a good indicator on whether the generated code is being used in a hand-written SDK.
Only generate Response-based APIs if the generated client will be in implementation. The hand-written layer can handle cracking open the Response to return T.

If the client is generated only for implementation, one might actually use the sync-methods=NONE to only generate the Response methods (that likely be the minimum), and wrapper client can do whatever they want?

authors: alzimmermsft weidongxu-microsoft


text:
Parameter naming and ParameterSet issues in Update/Set cmdlets

In most Set and update cmdlets in generated compute code, There is an InputObjectParameter with Type IComputeIdentity and there is a separate Parameter Parameter which is the body parameter for the underlying object
Set cmdlets
(For example Set-AzVmss has a parameter called Parameter of type IVirtualMachineScaleSet`
This is in parameter sets containing both RGName + Name, and the InputObject parameter set.
    [Parameter(ParameterSetName='Update1', Mandatory, HelpMessage='The name of the resource group.')]
    [Parameter(ParameterSetName='UpdateExpanded1', Mandatory, HelpMessage='The name of the resource group.')]
    [Microsoft.Azure.PowerShell.Cmdlets.Compute.Category('Path')]
    [System.String]
    ${ResourceGroupName},

    [Parameter(ParameterSetName='Update1', Mandatory, HelpMessage='Subscription credentials which uniquely identify Microsoft Azure subscription. The subscription ID forms part of the URI for every service call.')]
    [Parameter(ParameterSetName='UpdateExpanded1', Mandatory, HelpMessage='Subscription credentials which uniquely identify Microsoft Azure subscription. The subscription ID forms part of the URI for every service call.')]
    [Microsoft.Azure.PowerShell.Cmdlets.Compute.Category('Path')]
    [System.String]
    ${SubscriptionId},

    [Parameter(ParameterSetName='Update1', Mandatory, HelpMessage='The name of the VM scale set to create or update.')]
    [Parameter(ParameterSetName='UpdateExpanded1', Mandatory, HelpMessage='The name of the VM scale set to create or update.')]
    [Microsoft.Azure.PowerShell.Cmdlets.Compute.Category('Path')]
    [System.String]
    ${VMScaleSetName},

    [Parameter(ParameterSetName='UpdateViaIdentity1', Mandatory, ValueFromPipeline, HelpMessage='Identity Parameter')]
    [Parameter(ParameterSetName='UpdateViaIdentityExpanded1', Mandatory, ValueFromPipeline, HelpMessage='Identity Parameter')]
    [Microsoft.Azure.PowerShell.Cmdlets.Compute.Category('Path')]
    [Microsoft.Azure.PowerShell.Cmdlets.Compute.Models.IComputeIdentity]
    ${InputObject},

    [Parameter(ParameterSetName='Update1', ValueFromPipeline, HelpMessage='Describes a Virtual Machine Scale Set.')]
    [Parameter(ParameterSetName='UpdateViaIdentity1', ValueFromPipeline, HelpMessage='Describes a Virtual Machine Scale Set.')]
    [Microsoft.Azure.PowerShell.Cmdlets.Compute.Category('Body')]
    [Microsoft.Azure.PowerShell.Cmdlets.Compute.Models.Api20190301.IVirtualMachineScaleSet]
    ${Parameter},

There are a few problems:

Both InputObject and Parameter both have 'ValueFromPipeline' set int he same parameter set, which is not possible to bind
Parameter is the name of the body parameter (instead of InputObject

I think the parameter set should be:

Path Parameters + Flattened body parameters
IdentityObject + Flattened body parameters

Where, in the second parameter set, if the object is a Vmss, we override its properties with the other parameters that are set.  Or, thinking about it another way, we use this object as the body parameter and substitute the provided parameters
Update cmdlets
For Update (PATCH)
The same pattern occurs.  I think the proper pattern is:

Path parameters + Optional flattened body parameters
ComputeIdentityObject (from pipeline) + optional flattened body parameters

In this case, the semantics should be to always use the input object only for its identity (and potentially, ETags)

This is occuriing in the latest compuet spec (examples from Set/Updat-Vmss): https://github.com/Azure/azure-powershell/blob/generation/src/Compute/readme.md

@fearthecowboy I think tha, for Update cmdlets we have a wowrking model, and perhaps for New as well.
For set cmdlets, the simplest solution may be to remove ViaIdentity parameter sets, and ensure that the type converter allows piping to the non-expanded parameter sets.

Short term:

remove non expanded variants

Longer:

have a varaiant that is like via identity but gets path from values in object

For now, we will just remove the non-expanded parameter sets for Set

Closing; no more immediate work

authors: fearthecowboy markcowl