Intermittent test failure in activity email

https://travis-ci.org/oaeproject/Hilary/builds/53051441
1219 passing (14m)
  1 failing
  1) Activity Email verify sending emails uses the configured tenant default timezone:
     Uncaught AssertionError<EMAIL_ADDRESS>==<EMAIL_ADDRESS>      at /home/travis/build/oaeproject/Hilary/node_modules/oae-activity/tests/test-email.js:437:40
      at /home/travis/build/oaeproject/Hilary/node_modules/oae-email/lib/test/util.js:105:32
      at allDone (/home/travis/build/oaeproject/Hilary/node_modules/oae-activity/lib/internal/email.js:9:9781)
      at /home/travis/build/oaeproject/Hilary/node_modules/oae-activity/lib/internal/buckets.js:9:3854
      at _collectBuckets (/home/travis/build/oaeproject/Hilary/node_modules/oae-activity/lib/internal/buckets.js:9:4310)
      at /home/travis/build/oaeproject/Hilary/node_modules/oae-activity/lib/internal/buckets.js:9:5051
      at /home/travis/build/oaeproject/Hilary/node_modules/oae-activity/lib/internal/buckets.js:9:8270
      at /home/travis/build/oaeproject/Hilary/node_modules/redback/lib/advanced_structures/Lock.js:80:20
      at b (domain.js:183:18)
      at try_callback (/home/travis/build/oaeproject/Hilary/node_modules/redis/index.js:592:9)

$ bin/upload_logs
Uploaded "tests.log.gz" to Amazon S3: "oae-testing/travisci/logs/hilary/build-2258-e4dd4d940f593167cb43d97a36d488c0b483290f-Wed Mar 04 2015 15:12:35 GMT+0000 (UTC).log.gz"

@mrvisser : Did the intermittent test failures PR provide a solution for this?

@nicolaasmatthijs I had a look at this and it's unlikely.

Okay, leaving this ticket open

Update: I've ran tests countless times over the last 5 years and I can confidently say this doesn't happen anymore.


chore: promote jenkinsx-http-node to version 0.0.1

chore: promote jenkinsx-http-node to version 0.0.1
this commit will trigger a pipeline to generate the actual kubernetes resources to perform the promotion which will create a second commit on this Pull Request before it can merge

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To complete the pull request process, please assign
You can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@Sathwik-Anumala: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

verify
70886d36fbca1d64425ae86e446a2adb922a6e4c
link
/test verify

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the jenkins-x/lighthouse repository. I understand the commands that are listed here.


freelan-np: Add version 2.2.0

Closes https://github.com/ScoopInstaller/Extras/issues/10319

[x] I have read the Contributing Guide.

/verify

Got stuck:

Got stuck:

Did you not get a prompt asking you to install the free lan driver?

Did you not get a prompt asking you to install the free lan driver?

No, just a blank window, sorry.

@rasa, have you tried doing it on a Windows vm (ex. Oracle Virtual Box) instead of using a Windows sandbox?

@rasa, have you tried doing it on a Windows VM (ex. an Oracle Virtual Box VM) instead of using a Windows sandbox?

No, but that's a good idea. I had a script to quickly spin up a VM via Vagrant, but it's wired to a specific project, and would need to be rewritten.

Let me know how it gos for you once you set it up.


SASS Globbing ignoring sprite-feature

Hey there,
I have a main _component.scss and try to import multiple files in subdirectories with sass-globbing-pr-20:
@import '../Component/**/*';
In one of the inner files, I use the compass sprite feature to create a sprite:
@import 'icons/*.png';
@include all-icons-sprites;
Therefor compass throws "Undefined mixin 'all-icons-sprites'."
This happens even if I change the import-path to a non-existing path, so it seems the image-import is skipped/ignored.
As soon as I move the icons-import to my _component.scss, the sprite becomes generated and my included files can access it.
Unfortunately this is no option for me, because I want to take different sprites for different included files. So I still want to import the sprite seperatly.
Did anyone experience the same issues or knows a solution?
Greetings
Alex

fixed by c7dc8f3d341b9d5b76712be8b09439155e7bf858

will be in 1.1.4

Actually, I wasn't able to repro this so there's no fix. But try 1.1.5 and see if it helps.


i get some errors, but it means i dont know

ℹ Computer information

PowerToys version:
PowerToy Utility:
Running PowerToys as Admin:
Windows build number: [run "winver"]

📝 Provide detailed reproduction steps (if any)

…
…
…

✔ Expected result
What is the expected result of the above steps?
❌ Actual result
What is the actual result of the above steps?
📷 Screenshots
Are there any useful screenshots? WinKey+Shift+S and then just paste them directly into the form

Please don't spam issues.


Change UV glass laser hatch limitation to LuV

Improves parity between PSS and LSC

I’m with Colen on this one and prefer an additional challenge to the PSS instead of a buff to the LSC.

Is it that much of a buff when lasers are already gated so heavily?

This allows the use of low power laser on the LSC before UV glass. They're not used much but there's that to consider.

So this is buffing low power lasers, therefore giving them more use? That isn't necessarily a bad thing.

Yea that's my point. I don't think many people will use them but I mean they'd actually be usable

In all fairness, you cannot use TT lasers without active transformers which are ZPM level, so there is already a noticeable gating here. However when you are ZPM level making the UV glass is not a big deal either.

I just thought it was kind of a strange restriction in general to have next to PSS which doesn't impose such

In my opinion with the new Borosilicate recipe making UV glass is pretty cheap now and wasn't much of a gate already

I'm not really sure why you want more people to use lasers when lasers are in desperate need of a rebalance as is.

Yeah lasers need a whole balance rework. We may want to do this during the rebalance process instead of randomly doing it here and there, so we have a better overview of what's going on.


[Q] How to generate a pure carrier with Dektec?

I need a pure carrier for test , that the output S2 signal is replaced by a simple sine wave at the RF frequency.
The level of the unmodulated carrier corresponds to the output power. This mode allows, among others, phase noise measurement without requirement of an external amplifier.
How can I do it with Tsduck?

Which type of Dektec modulator do you use? Did you check that this device can do that?
TSDuck just drives the Dektec devices. If some software API can do that, TSDuck can be enhanced to use it.
However, please note that the dektec output plugin sends the transport stream from tsp to a Dektec device. What you want is just an empty signal into which there is no transport stream. So, using tsp is a bit useless. If the Dektec modulator can produce that type of empty signal, did you check if some Dektec-provided software can do that already?

lelegard, thanks lot for your help.
I got the solution from Dektec just now.
To set RF Mode to CW-channel only, you can use the DTAPI_UPCONV_CW option for the DtOutpChannel::SetRfMode, as described on page 438.

The option --carrier-only has been added to the dektec output plugin. Please try it because I do not have any Dektec modulator here today to test. I am not 100% sure of the result because there is some inconsistency here: we set the modulator output to "carrier only" and we sent a TS at the same time. Please let us know what is the behaviour, either on the software side (any error when sending the TS?) and on the signal side (is this really an empty carrier?)

@lelegard You concern is right. The manual does not make it clear. It should set the option but not send out the TS to get a "carrier   only". Otherwise we can only get normal wave.

I modified the "DtPlay" to get the result.

Do you mean that you added DTAPI_UPCONV_CW to SetRfMode, sent the TS, analyzed the signal and saw the modulated TS on the analyzer? I mean not only the carrier but also the modulation over it.

Perhaps we should use tsdektec alternatively instead of tsp.

Yes, that makes sense in theory. In practice, I wonder what kind of options we need to add. Normally, the frequency should be enough. But will it work in terms of software sequence? Do we need to set at least the modulation type ? Even though there is no modulation over the carrier. There are tests to be done and potentially a lot of code to add. Working from home because of the pandemic, I have no access to any Dektec modulator or signal analyzer.
I have added option --carrier-only to the dektec output plugin. Could you please test it with a signal analyzer and check if the modulated TS is present or not? If the modulation is still present, a compromise could be to drop output packets when --carrier-only is set. Not fully satisfactory, intellectually speaking, but this is a very specific and rare use case.

When I start tsp with --carrier-only , the modulated TS is still present. It need to drop all output packets to get a pure carrier as I did with DtPlay.  It said tsp need at most 1 output plugins .Can you give me some idea about how to drop packtes with tsp command?

OK, just I added two things

In output plugin dektec, when --carrier-only is present, all packets are dropped instead of being sent. This seems consistent with "carrier only", meaning no TS.
In command tsp, I added the option --final-wait to add a wait time after getting the end of input. Zero means infinite. This may be useful is other cases also.

Thus, you can try the following command:
tsp --final-wait 0 -I null 1 -O dektec --carrier-only --frequency ...

In fact, any input TS is fine with -O dektec --carrier-only but using -I null alone would saturate the CPU, generating and dropping null packets. With tsp --final-wait 0 -I null 1, we generate one single null packet (required to start everyhting) and then wait forever, maintaining the output carrier. Could you please test with the latest repo?
I admit this is a bit awkward but this is a very specific use case.

The output wave is just as expected, "carrier only",  when I use  tsp --final-wait 0 ... with the latest repo.
By the way, when I terminated the command , the carrier is still there . Not like the normal tsp command's behavior.

By the way, when I terminated the command , the carrier is still there . Not like the normal tsp command's behavior.

I assume that this is because the output channel of the Dektec device is not properly closed after the interrupt. When tsp terminates or is "properly" interrupted, all plugins are explicitly closed. In the case of the dektec output plugin, the channel is explicitly detached. I assume that this mutes the carrier.
In the current case, you probably have to hit Ctrl-C twice. The first time is a user interrupt, what I call "properly interrupted". This interrupt is handled by tsp which notifies all plugins to terminate. However, the plugins take that interrupt request synchronously, when they process the next packet. This is immediate in the usual case since hundredths or thousands of packets are processed each second. However, in our case, all plugins remain asleep, no packet is ever processed, no plugin can synchronously process the interrupt request and tsp does not terminate. This is why you have to hit Ctrl-C a second time. In this case, this is a hard interrupt and command is killed without executing any code, meaning not explicitly detaching the Dektec channels.
To validate this hypothesis, you may try tsp --final-wait 1000 ..... In that case, tsp waits only 1 second instead of forever and all plugins are properly terminated.
Which model of modulator do you use?

I'm using Dta-2115B.
No matter pressing Ctrl-C twice or tsp --final-wait 1000 ...., the carrier remains.
When I use normal tsp  without --carrier-only , the modulated carrier will disapear immediatly after the command  been terminated

OK, so that means that the channel detach operations treats "carrier only" and modulated TS in different manners.
I have made a modification to explicitly mute the carrier on close on all modulators. This code was rather old. In the first Dektec modulators, RF muting was not supported. When the DTA-115 was introduced (~12 years ago), it became possible. Code from that time explicitly sets RF mute on close on DTA-115 only. I just change it to set it on all modulators, except the oldest ones, before DTA-115 (DTA-107 and DTA-110T). Please check if the carrier disappears after a normal close (tsp --final-wait 1000 -I null 1 -O dektec --carrier-only ...).
Note that this won't change in the case of hard interrupt (Ctrl-C twice) since there is no proper close.

The result as below:
1、tsp --final-wait 1000 ... wait forever instead of 1 second. Neet  hard interrupt.
2、 --final-wait 1000 ... with      --carrier-only ， Ctrl-C once , unmodulated carrier disappers immediately
3、 --final-wait 1000 ...without --carrier-only ， Ctrl-C twice , modulated carrier disappers after about 30 seconds.

tsp --final-wait 1000 ... wait forever instead of 1 second.

This does not make sense. Did you actually specify -I null 1 (1 packet and ends) and not -I null (infinite packet generation)?

You are right. When I specfiy -I null 1, everything runs as you predicted.
Thank you so much.

Automatically closed after 256 days without update and "close pending" label set.


help with a new font

Hello, any chance you could help me figure out how to load a new font?  I'm on linux so not using glcd-font-creator, I found a different utility that make a .c font file and I was able to load it after some struggles (the font has a different bytes per letter than specified in the load code from the xglcd_font script.  Now I am getting an error in the get_letters function (in the xglcd_font scrip) when i try to draw letters on the screen.  the error is on line 144 of the xglcd script, and I do not have much experience dealing with raw memmory.  If you are willing to take a look I can share the font file etc.  Thank you in advance, Brian

I'd try manually decoding and drawing a letter from your .c file on some graph paper or a spreadsheet.  Then draw the same letter from one of my existing fonts (preferably the same size or close to the same size).  Pick a letter such as "F" that doesn't have line symmetry.
I don't think there is any standard for generating .c font files.  Your program may be incompatible.  You can post the line for the letter F and I will take a look.  I might be able to create a converter.

Thank you for getting back to me and taking a look at this.  I understand my font might be incomparable.  The font file can  be viewed at : https://github.com/fieldofhats/c_font_example/blob/main/28_pt_font.c.  Ill paste the line for F below.
fyi this is the app that I used: https://github.com/ayoy/fontedit/releases/v1.1.0
0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00, 0x00,0x00,0x00,0x00,0x00,0xF0,0xFF,0x0F,0xF0,0xFF,0x0F,0xF0,0xFF,0x0F,0xF0,0x00, 0x00,0xF0,0x00,0x00,0xF0,0x00,0x00,0xF0,0x00,0x00,0xF0,0x00,0x00,0xF0,0x00,0x00, 0xF0,0x00,0x00,0xF0,0x00,0x00,0xF0,0xFF,0x07,0xF0,0xFF,0x07,0xF0,0xFF,0x07,0xF0, 0x00,0x00,0xF0,0x00,0x00,0xF0,0x00,0x00,0xF0,0x00,0x00,0xF0,0x00,0x00,0xF0,0x00, 0x00,0xF0,0x00,0x00,0xF0,0x00,0x00,0xF0,0x00,0x00,0xF0,0x00,0x00,0xF0,0x00,0x00, 0xF0,0x00,0x00,0xF0,0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00, 0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00,0x00, 0x00, // Character 0x46 (70: 'F')

Please generate a font the same size as one of my fonts such as 13x21 pixels.  Also, a variable width font would be better than fixed width.

A quick review of the coding of your letter F show several differences from GLCD Font Creator format.

The bits of the provided hex values encodes pixels in row format as opposed to column format.
The byte order is Big-Endian as opposed to Little-Endian.
There is no leading hex value to indicate the width of the individual letter.

Create a function to transpose the rows to columns. This function will convert the row-based representation to a column-based one.
Since the original format is little-endian, reverse the bits in each byte after transposition.
The GLCD font code requires width information for each character, calculate the width based on the transposed data and prepend it.

great, Ill give it a try.  Thank you for the advice.

I went ahead and created a FontEdit conversion utility for you.
This is a very quick release that I didn't thoroughly test.
I converted your Deja Vu font and it worked.

Wow, Rad! thank you, ill try it out.


fix: Dont show home collections rail spinner

The type of this PR is: Bugfix

This PR resolves CX-1677
Description
Prevents the collections home rail from showing a spinner if no collections are available.

PR Checklist (tick all before merging)

[x] I have included screenshots or videos to illustrate my changes, or I have not changed anything that impacts the UI.
[x] I have tested my changes on iOS and Android.
[x] I have added tests for my changes, or my changes don't require testing, or I have included a link to a separate Jira ticket covering the tests.
[x] I have added a feature flag, or my changes don't require a feature flag. (How do I add one?)
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added an app state migration, or my changes do not require one. (What are migrations?)
[x] I have added a CHANGELOG.yml entry or my changes do not require one.

Changelog updates

Cross-platform user-facing changes

iOS user-facing changes

Android user-facing changes

Dev changes

#squashongreen


Uncaught errors in AbortableOperation

Have you read the FAQ and checked for duplicate open issues?
Yes
What version of Shaka Player are you using?
Latest from master, or, very close to it
Can you reproduce the issue with our latest release version?
Did not test
Can you reproduce the issue with the latest code from master?
Yes
Are you using the demo app or your own custom app?
Demo app
What browser and OS are you using?
Chrome 64 macOS
What are the manifest and license server URIs?
Any demo content should do
What did you do?
You can reproduce this a few ways. Try adding this code to shakaDemo.preparePlayer_:
player.configure({
  manifest: {
    retryParameters: {
      timeout: 1
    }
  }
});

or this:
player.getNetworkingEngine().registerRequestFilter(function () {
  throw Error('aw crap');
});

What did you expect to happen?
No uncaught errors
What actually happened?
Chrome logs Uncaught (in promise). This was identified by @theodab here: https://github.com/google/shaka-player/pull/1303#pullrequestreview-97997402, but this is different from #1302 (which was specific to uncaught errors in XHR callbacks), and will require a different solution.
As far as I can tell this affects any AbortableOperation. If the wrapped Promise rejects, the error will be uncaught, even if there are error handlers attached to the AbortableOperation. I think this is because of how AbortableOperation wraps the Promise and translates its success/failure to a separate Promise instance. But, I could be wrong; AbortableOperation is a maze of callbacks, so I will pass it off to you folks for further investigation.

Confirmed.  The cause is a little bit subtle, though.  The issue is not in AbortableOperation so much as in the way it's used in one particular part of NetworkingEngine.
We chain the operations together, so a failure in one ripples to the end, just like you would find in a Promise chain:
  let responseFilterOperation = requestOperation.chain(
      (response) => this.filterResponse_(type, response));

But it's this pattern which causes the problem:
  requestOperation.promise.then(() => {
    responseFilterStartTime = Date.now();
  });

Here we have created a fork in the Promise chain.  So the error is actually caught in the chain of operations, but not in this particular fork of the underlying Promise chain.
Here's some terrible ASCII art to illustrate:
(Operation 1: A -> B -> C -> D) -> (Operation 2: E -> F -> G -> H) -> finalPromise
                              \-> Fork1                          \-> Fork2

Within the operation chains, each Promise (A through H) is properly terminated, but not on the forks.

The only way to test that we cleaned up unhandled rejections in NetworkingEngine is to turn those into test failures.  Doing that globally turned up several issues all over the code base, some of which might be actual bugs.

The fix will appear in v2.4, because v2.3.x does not have AbortableOperation.


Initializer with returning value

I think it could be useful to allow the InitializerInterface to return a new instance of the object it is initializing.
For example we could have an initializer like
function ($container, $instance) {
    if (/*some condition on $instance*/) {
        return new OtherClass($instance);
    }
}

This could be useful, for example, for decorating a set of objects based on an interface they are implementing.
Moreover, this would not influence the current behaviour of the initializers.
Is there any reason not to do this I am not seeing?

@marcosh that's why we have delegators http://zendframework.github.io/zend-servicemanager/delegators/

@snapshotpl thanks for your answer. I know about delagators, but, as far as I know, you can use them to override single explicit classes. That is, you have to know which classes you are overriding at compile time. I would like to have that at run time...
To be more explicit, I would like to override every class implementing an interface I.
I would like to do have an initializer like
function ($container, $instance) {
    if ($instance instanceof I) {
        return $somethingElse;
    }
}

If this is actually possible using delegators, could you give me an hint on how?

If this is actually possible using delegators, could you give me an hint on how?

Not really supported

Thanks for your reply, @Ocramius. Since what I'd like to achieve is not possible using delegators, would it make sense to alter a bit how initializers work, just using their return value, to do it? It shouldn't break anything and it would allow use cases that are not currently possible (at least I think so...)

just using their return value, to do it?

That would change initializers massively, no?

It would allow to use initializars for other use cases and to do more things with them. The one already present would work exactly as they do now, so nothing would break. Maybe the name "initializers" itself would not be still very appropriate...
To be clear, what I had in mind was something like the following (it could probably be polished) in the doCreate method of the ServiceManager class:
foreach ($this->initializers as $initializer) {
    $newObject = $initializer($this->creationContext, $object);

    if (!is_null($newObject)) {
        $object = $newObject;
    }
}

I guess that the question now is: is this something that has a reason to be? Or is it more reasonable to leave this out and do not address the possible use cases that this would allow to solve?

In delegators, you get the requested instance by calling the given callable. Would not it fix your problem?

@danizord he'd still need it for a set of services, not just a single service

Ha yeah :sweat_smile:


Tool Not working in Arc 10.7 and 10.7.1 for TomTom processing

I found on 3 different PCs having all ArcMap 10.7 or 10.7.1 that the toolbox was not running correctly. Access to "Unblock" (Security) needed to uncheck in the installation was possible on 1 of 3 PCs.
When I ran the "Processes MultiNet Street Data" tool (with TomTom dataset), only the computer that installed correctly worked but stopped running (after 2:30 am) reporting this problem:
Exception from HRESULT: 0x800422B8
at ESRI.ArcGIS.Geodatabase.IDatasetContainer2.CreateDataset (IDEDataset DataElement)
Has GPProcessVendorDataFunctions.ProcessMultiNetDataFunction.CreateAndBuildNetworkDataset (String outputFileGdbPath Double fgdbVersion, fdsName String, String ndsName, createNetworkAttributesInMetric Boolean, Boolean createTwoDistanceAttributes, timeZoneIDBaseFieldName String, Boolean directedTimeZoneIDFields, commonTimeZone String, Boolean usesHistoricalTraffic, ITrafficFeedLocation trafficFeedLocation, usesRSTable Boolean, Boolean usesLTRTable, usesLRSTable Boolean, String lrsStatsTableName)
at GPProcessVendorDataFunctions.ProcessMultiNetDataFunction.Execute (IArray paramvalues, ITrackCancel trackcancel, IGPEnvironmentManager envMgr, IGPMessages messages)
Failed to execute (ProcessMultiNetData).
I succeeded in ArcMap 10.6.1. My problem is solved.
Hoping to help you improve this toolbox.

I apologize that we never responded to this problem, but I'm glad switching to ArcMap 10.6.1 resolved your problem back in 2019 when you logged the issue.
The ArcMap version of the tool was rebuilt at some point for the final release of ArcMap (10.8.*), and ArcMap is now in mature support.  A new version of the Street Data Processing Tools for ArcGIS Pro is now available, and this version of the toolbox does not require installation or have any dlls that have to be unblocked.
Closing this issue as no longer relevant.


Error Enabling Addon "metallb"

Summary

3 x Raspberry Pi Ubuntu 22.10 / MicroK8s via. snap

Enabling Addon metallb throws error:
deployment.apps/controller condition met
Error from server (InternalError): error when creating "STDIN": Internal error occurred: failed calling webhook "ipaddresspoolvalidationwebhook.metallb.io": failed to call webhook: Post "https://webhook-service.metallb-system.svc:443/validate-metallb-io-v1beta1-ipaddresspool?timeout=10s": context deadline exceeded
Error from server (InternalError): error when creating "STDIN": Internal error occurred: failed calling webhook "l2advertisementvalidationwebhook.metallb.io": failed to call webhook: Post "https://webhook-service.metallb-system.svc:443/validate-metallb-io-v1beta1-l2advertisement?timeout=10s": context deadline exceeded

Services with Type "LoadBalancer" are in state "pending":
k get svc:
default          test              LoadBalancer   <IP_ADDRESS>   <pending>     80:31110/TCP             44m
What Should Happen Instead?
No errors while activating metallb-Addon and Services with type LoadBalancer should get an IP
Reproduction Steps
Following the Guide: https://ubuntu.com/tutorials/how-to-kubernetes-cluster-on-raspberry-pi#1-overview

Install Ubuntu on PIs
Edited cmdline-file with
cgroup_enable=memory cgroup_memory=1  -> whole line looks like:
cgroup_enable=memory cgroup_memory=1 console=serial0,115200 dwc_otg.lpm_enable=0 console=tty1 root=LABEL=writable rootfstype=ext4 rootwait fixrtc quiet splash

3)reboot
4) install microk8s via snap
5) Build Cluster via microk8s
6) Enabling Microk8s Addons:

dns
rbac
metallb (range <IP_ADDRESS>-<IP_ADDRESS>)

Pods seem to run fine:
NAMESPACE        NAME                                       READY   STATUS    RESTARTS   AGE   IP              NODE   NOMINATED NODE   READINESS GATES
kube-system      calico-kube-controllers-5d7dbf4c7d-vtgzs   1/1     Running   0          65m   <IP_ADDRESS>    k8s1   <none>           <none>
kube-system      coredns-d489fb88-8wtb9                     1/1     Running   0          53m   <IP_ADDRESS>      k8s3   <none>           <none>
ingress          nginx-ingress-microk8s-controller-bf7r7    1/1     Running   0          48m   <IP_ADDRESS>      k8s3   <none>           <none>
ingress          nginx-ingress-microk8s-controller-cc8c5    1/1     Running   0          48m   <IP_ADDRESS>    k8s1   <none>           <none>
ingress          nginx-ingress-microk8s-controller-9sv6t    1/1     Running   0          48m   <IP_ADDRESS>     k8s2   <none>           <none>
default          test-75d6d47c7f-rrrgx                      1/1     Running   0          43m   <IP_ADDRESS>     k8s2   <none>           <none>
default          test-75d6d47c7f-8r9s2                      1/1     Running   0          43m   <IP_ADDRESS>      k8s3   <none>           <none>
default          test-75d6d47c7f-75ttz                      1/1     Running   0          43m   <IP_ADDRESS>    k8s1   <none>           <none>
metallb-system   controller-56c4696b5-gsxpc                 1/1     Running   0          16m   <IP_ADDRESS>     k8s2   <none>           <none>
metallb-system   speaker-bx9c8                              1/1     Running   0          16m   <IP_ADDRESS>   k8s2   <none>           <none>
metallb-system   speaker-h5h4x                              1/1     Running   0          16m   <IP_ADDRESS>   k8s1   <none>           <none>
metallb-system   speaker-xstvh                              1/1     Running   0          16m   <IP_ADDRESS>   k8s3   <none>           <none>
kube-system      calico-node-pm6cw                          1/1     Running   0          57m   <IP_ADDRESS>   k8s1   <none>           <none>
kube-system      calico-node-krhc2                          1/1     Running   0          56m   <IP_ADDRESS>   k8s2   <none>           <none>
kube-system      calico-node-bszf9                          1/1     Running   0          56m   <IP_ADDRESS>   k8s3   <none>           <none>

Introspection Report
After inspect there is an error:
The memory cgroup is not enabled, but it should be
-> (please look at Reproduction Steps)
Can you suggest a fix?
I tried the same setup, but with Ubuntu 20.04.5:  no Errors, Services with Type LoadBalancer are receiving an IP.
So the the error must have something to do with Ubuntu 22.10
Are you interested in contributing with a fix?
I would like to help, but dont know how

inspection-report-20221027_133229.tar.gz

I was recently working on a similar issue, and I determined the proxy wasn't set correctly in the /etc/environment or the /etc/profile.d/proxy.sh, do you get anything running https://webhook-service.metallb-system.svc -vvv or dig https://webhook-service.metallb-system.svc

I'm seeing the same issue on two 22.10 ubuntu systems, I opened a thread in discourse: https://discuss.kubernetes.io/t/error-enabling-metallb-internal-error-context-deadline-exceeded/22092/

I repeated my same steps on an Ubuntu 22.04 LTS install, and this time it all worked.

I'm having the same issues with 3 miniforum nucs and Ubuntu 22.10.

looking at the discussion panlinux posted it looks like you were having webhook error
Error from server (InternalError): error when creating "STDIN": Internal error occurred: failed calling webhook "ipaddresspoolvalidationwebhook.metallb.io": failed to call webhook: Post "https://webhook-service.metallb-system.svc:443/validate-metallb-io-v1beta1-ipaddresspool?timeout=10s": context deadline exceeded
Error from server (InternalError): error when creating "STDIN": Internal error occurred: failed calling webhook "l2advertisementvalidationwebhook.metallb.io": failed to call webhook: Post "https://webhook-service.metallb-system.svc:443/validate-metallb-io-v1beta1-l2advertisement?timeout=10s": context deadline exceeded

try running
kubectl -n metallb get validatingwebhookconfiguration -o yaml
and see if failurePolicy is set to Fail, I believe you can set it to ignore.
It looks like it might be getting hung on proxy? so that might be another option to look into

and see if failurePolicy is set to Fail, I believe you can set it to ignore.

It's Fail indeed, but before ignoring an error it's important to understand why it's happening, and only in ubuntu kinetic. It works in jammy.

It looks like it might be getting hung on proxy? so that might be another option to look into

No proxy. This can be easily replicated in a kinetic vm. I just did it now, with microk8s 1.25.4 and two kinetic vms.

Hi @panlinux, this could be related to a vxlan bug that breaks checksum calculation.
Could you try to see whether:
microk8s kubectl patch felixconfigurations default --patch '{"spec":{"featureDetectOverride":"ChecksumOffloadBroken=true"}}' --type=merge

helps with your issue?

It seems to help to put e.g. metallb-system.svc into the set ofno_proxy variables
$ cat /etc/environment
...
NO_PROXY=<IP_ADDRESS>,::1,localhost,<IP_ADDRESS>/24,<IP_ADDRESS>/16,metallb-system.svc
no_proxy=<IP_ADDRESS>,::1,localhost,<IP_ADDRESS>/24,<IP_ADDRESS>/16,metallb-system.svc
...

as suggested somewhere above and in this metallb repo issue. To me, having to do this seems a bit odd, though, as one should expect that these kind of internal requests should be correctly resolved by the Kubernetes DNS service? (I had activated the DNS addon before activating the metallb addon, i.e. microk8s enable dns.)

@neoaggelos thanks for the workaround. I just tried your fix and it works. My setup
4 x RPI 4B running on 22.10 server with DNS addon.
I patched felixconfigurations and enabled metallb and not seeing any errors.
I had existing BGP configurations and all are working as expected. Thank you 👍

spec:
KUBE_VER="v1.29"
METALLB_VER="v0.13.12"
CALICO_VER="v3.27.0"
Ubuntu 22.04.3 LTS
The same unreachable error
Cause: Networking misconfiguration
Check your firewall and connectivity before proceeding with any installations.
that Answer the why @panlinux


Use sentence case for back links

Lots of our back links were incorrectly capitalised.
This uses sentence case for a bunch of them.
Before:

After:

the indentation on the multiline = link_to is a bit off. But we don't have any linter nor decided on how we make our Erb template readable so I'm going to let that slide.

What should the indentation be?

@edwardhorsford  wonder if we want to fix this too

@nasirkhanbeis that was addressed already in #612


Dropdown usability improvement - Open, close and select all text by click

@oferh I needed to implement an improvement to suit my project, if you think it's a good change, you could merge it.
I implemented a property openOnClick as default value false. When this property is true, the dropdown open and close by click. The all text also is selected when you click on the field.
In this case, it is not necessary to implement the extra code below:

I hope to help to improve your fantastic component.
Thanks.

Hi @marciomsm
Thanks for the PR!
Can't openOnFocus be used to get the same effect as openOnClick?

Hi @oferh...so I needed to open the dropdown all times that I click. When we already are focused on the field and click on it, the dropdown doesn't open again. I also need to select the whole text when I click on it. If you have any chance to test these changes, let me know what do you think. I did these changes because in my project the client complained about the usability. Before do these changes I was using openOnFocus.
Thanks.

merged to dev


Issue with dynamic namespaces?

Successfully managed to use this for a simple socketio connection but I'm having trouble when using a new feature called dynamic namespaces. It seems that the user is not present in the socket variable anymore.
Example of dynamic namespace:
io.of(/^\/seid-\d+$/).on('connect', (socket) => {
});

For what it's worth if I assign it to socket.client.user it works.


installations

hi am trying to install and run the project locally using xamp but its not happening Do you please mind helping me ou
..What's happening

@Kimson11 I dont know what you do and what versions of apache, php, etc. you use.  I cant help when i dont know whats going on in your server :)


Client Allocation Request for: TopX Lab

Client Allocation Questions
Core Information

Name: TopX Lab
Website / Social Media:
DataCap Requested: 2PiB
Addresses to be Notarized: https://filfox.info/en/address/f020940
Notary Requested: philippbanhardt

Hey @2019jack - at this time the Filecoin Foundation is only considering data cap requests <5TB. If you wish to request a lower data cap, then please make a new request via the app.
Thanks

@philippbanhardt ok, now me renew request


No coords atropisomers - fix smiles output of atrop wedges after reordering

Reference Issue
Needed for several things, including ("the Canonicalization of Enhanced stereo for](https://github.com/rdkit/rdkit/pull/7041")
What does this implement/fix? Explain your changes.
Allows atropisomers to be represented with wedge bonds even when there are no coordinates.  The assumption is that the lowest numbered "other atom" are each end are considered to be on the same side of the atropisomer bond.
Any other comments?

Thanks @tadhurst-cdd !


Python 2.6 support

Error response from daemon: manifest for circleci/python:2.6 not found

Giving up.


Can't find azure spring cloud service when use sp auth

Plugin name and version
OS: Mac
Plugin: azure-spring-cloud-maven-plugin 1.4.0-SNAPSHOT
Plugin configuration in your pom.xml
<plugin>
 <groupId>com.microsoft.azure</groupId>
 <artifactId>azure-spring-cloud-maven-plugin</artifactId>
 <version>1.4.0-SNAPSHOT</version>
 <configuration>
 <auth>
 <type>SERVICE_PRINCIPAL</type>
 <client>xxxxxx-xxxx-xxxx-xxxxx-xxxxxxx</client>
 <tenant>xxxxxxxxxxxxxxxxxxxxxx</tenant>
 <key>xxxxxxxxxxxx</key>
 <environment>AZURE</environment>
 </auth>
 </configuration>
</plugin>

Actual behavior
I haven created azure spring cloud service under this sb, but the plugin can't find it

Test in-branch andy-auth-wip, this issue is fixed, so close this issue

Test in-branch andy-auth-wip, this issue is fixed, so close this issue


Field and Method and Method Parameter annotations are broken

I added a broken test case to Annotations.java; field annotations aren't working.
I added reflection logic to set the annotations variable in java.lang.reflect.Field objects, but that didn't seem to be enough?
This is what is preventing the Closure Compiler from running (#317), since it uses field annotations for command  line options.

Method annotations are broken too; see test.

Ditto w/ method parameters.

Fixed.


Support AIOps 3.2

Is your feature request related to a problem? Please describe.
AIOps 3.2 introduces significant changes to the way the product is installed, so we need to ensure this repository can still install that latest version.
Describe the solution you'd like
Read through the updated documentation (https://www.ibm.com/docs/en/cloud-paks/cloud-pak-watson-aiops/3.2.0) and reflect eventual changes in the repo, watching for backward compatibility with the previous version.
Describe alternatives you've considered
None, the requirement is pretty narrow.
Additional context
Add any other context or screenshots about the feature request here.

https://www.ibm.com/docs/en/cloud-paks/cloud-pak-watson-aiops/3.2.0?topic=planning-deployment-scenarios

Reopened because I noticed the Event Manager gateway configuration was completely missed in the scenario with both AI and Event Manager.


Override gorm tags

Hi there,
Thanks a lot for putting together a bulk insert for gorm.
I would like to highlight an issue we experienced, let's say we have a struct with a field defined such as:
type fakeTable struct {
	TossACoin string `gorm:"column:toYourWitcher"`
}

Because there is this line:
dbColumns = append(dbColumns, mainScope.Quote(gorm.ToColumnName(key)))

The insert statement will be
...  INSERT INTO table (`toss_a_coin`,  ...

which is the default naming strategy of gorm.
The expected statement will be:
...  INSERT INTO table (`toYourWitcher`,  ...

Please, is there a way of not overriding the column names provided by tags when present?

Thanks for this! This is most definitely a but, I'll try to look into this asap.

I think I might have misunderstood this question. I thought the problem here was that with the above struct tag you were getting INSERT INTO table ('toss_a_coin',  ...  but that's not the case for me.
If I apply this change:
diff --git a/bulk_insert_test.go b/bulk_insert_test.go
index a1a77fc..1878036 100644
--- a/bulk_insert_test.go
+++ b/bulk_insert_test.go
@@ -16,7 +16,7 @@ type fakeRelationTable struct{}
 type fakeTable struct {
        ID        int `gorm:"primary_key;auto_increment"`
        Name      string
-       Email     string<EMAIL_ADDRESS>+       Email     string              `gorm:"column:this_is_custom<EMAIL_ADDRESS>        Relation  *fakeRelationTable  `gorm:"foreignkey:RelationID"`
        Relations []fakeRelationTable `gorm:"foreignkey:UserRefer"`
        Message   sql.NullString

The test will fail because the column name is bad:
=== RUN   Test_extractMapValue
    Test_extractMapValue: bulk_insert_test.go:67:
                Error Trace:    bulk_insert_test.go:67
                Error:          Not equal:
                                expected: []string{"created_at", "email", "message", "name", "publish", "updated_at"}
                                actual  : []string{"created_at", "message", "name", "publish", "this_is_custom", "updated_at"}

                                Diff:
                                --- Expected
                                +++ Actual
                                @@ -2,3 +2,2 @@
                                  (string) (len=10) "created_at",
                                - (string) (len=5) "email",
                                  (string) (len=7) "message",
                                @@ -6,2 +5,3 @@
                                  (string) (len=7) "publish",
                                + (string) (len=14) "this_is_custom",
                                  (string) (len=10) "updated_at"
                Test:           Test_extractMapValue

And if I print the SQL it reads:
INSERT INTO `fake_tables` (`created_at`, `message`, `name`, `publish`, `this_is_custom`, `updated_at`) VALUES (?, ?, ?, ?, ?, ?)

So I don't think I'm sure exactly what you mean. Do you expect the column to be named email even though the tag says i_am_custom? Or do you see email even with the tag applied?
Please provide some more information.

Hello @bombsimon
Thanks a ton for having a look to this case.
To clarify:

if I have this tag defined in the struct Email string gorm:"column:thisIsCustom
I expect the query to be:

INSERT INTO `fake_tables` (`thisIsCustom`, ... )

The test you've modified is giving you the desired outcome because it's testing the extractMapValue method in isolation, I think.
However, I think the culprit is mainScope.Quote(gorm.ToColumnName(key)), if you provide a custom camel case name, it will default it in underscore case, which is the issue we experienced.
Let me know if you need more details, sorry that I wasn't clear, I will amend the issue description.

@hervit0 Ah, gotcha!
This is a part of gorm which applies with or without this wrapper. You should have a look at the Namer signature and apply that to TheNamingStrategy.
gorm.TheNamingStrategy.Column = func(original string) string {
    // Optinally add conditions here, `if original == "thisIsCustom"`
    return original
}

db, _ := gorm.Open("mysql", "")
gormbulk.insertObjSet(db, []interface{}{value})

So instead given a camel case name:
iff --git a/bulk_insert_test.go b/bulk_insert_test.go
index a1a77fc..bb95dee 100644
--- a/bulk_insert_test.go
+++ b/bulk_insert_test.go
@@ -16,7 +16,7 @@ type fakeRelationTable struct{}
 type fakeTable struct {
        ID        int `gorm:"primary_key;auto_increment"`
        Name      string
-       Email     string<EMAIL_ADDRESS>+       Email     string              `gorm:"column:camelCaseForAll<EMAIL_ADDRESS>        Relation  *fakeRelationTable  `gorm:"foreignkey:RelationID"`
        Relations []fakeRelationTable `gorm:"foreignkey:UserRefer"`
        Message   sql.NullString

We now get
--- FAIL: Test_extractMapValue (0.00s)
    bulk_insert_test.go:67:
                Error Trace:    bulk_insert_test.go:67
                Error:          Not equal:
                                expected: []string{"created_at", "email", "message", "name", "publish", "updated_at"}
                                actual  : []string{"camelCaseForAll", "created_at", "message", "name", "publish", "updated_at"}

                                Diff:
                                --- Expected
                                +++ Actual
                                @@ -1,4 +1,4 @@
                                 ([]string) (len=6) {
                                + (string) (len=15) "camelCaseForAll",
                                  (string) (len=10) "created_at",
                                - (string) (len=5) "email",
                                  (string) (len=7) "message",
                Test:           Test_extractMapValue

And the SQL
INSERT INTO `fake_tables` (`camelCaseForAll`, `created_at`, `message`, `name`, `publish`, `updated_at`) VALUES (?, ?, ?, ?, ?, ?)

So just set this up when importing gorm and creating the connection you'll use with gormbulk and this should hopefully resolve your issue.

@hervit0 I've created a PR that shows how to keep the custom column name and also ensures that no future changes will break this support/feature. It may also serve as an example for your use case even though it's more or less the same as my post above.

Maybe I was too quick with this one, I didn’t test if native gorm uses the tag as is.
Sorry for the confusion, I’ll try to see how native gorm is handling tags tomorrow, you can se the code above as a workaround in the meantime.

Yeah I was too quick, native gorm uses the column name as is so I'll have another look at this issue.
Code to prove this
package main

import (
	"testing"

	"github.com/DATA-DOG/go-sqlmock"
	"github.com/jinzhu/gorm"
	"github.com/stretchr/testify/require"
	gormbulk "github.com/t-tiger/gorm-bulk-insert"
)

type Table struct {
	RegularColumn string
	Custom        string `gorm:"column:ThisIsCamelCase"`
}

func Test_insertObject(t *testing.T) {
	// Setup mock database and a gorm instance with said connection.
	db, mock, err := sqlmock.New()
	require.NoError(t, err)

	defer db.Close()

	gdb, err := gorm.Open("mysql", db)
	require.NoError(t, err)

	gdb.Debug()

	// Save the original naming strategy for the columns while we test.
	defaultColumnStrategy := gorm.TheNamingStrategy.Column

	// Set a custom naming strategy which will preserv a given column name,
	// otherwise the default strategy will be used.
	gorm.TheNamingStrategy.Column = func(original string) string {
		if original == "ThisIsCamelCase" {
			return original
		}

		return defaultColumnStrategy(original)
	}

	// With gormbulk we expect the statement to be sorted in order and that the
	// given name we added in the tag will be preserved. This is due to our
	// naming strategy function for columns.
	mock.ExpectExec(
		"INSERT INTO `tables` \\(`ThisIsCamelCase`, `regular_column`\\)",
	).WithArgs(
		"first custom", "first regular",
		"second custom", "second regular",
	).WillReturnResult(
		sqlmock.NewResult(1, 1),
	)

	// Insert a multi statement to test generated SQL.
	err = gormbulk.BulkInsert(gdb, []interface{}{
		Table{
			RegularColumn: "first regular",
			Custom:        "first custom",
		},
		Table{
			RegularColumn: "second regular",
			Custom:        "second custom",
		},
	}, 10)

	require.NoError(t, err)

	// Set the default naming strategy for native go.
	gorm.TheNamingStrategy.Column = defaultColumnStrategy

	// Add new mock expectations that will be executed by native gorm.
	mock.ExpectBegin()
	mock.ExpectExec(
		"INSERT INTO `tables` \\(`regular_column`,`ThisIsCamelCase`\\)",
	).WithArgs(
		"third regular", "third custom",
	).WillReturnResult(
		sqlmock.NewResult(1, 1),
	)
	mock.ExpectCommit()

	// Insert a single statement to test generated SQL.
	err = gdb.Create(&Table{
		RegularColumn: "third regular",
		Custom:        "third custom",
	}).Error

	require.NoError(t, err)
}

Sorry for all the confusion, you pointed me in the exactly right direction in your first post so I'm not sure why I took this long way to find the same conclusions as you did. I guess I thought you would've created a PR if you knew what needed to be fixed. :)
Anyhow, the PR now contains the requested changes which will preserve column tags just like native gorm.

Now resolved and released in v2.0.0.


Question: How are frameshifts handled?

I'm getting interesting results w/r/t frameshifts in my data.  A large percentage of my self-ligandome identical epitopes are from frameshifts (3511/3648), which was surprising.
My guess from a bit of reverse engineering is that you take the cDNA sequence, add the in/del, then run it through translation software, looking for the largest ORF.  Is that right?  I think that makes good sense too, I just want to understand.
Thanks,
Kipp

Hey Kipp,
There was a pretty tragic bug in Varcode which mis-translated some frameshifts: https://github.com/hammerlab/varcode/issues/151
I would upgrade Varcode before trusting any translated frameshifts. I'm sorry for the hassle!
As for how the translation happens: you have it pretty much right except that there's no need to select the longest ORF, a unique ORF should still be discernable after applying an indel to the reference sequence.

Ha! I missed that update by ~ 2 hours.  I'll give it a rerun and see how the frameshifts look.  Thanks!


Partial text on the navigation bar

This is a Feature Request
What would you like to be added
Make the navigation bar horizontally re-sizeable.
Why is this needed
I can't see the navigation index entirely unless I zoom out on my page.

/area web-development

related issue #38663

Looks like a duplicate of issue https://github.com/kubernetes/website/issues/39621
Currently, there is PR https://github.com/kubernetes/website/pull/41695 opened which attempts to fix this issue.
/triage duplicate

#41695 is merged now.
/close
in favor of the PR: #41695


Fix adoptable_pet issue and add tests

🔗 Issue
Fixes #522
✍ Description
Fixes linked issue and adds integration tests.
Expectation is that the adoptable_pets views are consistent across users - except for applicants being able to see their own application status
I don't know how to DRY up these tests - should this be a spec?

WE probably don't need to displat the Become an Adopter button, top right, if the user is staff


Delete Cargo.lock

Cargo.lock seems to cause dependabot to open new PR for every minor release.
Removing Cargo.lock should fix it, since a library crate won't need it anyway.

I prefer to keep the Cargo.lock to support usecases like bisection that need a known good set of dependencies. I think to satisfy dependabot I'm gonna just keep telling it to ignore versions until it stops opening PRs... (afaict from the docs there is no way to configure it to only PR on major updates, that's only done via auto-detection).

I think to satisfy dependabot I'm gonna just keep telling it to ignore versions until it stops opening PRs... (afaict from the docs there is no way to configure it to only PR on major updates, that's only done via auto-detection).

That's unfortunate.
If I find solution to this problem, I will open a PR here.


Adding support for GCS-stored files for consumer config overrides

r: @johnjcasey
This is for users to pass truststore / keystore files for Kafka consumers that are stored in GCS, and we can then store them in the local worker for Kafka to create a consumer properly. It's an 'ease-of-use' feature for x-lang (and syndeo) users.
It can be extended to other filesystems (e.g. s3)
fyi @chamikaramj @bvolpato I'd also like your thoughts on this...

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

Run Java_Kafka_IO_Direct PreCommit

cc @pranavbhandari24 -- as I know we have to workaround this for Kafka.

thanks y'all - yes, I can add this for the producer factory


Polygon could not be deleted

During a review, I couldn't delete polygon using Backspace, Delete or any other keys. Is it possible to delete polygon during assignment review?

@VyacheslavLukin, no, it is not possible to delete polygon during assignment review, as Yandex.Toloka doesn't update saved results during the assignment review in general.
To avoid any confusion: during the assignment review reviewer (or Researcher) can ONLY either accept the markup provided by the Mechanical Turk, or decline it.
We are currently working on the release in which all of the editing functionality will be unavailable during the assignment review. We will include this information in the docs during the preparation for this release.

Thanks for the information. I suppose we can close this issue when the new version will be released.


Restart kube-vip pods after updating ingress-expose svc

IMPORTANT: Please do not create a Pull Request without creating an issue first.
Problem:

The VIP is temporarily not accessible during upgrading.
Solution:

The Harvester pod adds the annotation kube-vip.io/ignore-service-security: "true" to the service ingress-expose, but the kube-vip can't handle the modification. Just delete the kube-vip pods to let the kube-vip watches service and handle the update even.
Related Issue: https://github.com/harvester/harvester/issues/4221
Test plan:

Spin up a Harvester environment with version v1.2.0-rc5
Delete annotation kube-vip.io/ignore-service-security: "true" of the service kube-system/ingress-expose.
Delete kube-vip pod, and we will be unable to ping the VIP.
Replace the Harvester image in the Harvester mangedchart. We'll be able to ping the VIP after the Harvester updates the service and restart the kube-vip.

LGTM, thanks for the fix, also @yaocw2020 can u please help to file an new to track the upstream VIP enhancement of this, thanks.

https://github.com/harvester/harvester/issues/4407

@mergify backport v1.2


[Question]Manage node role for HA cluster

I have a 3 nodes for Harvester cluster, version is latest 1.0.1 . After install cluster with 3 nodes all of nodes was set master role, I had testing for fail tolerance, after reboot first one I was rejoined like worker node, If I power off another master node cluster isn't available.
NAME              STATUS   ROLES                                         AGE    VERSION
hd-harvester-01   Ready                                       142m   v1.21.11+rke2r1
hd-harvester-02   Ready    control-plane,etcd,master   14h      v1.21.11+rke2r1
hd-harvester-04   Ready    control-plane,etcd,master   14h      v1.21.11+rke2r1
Can I change role in node manually, or how can I manage fail tolerance for getting HA cluster ?
In installation process it will not be able to change join method for managing ROLE for node, I want cluster should be able when other 2 nodes will fail and one will still alive

@Abbos92 Any updates after this issue, how did you recover the cluster, and did you test more?
(1) After you reboot one node, then it has 2-master and 1-worker. Is it reproducable? A support-bundle is helpful.
https://docs.harvesterhci.io/v1.0/troubleshooting/harvester/#generate-a-support-bundle.
(2)After you power off another master, I guess you mean the cluster management IP (VIP) was not available. Did you try to SSH into the node IP ?  kubectl get pods -A  kubectl get service -A  ... will reveal more.
(3) For each node, how about the CPU/memory/storage resources ?
thanks.


wrong posts tags

Hi. I noticed that there is a bug in creating nodes for the Blog Tags Endpoint ( /v2/blog/tags). My test store has four tags, but only the last one is displayed in gatsby graphql. I think the error is in the handleCreateNodeFromData method in generating the id for the node. You use 'id: createNodeId(${nodeType}-${item.id})' but the returned tag object does not have an id field and so all nodes generate the same gatsby id and because of that they replace each other.
Possible Solution:
'id: createNodeId(${nodeType}-**${item.id || item.tag}**)' or better yet use uuid for unique id's (https://github.com/uuidjs/uuid)

Hey @RadikKurmanaev . Im looking to upgrade the package with newer features like data build caching among other common plugin features. I will take note of this and will roll out a fix soon. Happy New Year to you sir. Thanks for using our plugin. Cheers 🎉🍾

thank you, happy new year to you too:)

@RadikKurmanaev Hey there. I just released a new plugin version that will resolve your issue and add new plugin options as well. Head over here for more info: https://www.npmjs.com/package/@epicdesignlabs/gatsby-source-bigcommerce.
Cheers! 🥂


Removed reference to mermaid.min.css in usage docs #797

Last PR I submitted was for develop branch, this is for the master branch.

Sorry for the back and forth! Thank you for your contribution!


Spfx field customizer for managed metadat field

Category

[x] Bug

Describe the bug
Hi,
I have been trying to create a simple Spfx field customizer and tried to get related info with @pnp/sp-taxonomy. I constantly get the following issue and the field no longer get rendered. I cannot acces debug in this case. I have verified if it is the async calls but other async calls to Graph for example work. Do you have any idea of what can be wrong?
Steps to reproduce
Create a hello world field customizer  called MmdCustomizerFieldCustomizer and attach it to a taxonomy field
In the code:
`export default class MmdCustomizerFieldCustomizer
extends BaseFieldCustomizer {
@override
public async onInit(): Promise {
const store: any = taxonomy.termStores.getByName("");
var term: ITerm = store.getTermById("${event.fieldValue.TermId}");
this.properties.descr = await term.getDescription(1031);
//var currentUser = (await this.getCurrentUserByGraph())["userPrincipalName"];
//alert(currentUser);
}`
Error  when I try to debug or after deploy on AllItems page load with taxonomy field in it:
listview-host-assembly_default_13c644876b8cb32e22df7c593449c80e.js:1 Uncaught (in promise) Error
at r.t [as constructor] (listview-host-assembly_default_13c644876b8cb32e22df7c593449c80e.js:1)
at new r (listview-host-assembly_default_13c644876b8cb32e22df7c593449c80e.js:1)
at Function.e.buildErrorWithVerboseLog (listview-host-assembly_default_13c644876b8cb32e22df7c593449c80e.js:1)
at Function.e.buildLoadComponentError (listview-host-assembly_default_13c644876b8cb32e22df7c593449c80e.js:1)
at listview-host-assembly_default_13c644876b8cb32e22df7c593449c80e.js:1
Expected behavior
In the this.properties.descr have the chosen term description
Environment details (development & target environment)
The environment is a development tenant and it is on targeted release

OS:  Windows 10
Target Environment: SharePoint Online
Browser(s): Chrome v79 |
Tooling: VS Code
dependencies: * "@microsoft/decorators": "1.9.1",
"@microsoft/sp-core-library": "1.9.1",
"@microsoft/sp-listview-extensibility": "1.9.1",
"@pnp/common": "^2.0.5",
"@pnp/logging": "^2.0.5",
"@pnp/odata": "^2.0.5",
"@pnp/sp": "^2.0.5",
"@pnp/sp-clientsvc": "^1.3.11",
"@pnp/sp-taxonomy": "^1.3.11",
"@types/es6-promise": "0.0.33",
"@types/webpack-env": "1.13.1"*

Thanks for your contribution! Sharing is caring.
Thank you in advance for your help.

If you use the taxonomy libraries, you can't use other versions of the PnPjs libraries. You'll need to stay on the 1.x versions for everything.
This will work:
npm install<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>--save


 This broke the build of the roxctl container, because for some reason the roxctl source directory got embedded into the image:
$ docker image save quay.io/rhacs-eng/roxctl:3.71.x-468-g9b6570ed50 | tar -xOf - */layer.tar | tar -tv | head
drwxr-xr-x  0 root   root        0 Sep  1 10:30 etc/
drwxr-xr-x  0 root   root        0 Sep  1 10:30 etc/ssl/
drwxr-xr-x  0 root   root        0 Sep  1 10:30 etc/ssl/certs/
-r--r--r--  0 root   root   200578 Aug 29 10:56 etc/ssl/certs/ca-certificates.crt
drwxr-xr-x  0 root   root        0 Sep  1 10:30 roxctl/
drwxr-xr-x  0 root   root        0 Sep  1 10:27 roxctl/central/
drwxr-xr-x  0 root   root        0 Sep  1 10:27 roxctl/central/backup/
-rw-r--r--  0 root   root     5062 Sep  1 10:27 roxctl/central/backup/backup.go
-rw-r--r--  0 root   root     2982 Sep  1 10:27 roxctl/central/backup/backup_test.go
-rw-r--r--  0 root   root     1093 Sep  1 10:27 roxctl/central/central.go

I suspect the culprit to be this line: https://github.com/stackrox/stackrox/pull/2820/files#diff-76ed074a9305c04054cdebb9e9aad2d818052b07091de1f20cad0bbac34ffb52R579
but I don't see how the source directory would make it there


Directory Listing & Web Scrapping Links

Finally a directory fuzzer that handles directory listings AND web scrapping links.
Branching checklist

[Yes] There is an issue associated with your PR (bug, feature, etc.. if not, create one)
[Yes ] Your PR description references the associated issue (i.e. fixes #123456)
https://github.com/epi052/feroxbuster/issues/291
https://github.com/epi052/feroxbuster/issues/245
[Yes] Code is in its own branch
[Yes ] Branch name is related to the PR contents
[Yes] PR targets main

Static analysis checks

[Yes] All rust files are formatted using cargo fmt
[Yes] All clippy checks pass when running cargo clippy --all-targets --all-features -- -D warnings -A clippy::mutex-atomic
[Yes] All existing tests pass

Documentation

[ Yes] New code is documented using doc comments
[N/A] Documentation about your PR is included in the README, as needed

Additional Tests

[Added 2 tests] New code is unit tested
[Added 2 tests] New code is integration tested, as needed
[Yes] New tests pass

Notes
For best performance on directory listings, I recommend Redirections and Extract Links are on
2 existing extraction tests required changes to account for the single additional request to directories to check for directory listings

great job with this, thank you! merging :fire:

Would it be possible (and wise) not to add the directory to the queue if the listing is empty ?

Would it be possible (and wise) not to add the directory to the queue if the listing is empty ?

@SecT0uch Do you have a target that has an example of this that your willing to share?
If it is deemed repeatable, it would be an easy fix. In the meantime, just cancel the scan on that directory if you manually verified there is not any files within.
This is a "new" feature so there may potentially be bugs. This is something that isn't "broken" per-se. No other directory fuzzing tool will even care that its a directory listing, so at least there's that.
@epi052 I'm actually surprised this occurs?

That would be reproducible like that:
mkdir /tmp/empty
cd /tmp/empty
python -m "http.server"

You get an empty listing on http://localhost:8000


[KED-2681] Setup a GraphQL endpoint with a healthcheck query

Description

Development notes

QA notes

Checklist

[ ] Read the contributing guidelines
[ ] Opened this PR as a 'Draft Pull Request' if it is work-in-progress
[ ] Updated the documentation to reflect the code changes
[ ] Added new entries to the RELEASE.md file
[ ] Added tests to cover my changes

Legal notice

[ ] I acknowledge and agree that, by checking this box and clicking "Submit Pull Request":

I submit this contribution under the Apache 2.0 license and represent that I am entitled to do so on behalf of myself, my employer, or relevant third parties, as applicable.

I certify that (a) this contribution is my original creation and / or (b) to the extent it is not my original creation, I am authorised to submit this contribution on behalf of the original creator(s) or their licensees.

I certify that the use of this contribution as authorised by the Apache 2.0 license does not violate the intellectual property rights of anyone else.

Thanks for setting this up! 2 things:

I've checked this branch out and tested this - currently the graphql healthcheck endpoints runs at http://localhost:4142/graphql instead of port 4141 as stated on the ticket - double checking if this is the intended port for serving the graphql endpoint and the playground?
FYI for BE tickets that relates to exposing an endpoint ( or anything relating to the change in the data), FE should be  included in the QA for the response, along with the expected results of the query.

Other than that, I've tried running this an it works well on port 4142 👍

I've checked this branch out and tested this - currently the graphql healthcheck endpoints runs at http://localhost:4142/graphql instead of port 4141 as stated on the ticket - double checking if this is the intended port for serving the graphql endpoint and the playground?

4142 is the default development server port. In production, i.e. when kedro viz is run as a plugin, it will run in 4141. The FE won't have to do anything. In dev mode, the FE will proxy all HTTP request to 4142.

@studioswong

I've checked this branch out and tested this - currently the graphql healthcheck endpoints runs at http://localhost:4142/graphql instead of port 4141 as stated on the ticket - double checking if this is the intended port for serving the graphql endpoint and the playground?

4142 is the default development server port of the backend. In production, i.e. when kedro viz is run through the CLI as a Kedro command kedro viz, it will run in 4141. The FE won't have to do anything. In dev mode, the FE will proxy all HTTP request to 4142.
In other words, the Graphql endpoint will be served on the same port as the REST endpoint.

Thanks for clarifying, I've expected the graphql playground (i.e graphiql) to be hosted on the same port as the REST endpoint as well, was just double checking given the ticket description. All good on our end.


refactor: wrap executable tx in account/l1handler enum

This change is

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#1788  👈
#1787
main

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

Join @yair-starkware and the rest of your teammates on  Graphite

crates/blockifier/src/transaction/transaction_execution.rs line 50 at r2 (raw file):

            }
            starknet_api::executable_transaction::Transaction::L1Handler(tx) => {
                Ok(Transaction::L1Handler(tx.clone()))

Shouldn't the TryFrom take ownership of the struct?
Suggestion:
Ok(Transaction::L1Handler(tx))

crates/blockifier/src/transaction/transaction_execution.rs line 50 at r2 (raw file):
Previously, yair-starkware (Yair) wrote…
Done.

Remove the clone :)

Merge activity

Nov 7, 6:14 AM EST: A user merged this pull request with Graphite.


Adding a disclaimer for adapters that have transformBidParams

prebid.js bid adapters have the possibility to add a transformBidParams method that alters the bid params before they are sent to a prebid server instance. This is necessary if the parameters differ on client and server side.
One value of prebid server is, that the client side javascript is not required anymore. The transformBidParams method however makes this impossible. In order to highlight this, we would like to add a disclaimer for all bid adapters that have a transformBidParams.
Related https://github.com/prebid/Prebid.js/issues/6361

@patmmccann would this be a suitable solution?


Question about spawning vehicles without Traffic Manager

Hello,
Is it possible to spawn surrounding vehicles and set them to autopilot without traffic manager? I tried once, and the surrounding vehicles did not move.
Thanks

I tried to spawn one vehicle with the autopilot and the vehicle didn't move. I wonder why.
This is my code and I can't find the error:
import carla
import time

client = carla.Client('localhost', 2000)
client.set_timeout(10.0) # seconds
# client.reload_world()

world = client.get_world()
world = client.load_world('Town03')

# traffic manager
tm = client.get_trafficmanager(8000)
tm.set_random_device_seed(0)
tm.global_percentage_speed_difference(-5.0)
# tm.global_distance_to_leading_vehicle(1)

blueprint_library = world.get_blueprint_library()

spectator = world.get_spectator()

vehicle_spawn_points = world.get_map().get_spawn_points()

tesla_bp = blueprint_library.filter('model3')[0]

tesla = world.try_spawn_actor(tesla_bp, vehicle_spawn_points[2])

spectator.set_transform(vehicle_spawn_points[2])

tm_port = tm.get_port()
tesla.set_autopilot(True, tm_port)
tm.auto_lane_change(tesla, True)
tm.distance_to_leading_vehicle(tesla, 1.0)
tm.vehicle_percentage_speed_difference(tesla, 0)

for i in range(10):
    time.sleep(1)
    snapshot = world.get_snapshot()
    print(snapshot.timestamp)

tesla.destroy()

I tried to spawn one vehicle with the autopilot and the vehicle didn't move. I wonder why.
This is my code and I can't find the error:
import carla
import time

client = carla.Client('localhost', 2000)
client.set_timeout(10.0) # seconds
# client.reload_world()

world = client.get_world()
world = client.load_world('Town03')

# traffic manager
tm = client.get_trafficmanager(8000)
tm.set_random_device_seed(0)
tm.global_percentage_speed_difference(-5.0)
# tm.global_distance_to_leading_vehicle(1)

blueprint_library = world.get_blueprint_library()

spectator = world.get_spectator()

vehicle_spawn_points = world.get_map().get_spawn_points()

tesla_bp = blueprint_library.filter('model3')[0]

tesla = world.try_spawn_actor(tesla_bp, vehicle_spawn_points[2])

spectator.set_transform(vehicle_spawn_points[2])

tm_port = tm.get_port()
tesla.set_autopilot(True, tm_port)
tm.auto_lane_change(tesla, True)
tm.distance_to_leading_vehicle(tesla, 1.0)
tm.vehicle_percentage_speed_difference(tesla, 0)

for i in range(10):
    time.sleep(1)
    snapshot = world.get_snapshot()
    print(snapshot.timestamp)

tesla.destroy()

That is a reduced spawn_npc.py file that worked for me:
#!/usr/bin/env python

import carla
import random
import logging
import time

def main():
    vehicles_list = []

    client = carla.Client('localhost', 2000)
    client.set_timeout(10.0) # seconds

    synchronous_master = False

    try:
        world = client.get_world()

        traffic_manager = client.get_trafficmanager(8000)
        traffic_manager.set_global_distance_to_leading_vehicle(1)
        traffic_manager.set_random_device_seed(0)

        blueprints = world.get_blueprint_library().filter('model3')

        spawn_points = world.get_map().get_spawn_points()

        SpawnActor = carla.command.SpawnActor
        SetAutopilot = carla.command.SetAutopilot
        # SetVehicleLightState = carla.command.SetVehicleLightState
        FutureActor = carla.command.FutureActor

        # spawn vehicle

        blueprint = blueprints[0]
        transform = spawn_points[2]

        batch = []

        # TODO what is the driver_id used for?
        if blueprint.has_attribute('driver_id'):
            driver_id = random.choice(blueprint.get_attribute('driver_id').recommended_values)
            blueprint.set_attribute('driver_id', driver_id)

        blueprint.set_attribute('role_name', 'autopilot')
        batch.append(SpawnActor(blueprint, transform)
            .then(SetAutopilot(FutureActor, True, traffic_manager.get_port())))

        for response in client.apply_batch_sync(batch, synchronous_master):
            if response.error:
                logging.error(response.error)
            else:
                vehicles_list.append(response.actor_id)

        spectator = world.get_spectator()
        spectator.set_transform(transform)

        time.sleep(20)
    finally:
        print('\ndestroying %d vehicles' % len(vehicles_list))
        client.apply_batch([carla.command.DestroyActor(x) for x in vehicles_list])

        time.sleep(0.5)

if __name__ == '__main__':

    try:
        main()
    except KeyboardInterrupt:
        pass
    finally:
        print('\ndone.')

That is a reduced spawn_npc.py file that worked for me:
#!/usr/bin/env python

import carla
import random
import logging
import time

def main():
    vehicles_list = []

    client = carla.Client('localhost', 2000)
    client.set_timeout(10.0) # seconds

    synchronous_master = False

    try:
        world = client.get_world()

        traffic_manager = client.get_trafficmanager(8000)
        traffic_manager.set_global_distance_to_leading_vehicle(1)
        traffic_manager.set_random_device_seed(0)

        blueprints = world.get_blueprint_library().filter('model3')

        spawn_points = world.get_map().get_spawn_points()

        SpawnActor = carla.command.SpawnActor
        SetAutopilot = carla.command.SetAutopilot
        # SetVehicleLightState = carla.command.SetVehicleLightState
        FutureActor = carla.command.FutureActor

        # spawn vehicle

        blueprint = blueprints[0]
        transform = spawn_points[2]

        batch = []

        # TODO what is the driver_id used for?
        if blueprint.has_attribute('driver_id'):
            driver_id = random.choice(blueprint.get_attribute('driver_id').recommended_values)
            blueprint.set_attribute('driver_id', driver_id)

        blueprint.set_attribute('role_name', 'autopilot')
        batch.append(SpawnActor(blueprint, transform)
            .then(SetAutopilot(FutureActor, True, traffic_manager.get_port())))

        for response in client.apply_batch_sync(batch, synchronous_master):
            if response.error:
                logging.error(response.error)
            else:
                vehicles_list.append(response.actor_id)

        spectator = world.get_spectator()
        spectator.set_transform(transform)

        time.sleep(20)
    finally:
        print('\ndestroying %d vehicles' % len(vehicles_list))
        client.apply_batch([carla.command.DestroyActor(x) for x in vehicles_list])

        time.sleep(0.5)

if __name__ == '__main__':

    try:
        main()
    except KeyboardInterrupt:
        pass
    finally:
        print('\ndone.')

Hello whathello .
I have a question .
I saw that autopilot is registered in tm and moves.
Is dirve_id an id to register with tm?


Add County to Partners Portal

What is this feature or what feature is this part of?
Doorway's Listings Page will need to provide search by County as a P0. In order to do so, we need to add the ability for Partners to designate a County when they add a new listing, as they currently don't have that ability in the UI in Bloom. Thankfully, the county field already exists and has been used off/on, so we only need to add a mandatory field for Partners.
Questions, I have that still need to be flushed out:

Do we want to show the County on the Public site, either in the Listings or the Listings Details pages? Yes - added.
I'm not convinced I've figured out how to make it required. Need help and/or dive deeper to verify.
Do we want to add a pick list for County? Or are we ok with freeform? Yes, we created a drop down w/counties.

Describe the solution you'd like

Add UI ability for Partners to see & edit the County field and make it required for all new listings or edits.
Ensure county is set for any seeding and/or any other necessary places in backend/core or shared resources
require county for all building addresses
Add County to Public Site
ASSUMPTION - imported data from Bloom will already be designated a county, if it doesn't exist - if this is not true, then need to ensure that happens  (FYI...creating a new issue for this for Chris to tackle.)

Describe alternatives you've considered
n/a
Additional context

What is the acceptance criteria/definition of done?

County can be added to an existing listing on the Partners Portal and publish (must have county to publish listing)
New listing created and must have county to publish
all imported data from Bloom must have county to publish (may be out of scope for this Issue)

Related Tasks/Issues/PRs

[x] #60
[x] #223
[x] #102
[x] #130
[x] #157
[x] #158
[x] #164

QA Review Instructions
To be added shortly...

Addressed by #130, #157, #158, #164, and #223


How to use a self-hosted pubsub emulator

PLEASE READ: If you have a support contract with Google, please create an issue in the support console instead of filing on GitHub. This will ensure a timely response.

There aren't really a lot of details in this question. You want to run the emulator locally? In a docker container?


Diagnose startup errors on Azure App Service

We want to provide an fwlink that will provide information for people who are having problems with their application not starting on Azure App Service.
Most of the information we would want is already available in some form, but not necessarily App Service specific. For example we have a doc that tells you how to turn on ANCM logging, but nothing that tells you how to edit that file on App Service and what locations are good to log to in App Service.
I would like to put the fwlink in the next VS update, and ideally update the Kudu deployment scripts as soon as the doc is ready. For example, Kudu can detect if the app failed to start at the end of a deployment and display the link to this doc if it did.
@RickAndMSFT Are you the right person to look at this?

Is it related to or a dup of https://github.com/aspnet/Docs/issues/4685?

Yeah, you could see this as a dupe I suppose. This just raises its priority and answers the question about it being its own document.

I deleted that remark since I figured that u already knew about the others (e.g., https://github.com/aspnet/Docs/issues/4685) and were probably asking for something else. I still think a full disclosure topic would be a big hit, but it's not really your ask here :point_right: https://github.com/aspnet/Docs/issues/2800 ... it would be nice to have everything in one spot (eventually).

@glennc  I changed the issue title to "Diagnose ASP.NET Core startup errors on Azure App Service"
I can get this done in a couple weeks if you provide an outline. I'll fix all or most of #4685


Display glitch with multiple levels of nested folders

Not sure yet whether this is an upstream dgrid issue or an issue with this app, but I've noticed when testing 2 or more levels of nesting of feeds in folders, that indent levels get screwed up.

Upstream bug: https://github.com/SitePen/dgrid/issues/1275


ClerkAPIResponseError is missing from @clerk/types

Preliminary Checks

[X] I have reviewed the documentation: https://clerk.com/docs

[X] I have searched for existing issues: https://github.com/clerk/javascript/issues

[X] I have not already reached out to Clerk support via email or Discord (if you have, no need to open an issue here)

[X] This issue is not a question, general help request, or anything other than a bug report directly related to Clerk. Please ask questions in our Discord community: https://clerk.com/discord.

Reproduction

Publishable key

Description

Steps to reproduce:

try to import [ClerkAPIResponseError] to properly type clerk errors https://github.com/clerk/javascript/blob/b34edd254d407613595cd567147a0a5f60d1a712/packages/shared/src/error.ts#L97
see that it does not exist

Expected behavior:
this error should be exposed so that we can properly type errors coming from clerk.
Actual behavior:
we can't type properly
Environment
"@clerk/types": "^4.39.2",

Hi! Have you tried using the isClerkAPIResponseError helper as a type guard for your error? If you're using the React SDK, you can import it from @clerk/clerk-react/errors:
import {
  isClerkAPIResponseError,
  // isEmailLinkError,
  // isKnownError,
  // isMetamaskError,
  // EmailLinkErrorCode,
} from '@clerk/clerk-react/errors'

thanks @wobsoriano that works! Is this documented anywhere? If not, would be great to add it, as we couldn't find it by ourselves🤔

@fogx great to know it works for you! I'll let the team know about it so we can add docs for it 🫡


Provide feedback on frontend project codebase

Provide general feedback on the codebase of the frontend part of the SSC Assistant.
Any code under the app/frontend folder.
definition of done
Provide general feedback on the follow aspects:

[ ] Code quality
[ ] Implementation techniques
[ ] Potential improvements, quick wins, long term goals.

Thanks for the feedback. In no particular order some comments:

styled was actually used by @kyleaitken. He helped a ton with this project! <3. I didn't  know about it before but it does help with readability.
I like the feedback about the ChatCompletionApi, ChatStreamManager, etc, this will definately help make things more readable and managable.
AssistantBubble splitting into sub components for bits/archibus/profiles is also a good idea ty.


IDF v.5.0 - problem with ASYNCIO (IDFGH-9333)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

IDF version.
v5.0-541-g885e501d99
Operating System used.
Windows
How did you build your project?
Command line with idf.py
If you are using Windows, please specify command line type.
CMD
What is the expected behavior?
building
What is the actual behavior?
F:\GitHub\esp-rainmaker\examples\fan>idf.py build
Executing action: all (aliases: build)
Running ninja in directory f:\github\esp-rainmaker\examples\fan\build
Executing "ninja all"...
ERROR: C:\Users\jacek.espressif\python_env\idf5.0_py3.7_env\Scripts\python.exe doesn't support asyncio. The issue can be worked around by re-running idf.py with the "--no-hints" argument.
Steps to reproduce.
standard compiling of Rainmaker example.
Build or installation Logs.
No response
More Information.
It works in v.4.4.3
I found that in v5. breaking changes:
ASIO: Internal ASIO component and ASIO examples were migrated from IDF to https://github.com/espressif/esp-protocols/components/asio
But still don't know what should do with it? (BTW: link is wrong)
Always have to type "--no-hints" ?

I am having the same problems. I had to re-install ESP-IDF 2 times now, this will be the third time.
This issue appears every time I use Espressif -> IDF Tools Manager -> Install Tools ... or do anything with the tools.
Don't know how to fix this so will re-install AGAIN.

This has nothing to do with the linked ASIO breaking change.
Please use the Windows tool installer in order to set up a working environment: https://dl.espressif.com/dl/esp-idf/. It will deploy among others a working Python 3.8 environment. Please don't select from the installer the Python 3.7 already available on your system. It is clearly broken because Python 3.7 should support asyncio (it was introduced in 3.4).

Thnak You, resolved by installing with online installer instead of cloning.


feature: set link time optimization flag

mini-chromium part of crashpad builds by default with /GL and /LTCG flags under windows. But it is important that you can deactivate this flags. Because if this flags are activated, conan can not deal with them at the moment, because conan uses instead of the msvc compiler version the Visual Studio IDE version. If this flags are activated, crashpad needs to be build for every new msvc version, even if this is a minor version change. (This not happens under conan at the moment) Additionally this flags causes much more linking time, what during the developing phase is not needed.

@uilianries CI for macOS is green now.


ES6 Spread operator at beginning of array

This is firing an error that says "Spread must be the final element of an element list"
 var xy = [0, 0];
 var wh = [100, 100];
 var dims = [...xy, ..wh];

This seems incorrect behavior, as spreads can be anywhere in a list. (Tested FF, Safari 8)

This looks like the relevant bug over there: https://github.com/facebook/jstransform/issues/52

Can we reopen this issue?
The following is perfectly valid, but raises the "Spread must be the final element of an element list" warning.
let small = [2, 3, 4];
let bigger = [1, ...small, 5];      // Warning flag here
console.log(bigger);                // [1, 2, 3, 4, 5]

See the earlier comment I posted years ago. jshint has unit tests guarding that this does not cause an error, and the error you mention is not in src/messages.js.
Could be a very old version, maybe, or not jshint at all?

Doh, yup problem wasn't with jshint after all. Thanks!

I'm getting this with the jshint version currently used in Atom's jshint plugin.
Here's my code:
import {addLocaleData} from 'react-intl';
import en from 'react-intl/locale-data/en';
import de from 'react-intl/locale-data/de';

addLocaleData([
  ...en, // <-- here jshint complains "spread must be the final element of an element l..."
  ...de
]);

This sample code is taken from https://github.com/yahoo/react-intl/wiki#locale-data-in-browsers

@derwaldgeist As Cait has mentioned a few times, this has not been an issue in JSHint for many years. Please ensure that the version of JSHint in use by your plugin is up-to-date (2.9.4 at the time of this writing)

@jugglinmike The plugin uses jshint 2.4.4 and jshint-jsx 0.4.0. Is this too old?

I can't comment on the version/behavior of jshint-jsx, but 2.4.4 is certainly outdated.

Ok, thanks for the info.


Supporting changes for work items 5 and 6

Remove blob byte data types
Property changed event is now the only event
Added missing sequenceItemIndex field

Reviewed on the call on the 10th of January 2023.


Vertical/Horizontal flip hot keys don't work correctly

Tested on Ketcher v2.2.1-104-gc7896113
Steps to reproduce:

Launch Ketcher.
Open the structure from file
struct.zip
Click the 'Horizontal Flip' button.
Click 'Alt+H' hot key.

Expected result: It's possible to make the horizontal flip using the 'Alt+H' hot key.
Actual result: The structure is not flipped with the 'Alt+H' hot key.

The issue is reproduced with 'Vertical Flip' tool.

In the course of the work, there were problems associated with the architecture. The task is suspended until the architecture is updated


🤖 Update one or more manifest files in the .ci directory

This is to get https://github.com/JuliaRegistries/RegistryCI.jl/pull/582


[MIG-284] Missing phase descriptions in DVM

Describe the bug
Related to #820
Following phases in DVM controller are missing descriptions :

CleanStaleRsyncResources
WaitForRsyncResourcesTerminated
WaitForStaleRsyncResourcesTerminated
Verification
CreateStunnelPodOnSource
EnsureStunnelPodOnSourceHealthy

Expected behavior
Phases should have descriptions

@pranavgaikwad @alaypatel07 Is this currently in progress, or has been completed?

@eriknelson not completed, just got to know this is an issue, will pick it up this sprint


Wrong reg key for "verify per-machine install" on OneDrive sync app "Per-machine installation" page?

The "Per-machine installation" page for the SharePoint sync app (that's the same thing as "OneDrive", right?) has instructions for verifying that your computer has a per-machine installation of OneDrive. It tells you to check the registry key HKLM\SOFTWARE\Microsoft\OneDrive\Version and see if it contains the value "19.043.0304.0007".

That seems like it's checking the version of OneDrive you have installed, and not whether it's a per-machine installation? At the least, OneDrive is now on version 23.147.x, so won't that test return false unless your machine has a specific older version of OneDrive?
Does the mere existence of the registry key HKLM\SOFTWARE\Microsoft\OneDrive under HKEY_LOCAL_MACHINE instead of HKEY_CURRENT_USER imply that it's a per-machine installation? In that case, maybe it should be checking for just the existence of the key, and not that it has a particular value for Version?
I see an "AllUsersInstall" item with a value of 1 under the HKLM\SOFTWARE\Microsoft\OneDrive key on my machine which I think has a per-machine installation (because this is a VM from an Azure "Windows 10 Multi-Session with Office Apps" image (win10-22h2-avd-m365-g2) and that OneDrive is what came with the image). Should this verification test be checking that thing instead?

Cheers,
Andrew

Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: d03e432e-17b8-cdf8-5cd1-158f19d493e4
Version Independent ID: 699feed2-7123-c54d-8e01-4be30466f3e4
Content: Install the sync app per-machine (Windows) - SharePoint in Microsoft 365
Content Source: SharePoint/SharePointOnline/per-machine-installation.md
Service: one-drive
GitHub Login: @MikePlumleyMSFT
Microsoft Alias: mikeplum

@apjanke-demex
If you do not have an installtion made with /allusers

You not have the HKLM\Software\Microsoft\    No OneDrive folder OR
You see HKLM\Software\Microsoft\OneDrive  and the miss the Key AllUsersInstall
Then ist is a Installation in the users Path  C:\Users[User]\Appdata\local\Microsoft\OneDrive\  OR
C:\Program Files x86\Microsoft OneDrive\ for 32 Bit OR
C:\Program Files\Microsoft OneDrive\  for 64 Bit

But if you have taken a preconfigured Version (as you described) OR
Deinstall a configured version and install  it with the /Allusers parameter,
then the Key HKLM\Software\Microsoft\OneDrive  AllUsers is set to 1
so all users on this device uses the same OneDrive version, and  only one automatic Update will be done for all users.
The Item AllUsersInstall  with an Value=1 makes the difference
@gacarini


ClusterODM support? Advice on approaches

Hi folks,
What level of interest is there in ClusterODM support?
I'm willing to do the initial development, and thought I'd share my thoughts/plans here for comment.
I can see two approaches, which I'd like comment on:
Option 1: Use the ClusterODM REST API.
The current PyODM approach to calling REST apis could be  extended to the ClusterODM REST API (currently used in the ClusterODM web interface). This extends the current PyODM patterns which will keep the code cleaner and simpler; conveniently, I'm also familiar with this approach and it feels doable.
Option 2: Connect to the ClusterODM Telnet interface.
This is consistent with the (apparently-preferred?) ClusterODM interface patterns, as per it's documentation; also telnet looks like it might support  additional features (e.g. the "NODE BEST" command doesn't seem to be replicated in the REST API?) - is this true? The downside is Telnet is quite a different approach to the current PyODM/NodeODM REST approach, making the pyodm code more complicated to maintain.
I'm leaning toward option 1. Will make a start when I get a moment; in the meantime I thought I'd open up for comment!
Steve.

I think an interface that communicates with ClusterODM's admin features could be part of a separate repository (PyODM's goal is to speak with the NodeODM API)?
Would be a cool project to work on!


Created C1_W1_Lab_1_functional_practice.ipynb

I have created a "Keras Functional API with TensorFlow" notebook, As an example of Course - Custom Models, Layers, and Loss Functions with TensorFlow from Coursera, under tensorflow/examples/courses folder

@JosBeckerI and @MarkDaoust, I have made a Pull Request in the courses directory, please review it and give your valuable feedback.


error in  import * fs from "file-system change it to  import * as  fs from

error in  import * fs from "file-system in Angular
Please  change it to  import * as  fs from
in documentation

This can be closed now.


Sqlite provider dosen't return new line charachter from database

I use sqlite provider with entity framework 7.
I try to save and get from database text contains new line charachters. As far i understand text saves properly (because i see it properly formatted at DB Browser for SQLite) but when i getting it from database, it dosen't contain any new line charachters.
I use  "EntityFramework.Sqlite": "7.0.0-rc1-final" package.

Could you provide code that reproduces the issue?

Exuse me please, that was my mistake!


Respond to requests from peers with invalid ENRs

It can be the case that peers have invalid ENRs (i.e report socket addresses that do not match the socket src of the packet).
Previously discovery would drop packets from these peers and log a warning. This PR shifts this logic to instead respond to the src socket but exclude the ENR from reaching our routing table.
We will not advertise these peers, but we will respond to them.
I think this change will improve overall connectivity.
There are some legitimate cases where an ENR can be invalid, such as the external IP address changing or some funky NATs.

I'm testing this change with the ip-change test case. I'll let you know when it's done.

Awesome. Thanks @ackintosh :pray:

@AgeManning During the testing, I noticed that the FINDNODE request (which is initiated from here) is timing out. Please see the attached image below. What do you think?

For reference, the issue I mentioned above doesn’t happen in the current implementation, which uses a one-time session:

@ackintosh
Awesome!! Thanks for catching this!
I was thinking in the transition its probably best to simply remove the transitioning peer from the local routing table. There is likely some time between us noticing it has changed its IP and it actually adjusting its IP. In the meantime we probably should prioritize better peers.
I suspect also if we remove it from the table, when it does eventually update, it can then have a second chance of getting back in there.
I've made this change. I'm hoping it now fixes the issue you've found.
Thanks again!

@AgeManning I have tested it again. The updated ENR doesn't get back into Node B's routing table unless the session has expired. 🤔

Hey @ackintosh
Nice. Yeah, I'm trying to think of any better way to improve on this behaviour. There's a number of things we want and facts we have to deal with:

We want to maintain the connection and respond to invalid ENRs now, so we dont want to terminate the session now in the process of a peer changing its IP
While its ENR is invalid, we dont want to advertise it and therefore don't want it in the routing table
There is some indeterminate time before the node updates its ENR (it might not even update it).

Therefore, I dont think we should:

Download every ENR we see that could be new on an established connection. Especially if we dont have them in our routing table, then we don't care about the ENR update
Keep the ENR in the routing table while we know its non-contactable
Drop and re-establish connections in hope that the ENR is now correct.

Given these, I can't think of an efficient and useful way that we can re-add this ENR to the table without the session expiring. We could keep track of expired node-ids and re-download ENRs if their sequence increases, but we could be downloading lots of ENRs with no benefit. Also the code complexity would increase a bit for this edge case.
I'm open to ideas of how to do this nicely, but currently I'm thinking to just leave this behaviour. In a real DHT, we have lots of peers. If we respond to everyone now and we kick out non-contactable ones, it should make the whole thing cleaner, i'd imagine.

@AgeManning The idea that came to mind was expanding the use of one-time sessions not only for PING, but also for all other messages. If the ENR is invalid, then remove it from the routing table and respond to the peer using a one-time session. This gives the ENR a chance to get back to our routing table when the peer updates it, while still allowing us to respond to the peer.
However, this might increase the code complexity a bit, and yeah, this might not be a good solution, especially given that the peer might not update its ENR.


Bug/Feature request: Limit crafters to one pattern per slot

Issue description:
Limit crafters to one pattern per slot.
What happens:
If you have a stack of patterns (they are the same pattern so they stack) if you shift click the stack in the crafter GUI, the whole stack gets added to one crafter slot.
What you expected to happen:
Only one of the patterns to be added to a crafter slot.
Steps to reproduce:
Create a stack of patterns of the same recipe (This is a common use case, when for example adding a pattern for printed silicon to multiple crafters attached to solderers)
Click a crafter to open its GUI
Shift click the stack of patterns in your inventory to move a pattern to one of the empty crafter slots, the whole stack is moved.
Version (make sure you are on the latest version before reporting):

Minecraft: 1.12.2
Forge:
Refined Storage:refinedstorage-1.5.21

Does this issue occur on a server? [yes/no]
N/A

I already implemented this for the next version.
https://github.com/raoulvdberge/refinedstorage/blob/mc1.12/CHANGELOG.md#1522


Multiple Rows for same project _id, region and quota_metric in Notification Table

While creating the Notification Table, multiple rows are inserted for same projectId, region and quota_metric with different added_at times. This leads to confusion when alerts are raised which show similar rows without added_at column.

Fixed by PR #83


CVAT Dashboard: add the "copy jobs' url" button

Sometimes it's necessary to copy all URL -addresses. It'll be good to have a corresponding button somewhere near the list of jobs.

@achernovx , thanks! Great report 👍


Very slow compilation of substrate-browser-utils

I've noticed that Compiling substrate-browser-utils v0.8.0 takes ages on my machine and some not very powerful servers, it maybe even hangs somewhere in the process.
Compiling seems to hang forever.
The command is
cargo +nightly build --manifest-path=bin/node/cli/Cargo.toml --no-default-features --features browser --target=wasm32-unknown-unknown -Z features=itarget

@bkchr 's guess that it's rust-lang/rust#75992 regression
I went through
production:
rustc 1.50.0-nightly (825637983 2020-11-18) # image docker.io/paritytech/ci-linux:production
rustc 1.50.0-nightly (eb4fc71dc 2020-12-17)
rustc 1.50.0-nightly (803c60218 2020-12-13)
rustc 1.50.0-nightly (d32c320d7 2020-12-10)
2020-12-01
2020-11-25 (24)
2020-11-25 (21)
2020-11-19
2020-11-18

docker.io/paritytech/ci-linux:d0f2b553-20201118
docker.io/paritytech/ci-linux:3ec7097e-20201117
docker.io/paritytech/ci-linux:3ec7097e-20201116
docker.io/paritytech/ci-linux:4cc65dc0-20201102
docker.io/paritytech/ci-linux:974ba3ac-20201001

It freezes on
   Compiling substrate-browser-utils v0.8.0 (/builds/utils/browser)
    Building [=======================> ] 647/652: node-runtime(build)

But beefy CI machines, however, run this test on production image quite fast.
Originally posted by @TriplEight in https://github.com/paritytech/substrate/issues/7722

This crate was removed.