certipy: error: unrecognized arguments:

Version 2.09
Commands conducted:
certipy find<EMAIL_ADDRESS>usage: certipy [-h] [--ca-name CA_NAME] [--overwrite] [--rm] [--cert-type {rsa,dsa}] [--bits BITS] [--valid VALID] [--alt-names ALT_NAMES] [--store-dir STORE_DIR] name
certipy: error: unrecognized arguments<EMAIL_ADDRESS>

I have now tried it both ways. See below
pip install /opt/Certipy
Processing /opt/Certipy
Preparing metadata (setup.py) ... done
Collecting asn1crypto
Using cached asn1crypto-1.5.1-py2.py3-none-any.whl (105 kB)
Collecting cryptography>=37.0
Using cached cryptography-37.0.4-cp36-abi3-manylinux_2_24_x86_64.whl (4.1 MB)
Collecting dnspython
Using cached dnspython-2.2.1-py3-none-any.whl (269 kB)
Collecting dsinternals
Using cached dsinternals-1.2.4-py3-none-any.whl
Collecting impacket
Using cached impacket-0.10.0-py3-none-any.whl
Collecting ldap3
Using cached ldap3-2.9.1-py2.py3-none-any.whl (432 kB)
Collecting pyasn1==0.4.8
Using cached pyasn1-0.4.8-py2.py3-none-any.whl (77 kB)
Collecting pyopenssl>=22.0.0
Using cached pyOpenSSL-22.0.0-py2.py3-none-any.whl (55 kB)
Collecting requests
Using cached requests-2.28.1-py3-none-any.whl (62 kB)
Collecting requests_ntlm
Using cached requests_ntlm-1.1.0-py2.py3-none-any.whl (5.7 kB)
Collecting cffi>=1.12
Using cached cffi-1.15.1-cp310-cp310-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (441 kB)
Collecting flask>=1.0
Using cached Flask-2.2.2-py3-none-any.whl (101 kB)
Collecting chardet
Using cached chardet-5.0.0-py3-none-any.whl (193 kB)
Collecting pycryptodomex
Using cached pycryptodomex-3.15.0-cp35-abi3-manylinux2010_x86_64.whl (2.3 MB)
Collecting ldapdomaindump>=0.9.0
Using cached ldapdomaindump-0.9.3-py3-none-any.whl (18 kB)
Collecting future
Using cached future-0.18.2-py3-none-any.whl
Collecting six
Using cached six-1.16.0-py2.py3-none-any.whl (11 kB)
Collecting urllib3<1.27,>=1.21.1
Using cached urllib3-1.26.12-py2.py3-none-any.whl (140 kB)
Collecting idna<4,>=2.5
Using cached idna-3.3-py3-none-any.whl (61 kB)
Collecting certifi>=2017.4.17
Using cached certifi-2022.6.15-py3-none-any.whl (160 kB)
Collecting charset-normalizer<3,>=2
Using cached charset_normalizer-2.1.1-py3-none-any.whl (39 kB)
Collecting ntlm-auth>=1.0.2
Using cached ntlm_auth-1.5.0-py2.py3-none-any.whl (29 kB)
Collecting pycparser
Using cached pycparser-2.21-py2.py3-none-any.whl (118 kB)
Collecting itsdangerous>=2.0
Using cached itsdangerous-2.1.2-py3-none-any.whl (15 kB)
Collecting Jinja2>=3.0
Using cached Jinja2-3.1.2-py3-none-any.whl (133 kB)
Collecting click>=8.0
Using cached click-8.1.3-py3-none-any.whl (96 kB)
Collecting Werkzeug>=2.2.2
Using cached Werkzeug-2.2.2-py3-none-any.whl (232 kB)
Collecting MarkupSafe>=2.0
Using cached MarkupSafe-2.1.1-cp310-cp310-manylinux_2_17_x86_64.manylinux2014_x86_64.whl (25 kB)
Certipy find -u<EMAIL_ADDRESS>-p Passw0rd -dc-ip <IP_ADDRESS>
usage: certipy [-h] [--ca-name CA_NAME] [--overwrite] [--rm] [--cert-type {rsa,dsa}] [--bits BITS] [--valid VALID] [--alt-names ALT_NAMES] [--store-dir STORE_DIR] name
certipy: error: unrecognized arguments: -u<EMAIL_ADDRESS>-p Passw0rd -dc-ip <IP_ADDRESS>

Hello. You should probably uninstall the other certipy before installing my Certipy.

Thank you! Fixed.


Create a pull request template to standardize pull requests

Dependencies :  #21  Needs to be merged before this could be done
@reshamas
Please let me know if you have some good templates.
I found couple of them, and looked into some  popular libraries too.
Thanks.

@SangamSwadiK I've no experience with templates for PRs, it's a great idea!
Let's go through this at our next website meeting and set up a template or two.


⚠ Bofrost.fr has degraded performance

In ce3e1a7, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1092 ms

Resolved: Bofrost.fr performance has improved in a7da344 after 36 minutes.


Feature Request: Widget Auto-Refresh, Fullscreen

I use some cams of Nova Scotia in widget mode. Each widget shows a refresh button below. I suggest following changes:
a) Make an (optional) auto-refresh mode, so stream-like updates are possible. Many cams feed updates constantly.  For battery saving this continuous mode could be linked to the actual visibility (screen with widget in view or not) and device sleep mode.
b) The refresh button is a waste of space. A (manual) refresh could be triggered on tapping the picture itself. Because of the much larger tap area, you won't miss the refresh bar any more.
c) As alternative to b) a picture tap could trigger fullscreen mode.
Thanks for the great work!
Best regards!
BlueStar88

B and C variant finished ;) Today I want to try make also "A" variant, but work on Widgets is very limited and frustrating ;)

Released! Please test it, thank you ;)

Just tested. Option your implementation a) allows a minimum refresh rate of 1 minute. The idea was to support continuous ("stream-like") refreshing (maybe using a 500 ms to 1 second refresh rate). Many cams give a new image, if you hammer manual refresh continuously. So why not making use of it, while the widged containing screen is in view and standby mode is absent?

Normally with the parameter updatePeriodMillis is minimum refresh interval 30 minutes. With Alarm manager was the value from 0 to unlimited. But now from API 22 and up you can see here:

But I think, globaly for the webcameras providers everything less than minute is very bad for their overload. I try now to implement Webcams.travel api and their politics is very strict to server load. So one minute as minimum inteval is, I think, good compromise for both sides. For example, the developers from Webcams.travel refreshing their shots only one time per hour and I understand why...

So tell me, why many webcam providers are serving motion jpeg?
To cover overload or battery drain, we could rely on having only smaller time frames, where the user actually looks at those widgets. If those widgets on a hidden screen or the device is sleeping, no refreshing is needed at all. So we are not really talking about constant server loads.
The idea is, to have the beautiful experience of a real time video for some seconds.
Another option to avoid server load would be, to limit the video show to a minute, each time the widget comes into the users view. This will ensure nobody can hog the server(s) for a longer time.

http://www.earthcam.com/usa/newyork/timessquare/?cam=tsrobo3
Realtime video is no problem at all these days, it's just the next evolutionary step, having all that bandwidth and powerful servers.

So, if a webcam provider delivers motion jpeg, the user is allowed to use it.
Althought there could be mentioned measures for the good of both sides.

https://www.cs.dal.ca/webcams/university
Another example. Even a second refresh rate gives a better eperience.

http://www.novascotiawebcams.com/en/webcams/museum-wharves/
This one has the same one second rate, but with a very nice blend over effect.


User configuration from ES files

We already support user configuration written in ESM.

https://github.com/facebook/jest/issues/11167
= Should be able to convert jest.config.cjs:s to ES

done


extract error logging from Server#request

fixes #480

@pitr is this still something that might be merged in any time soon? Just come across this and something we'd really appreciate at work.

I'm not using this gem in any of my current projects, but if you need this functionality talk to @mperham and @petergoldstein. I know they've been meaning to merge it into release 3.0, but I'm not sure if there's a plan for major bump

ok cool. Thanks @pitr

I'm going to target some version of this for Dalli 3.0.


CI docs dev failing

Looks like it's a poetry lock issue that should be resolved when I merge #728 but just noting it here anyway
https://github.com/askap-vast/vast-pipeline/actions/runs/10293250744/job/28489187783

The initial problem outlined stems from a markdown issue that is fixed by forcing markdown==3.3.7.
However this reveals an initial problem that's first pointed out by @ajstewart in #486. pytkdocs should not be updated past 0.11.1, but looking at the associated PR (#627), pytkdocs gets bumped to 0.15 in the poetry.lock file.


Refactor SVG events

This is a work item for #685.
Pages:

[x] https://developer.mozilla.org/en-US/docs/Web/Events/beginEvent
[x] https://developer.mozilla.org/en-US/docs/Web/Events/endEvent
[x] https://developer.mozilla.org/en-US/docs/Web/Events/repeatEvent
[x] https://developer.mozilla.org/en-US/docs/Web/Events/SVGAbort
[x] https://developer.mozilla.org/en-US/docs/Web/Events/SVGError
[x] https://developer.mozilla.org/en-US/docs/Web/Events/SVGLoad
[x] https://developer.mozilla.org/en-US/docs/Web/Events/SVGResize
[x] https://developer.mozilla.org/en-US/docs/Web/Events/SVGScroll
[x] https://developer.mozilla.org/en-US/docs/Web/Events/SVGUnload
[x] https://developer.mozilla.org/en-US/docs/Web/Events/SVGZoom

SVG Event research
The pages we have
https://developer.mozilla.org/en-US/docs/Web/Events/beginEvent
https://developer.mozilla.org/en-US/docs/Web/Events/endEvent
https://developer.mozilla.org/en-US/docs/Web/Events/repeatEvent
These three fire relatively obviously — when a SMIL animation (e.g.  element) begins, ends (e.g. gets to the end of its repeatCount), and each time it repeats.
https://developer.mozilla.org/en-US/docs/Web/Events/SVGAbort

replaced by "standard" DOM event (see below)

https://developer.mozilla.org/en-US/docs/Web/Events/SVGError

replaced by "standard" DOM event (see below)

https://developer.mozilla.org/en-US/docs/Web/Events/SVGLoad

seems to fire logically when SVG is loaded in the browser, e.g. in the DOM in the case of an embedded . But, it has been replaced by "standard" DOM event (see below).

Note: "load" fires on SVG as expected in Chrome, but Firefox still supports the older "SVGLoad"
https://developer.mozilla.org/en-US/docs/Web/Events/SVGResize

replaced by "standard" DOM event (see below)

https://developer.mozilla.org/en-US/docs/Web/Events/SVGScroll

replaced by "standard" DOM event (see below)

https://developer.mozilla.org/en-US/docs/Web/Events/SVGUnload

replaced by "standard" DOM event (see below)

https://developer.mozilla.org/en-US/docs/Web/Events/SVGZoom

removed (see below)

Removed SVG events
From jwatt's mail:
"As per https://svgwg.org/svg2-draft/single-page.html#changes-interact SVGZoom
was removed from the spec (and from our implementation) and all of the others
were replaced with the standard DOM events (so 'abort' etc.)."
Specifically:
"Replaced SVGLoad, SVGAbort, SVGError and SVGUnload with load, abort, error and unload respectively.
Required that only structurally external elements and the outermost svg element must fire load events.
Replaced SVGResize and SVGScroll with resize and scroll respectively.
Removed SVGZoomEvent"
Also from jwatt mail:
"If you're listening for the correct event name, on the correct elements, for the
correct triggering action, then probably we don't implement those events. From a
skim of the code I don't think we dispatch abort, error or scroll."

From https://svgwg.org/svg2-draft/single-page.html#attindex-RegularAttributes:
onbegin, onend, and onrepeat can be specified on , , , and
onload, onerror can be specified on:
a, animate, animateMotion, animateTransform, audio, canvas, circle, defs, desc, ellipse, foreignObject, g, iframe, image, line, linearGradient, marker, metadata, mpath, path, pattern, polygon, polyline, radialGradient, rect, script, set, stop, style, svg, switch, symbol, text, textPath, title, tspan, unknown, use, video, view
onabort, onunload, onresize, and onscroll can be specified on  only
onzoom doesn't exist; SVGZoom is removed from spec.

See https://svgwg.org/svg2-draft/single-page.html#interact-LoadEvent for more descriptions of how the events work.

Work to do:

for onbegin, onend, and onrepeat
Move these event pages to hang off the https://developer.mozilla.org/en-US/docs/Web/API/SVGAnimationElement page; this is where the events are defined

SVGZoom — delete this as it no longer exists

onload and onerror are defined on GlobalEventHandlers, which is implemented by SVGElement. But according to https://svgwg.org/svg2-draft/single-page.html#struct-SVGElementEventHandlerAttributes, SVGElement seems to expose these directly, along with the on... handler properties. Same for onscroll.

onscroll - Does exist in Firefox/Chrome, but I'm really not sure how you'd get the actual SVG to show scrollbars.

onabort/onload/onresize - we could probably link to the standard pages for these.

onabort - Does exist in Firefox/Chrome, fired when a document is stopped from loading completely. This is pretty hard to trigger, so probably just show a trivial example on this page. Also create abort_event page on SVGElement.
onunload - onunload doesn't exist on embedded SVG elements in Fx or Chrome. Not sure where this exists, to be honest.
onresize - Does exist in Firefox, on external SVG documents, e.g. referenced inside an  element by e.currentTarget.contentWindow. In Chrome they only exist on the actual  element. Create two quick pages for onresize and resize_event, to explain this? I can get it to fire, by resizing the  element that embeds the SVG.
Note: Doesn't fire when you load the svg in an  element then resize it. In this case by default the SVG is stretched to fill the
Summary:
onbegin, onend, and onrepeat - move to hang off of SVGAnimationElement.
create events section on the interface page
create onbegin_event, onend_event, and onrepeat_event
mention the events on , , and  pages?
Create BCD - yes for chrome/opera, yes for Fx, null for others.
Delete SVGZoom
onload, onerror, onscroll, onabort, onresize - create pages for these on SVGElement
Also create onload_event, onerror_event, onscroll_event, onabort_event, onload_event, onresize_event
Make them a brief summary, and link on to regular event pages
Create events section on SVGElement
onunload - doesn't seem to exist.
HRM, THESE ONES ARE NIGHTMARES. MIGHT JUST REDIRECT TO THE STANDARD PAGES, AND CREATE AN EVENT SECTION ON SVGELEMENT THAT EXPLAINS THE IDIOSYNCRACIES. Or something.

OK, so the animation events are done.
See here for the events: https://developer.mozilla.org/en-US/docs/Web/API/SVGAnimationElement#Events
See here for the associated BCD: https://github.com/mdn/browser-compat-data/pull/3767

Deleted the SVGZoom page.

All the others have been moved to underneath the SVGElement interface page: https://developer.mozilla.org/en-US/docs/Web/API/SVGElement#Events
See also here for BCD: https://github.com/mdn/browser-compat-data/pull/3771
@irenesmith , you're on the list to review this one. Sorry. This is really horrible ;-)

@chrisdavidmills It's easier to review than to write by a whole lot.
The events were added to either SVGAnimationElement or SVGElement.
Overall, it looks good. The only nitpick I have is that most of the pages don't have "See also" sections.
I reviewed the pull request but it had already been merged. Oh well.

Thanks @irenesmith . Yeah, I didn't really think they needed "See also" sections. If you think there are links that could be added, I'm happy to add them.

All done — now just need to wait until BCD is available, and then refresh pages.

Pages refreshed; BCD now visible on pages.
@wbamberg, this can be moved to done, imo. Do you want give it a final check, as project lead?

Thanks Chris!
The mdn-url links for these events seems wrong: https://github.com/mdn/browser-compat-data/pull/3771/files, shouldn't it link to SVGElement/thing_event, not Window/thing_event?
The live example (e.g. https://developer.mozilla.org/en-US/docs/Web/API/SVGAnimationElement/beginEvent_event#Examples) has a few problems.
First I don't think it's ideal to use console.log to show output, because the user has to read all the example to see that they have to open the console, then open the console and pick through it. It's much better to display the output in the example (as we do for example in https://developer.mozilla.org/en-US/docs/Web/API/FileReader/progress_event#Examples).
Second, if I do open the console, I just see an error:

SyntaxError: unexpected token: identifier

and the script the example is using looks very weird (to see this code you could for example click "open in CodePen"):
let svgElem = document.querySelector('svg');
let animateElem = document.querySelector('animateMotion');

animateElem.addEventListener('beginEvent', () => {
  console.log('beginEvent fired');
})

animateElem.addEventListener('endEvent', () => {
  console.log('endEvent fired');
})

animateElem.addEventListener('repeatEvent', (e) => {
  console.log('repeatEvent fired');
  if(e.detail) {
    console.log('Repeat number: ' + e.detail);
  }
})animateElem.onbegin = () => {
  console.log('beginEvent fired');
})

(see the last three lines). What's happening I think is that the snippet showing the onbegin event handler property is being included in the live example. Also, that snippet contains a syntax error (the extra ) at the end).
Finally I think it's problematic for an example to run on page load, then to finish running after a bit. Because suppose I open the page and read it a bit, then eventually get to the live example after it's finished (this is a very very likely scenario). Now the example is finished and there's no way to run it again, so it's unusable. So I think it is better for the example to do nothing on page load, but offer a way for the user to start it. Again this is a thing we do in pages like https://developer.mozilla.org/en-US/docs/Web/API/HTMLMediaElement/progress_event#Examples).

@wbamberg ooops, yes, the URLs were wrong!
I've fixed that now, in https://github.com/mdn/browser-compat-data/pull/3896
Now on to the examples.

@wbamberg OK, on to the live examples.

I've fixed the console error, and broken up the examples into two subheadings so that the on... version doesn't get drawn into the live example.
I've also removed the erroroneous extra bracket in the on... versions.
I've created a unordered list and dumped my console.log messages into there instead so the reader can instantly see them.
I've made the animation run infinitely so you can see there is something going on even if you don't get to the example for a while.
for the end example, I've added a button that you can press to make the animation end show the end event being fired.

I am still not 100% convinced that the examples are perfect, but they are a lot better now. SVG is such a pain to manipulate with JS, compared to HTML.

Thank you @chrisdavidmills , that's much better!

cool.

Closing this as it looks done, thanks @chrisdavidmills !


Disable enforcing git UID = repo file UID by git

Can we move the gitconfig to .docker/gitconfig? It would document the fact that it's only relevant for docker?

This doesn't fix the issue of "config value 'safe.directory' was not found", it only changes the error into "repository path '/app/radicle/storage/<rad>/' is not owned by current user".


internal: Ensure security (only) updates have correct commit msg

Motivation

Still use security updates from dependabot - but we need commit messages to conform.
Solution

Using open-pull-requests-limit: 0 will make dependabot only make security PRs

Codecov Report

Merging #2072 (1bfa700) into master (cdb5adb) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2072   +/-   ##
=======================================
  Coverage   98.48%   98.48%
=======================================
  Files         102      102
  Lines        1918     1918
  Branches      404      404
=======================================
  Hits         1889     1889
  Misses         13       13
  Partials       16       16

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Add Update User Endpoint

https://github.com/ariesprayoga/swagger-java-sample-app/blob/master/src/main/java/com/wordnik/swagger/sample/resource/UserResource.java

I think this is already implemented but usually the update on a REST interface looks a little bit like:
PUT /users/{username}
That seems to be the case for the API described in the URL you shared. Do you want the scala-pet-store to be similar, because currently, it is not.

Good point @daddykotex, I am unclear if we have to keep in sync with that project.  I would be more interested to know if we were correct by REST standards and do what is appropriate there.
I think PUT /users/username is correct as the username is the unique key for users.
Probably worthwhile to open an issue and make that small change

Looks like that was following the PET endpoint, so we should be consistent in all places where we do an update.

👍

Closed by #74


Top bar is still clickable in Music Videos

When playing a Music Video, the top area of the video where the top bar is can still be clicked, this means you can click anywhere up there and it will close out the music video and go to whatever you clicked on, you can try it by playing a music video and hovering your mouse cursor over the top area, you will notice it will show you can click on it, which will close the music video. It should not be able to be clicked.

Can you check if this is only present in iTheme or also in normal Cider, might be a change there

Cider issue, can add this there


Add default timeout to pkg/plugins/client

Happened to find some docker create command hangs for a long time due to remote volume plugin. The goroutine stack is as follows. I believe timeout is necessary.

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "timeout" git@github.com:chenchun/docker.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "timeout" git@github.com:chenchun/docker.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

Signed my patch.

ping @anusha-ragunathan PTAL

gofmt errors:
--> Making bundle: validate-gofmt (in bundles/1.13.0-dev/validate-gofmt)
These files are not properly gofmt'd:

pkg/plugins/client.go

@chenchun also
---> Making bundle: validate-lint (in bundles/1.13.0-dev/validate-lint)
Errors from golint:
pkg/plugins/client.go:20:2: const defaultHttpTimeOut should be defaultHTTPTimeOut

Please fix the above errors. You can test via "golint" and commit the result.

@anusha-ragunathan @tophj-ibm Fixed the format errors.

@chenchun : Plugin calls already have a timeout mechanism. It is not at the HTTP Client request level. It is application specific (plugin specific in this case). There's exponential backoff (2, 4, 8, 16 seconds...), with retries, with a maximum timeout of 30 seconds. You were probably waiting in a longer backoff time period (eg 16 seconds) when you experienced the issue. You should see daemon log entries such as https://github.com/docker/docker/blob/master/pkg/plugins/client.go#L135
Given that, I think this PR can be closed.

I don't keep the log. But I believe it is not the backoff retry that was happening. HTTP request timeout is a basic need if the server takes a long time to reply or even not given a reply like the test I write. In this situation the application specific retry won't work. So I don't agree with you.

Both cases you mention should be handled by the current plugin timeout mechanism.

@anusha-ragunathan  It happens again and it hangs on callWithRetry for 182 minutes.
And there isn't any "Unable to connect to plugin" warnings in the log. I hope you reconsider this.

You are correct. The application specific backoff/retry logic is for errors that are returned from the http request (eg, connection refused, no socket available to connect, etc). If the call is blocked, then we will not timeout.
Reopening, since this PR fixes a valid issue.

LGTM

@chenchun I know it was there before your PR, but if we could add to the timeout constants a * time.Second that'd be appreciated. It will need some type conversions modifications in the code I believe. If it's too complex then forget it. Otherwise PR looks good.


Prometheus Staleness Issue on Fedora 39

What did you do?
I am currently using Fedora 39 and have both Prometheus (version 2.32) and Prometheus Node Exporter (version 1.6.1) installed. I have encountered an issue with Prometheus where it continues to write the last value for an additional 5 minutes after I stop the prometheus-node-exporter service.
But when I use any aggregated function, those 5 minutes disappear.

It seems similar to the staleness behavior for metrics with their own timestamps, but that does not appear to be the case here.
What did you expect to see?
I expected that the last point on my graph would be at the same time as when I stopped the prometheus-node-exporter service.
What did you see instead? Under which circumstances?
Instead, I observed that Prometheus continued to write the last value for an additional 5 minutes after I stopped the prometheus-node-exporter service. This behavior occurred every time I stopped the service.
System information
Linux 6.8.11-200.fc39.x86_64 x86_64
Prometheus version
prometheus, version  (branch: , revision: )
  build user:
  build date:
  go version:       go1.21.6
  platform:         linux/amd64

Prometheus configuration file
No response
Alertmanager version
No response
Alertmanager configuration file
No response
Logs
No response

It seems like you're familiar with Prometheus's staleness handling, where it shows the last metric value for up to 5 minutes if no new data is received. However, if you're also stopping Prometheus at the same time as the prometheus-node-exporter, it might prevent the insertion of the invisible staleness marker. This marker indicates when data is considered stale after the 5-minute window. Without it, aggregation functions might not recognize the data as stale, resulting in the last data point being absent from your charts. It’s important to keep Prometheus running continuously to ensure proper handling of staleness markers.

Hi @roidelapluie
Thank you for the response. The reason was that to pause metric collection, targets are removed from the Prometheus configuration. Unfortunately, I don't know of a better way to pause Prometheus. Anyway, I believe the issue can be closed.


Update docs

latest preview link
https://styled-ppx-git-update-docs-mackarelfish93.vercel.app/?_vercel_share=sTDXMjr19TksmSukcfO3wGW3eF8hPvYV

Amazing progress, it's great.
I got a few questions regarding the amount of vendor stuff:

Do we need to keep a fork of nextra as well to just fork the theme-docs?
can we keep the theme-docs inside the website folder and we need turbo/etc?

I initially wanted to only fork nextra-theme-docs but got into a problem where some of the CSS are not showing (the output style.css when running of npm run build:tailwind is different from the npm one) . It turns out that they reference the components folder from nextra on tailwind.config.js.

I also upgraded shiki and rehype-pretty-code on nextra  (latest version still using shiki 0.14.7) to add support for dual color scheme. Details are on these commits (https://github.com/davesnx/styled-ppx/pull/457/commits/45145c582b21f0226080aca4d21057ce77036c39, https://github.com/davesnx/styled-ppx/pull/457/commits/e60844b2c3317e77be1525eef53858963b25f736)


First iteration of flexible grid layout for VSM

Current capabilities:

Resize row/column
Drag and drop single tab

LGTM!!!
I am missing a bit of code-documentation, explaining what each component is.

Added docs!

@RaananW and @carolhmj Ready to be merged ? looks like it but double checking just in case :-)


Bump up clang version

Looks like the version 1:15.0.7-0ubuntu<IP_ADDRESS> no longer exists.
Ref: https://github.com/cilium/tetragon/actions/runs/6355442579/job/17263533977#step:6:239
Ref: https://packages.ubuntu.com/jammy/clang-15

✅ success: https://github.com/cilium/tetragon/actions/runs/6355641458/job/17264006923?pr=1530

This is probably because the base image is ubuntu:22.04 and a recent update went from 22.04.2 to 22.04.3 is my guess. Is there a reason the apt install is specifying specific versions? You're basically getting the latest of the package anyways, and have to use the correct version of the package for the base image's Ubuntu patch release version anyways.

yeah not sure why we are pinning the version. @jrfastab might know 🤔

Our BPF programs are very sensitive to compiler version. That's why we're trying to pin it.

Our BPF programs are very sensitive to compiler version. That's why we're trying to pin it.

i see, but what can we do if the package keeps disappearing 💭

@michi-covalent I'm not super familiar with apt but if there's a way to pin it to 1.15 without caring about patch version that would be ideal

yeah it's already pinned, they have different packages for claing, like clang-15, so it won't randomly get bumped up to clang-16 for example 🥰 let me update this PR.

ERROR: failed to solve: docker.io/library/golang:1.21.1@sha256:19600fdcae402165dcdab18cb9649540bde6be7274dedb5d205b2f84029fe909: failed to copy: httpReadSeeker: failed open: unexpected status code https://registry-1.docker.io/v2/library/golang/manifests/sha256:13a7dc7b46068caeaf811cc8af56d394a89c97d9b194550f8be5a4de05ca5f99: 429 Too Many Requests - Server message: toomanyrequests: You have reached your pull rate limit. You may increase the limit by authenticating and upgrading: https://www.docker.com/increase-rate-limit

:jarno-eclipse:

:jarno-eclipse:

yeah dockerhub rate limits actions runners that are not hosted by github. opened https://github.com/cilium/tetragon/pull/1532 to make things slightly better.

ok but wish we were using clang-16

I'm ok to try this


[PE-D] Able to create an interview for a person that does not exist by editing JSON

By editing the JSON, I'm able to have an interview for a non-existent person.
The person list:

The interview list:

Labels: severity.Medium type.FunctionalityBug
original: bryans17/ped#5

The error here is that even though the person does not exist in the database, you are still able to add an interview for the missing person. This flaw is not to expose that you can edit the JSON file, rather its that there does not exist a check to see if the person in the interview objects actually exists.


fail to optimize for generation speed

fairseq optimize-fconv -input_model ./trainings/zh2en/model_best.th7 -output_model ./trainings/zh2en/model_best_opt.th7
/mnt/data/install/torch/install/bin/luajit: ...install/share/lua/5.1/fairseq/scripts/optimize-fconv.lua:25: "FConvModel" expected
stack traceback:
[C]: in function 'error'
...install/share/lua/5.1/fairseq/scripts/optimize-fconv.lua:25: in main chunk
[C]: in function 'require'
...rch/install/lib/luarocks/rocks/fairseq/scm-1/bin/fairseq:17: in main chunk
[C]: at 0x00405e90

hi, @zhenbiaochen , is the model_best.th7 a FConvModel ?

no ， the model model_best.th7 is blstm model.
is just FConvModel could speed up for cpu ?

I am not really sure, but the file optimize-fconv.lua is just for FConvModel, you can look at the code in optimize-fconv.lua and fconv_model.lua->FConvModel.makeDecoderFast for what it is done for speed up.

@nikefd is correct, optimize-fconv requires a fconv model.

I am having similar issue for an fconv model, as shown blow.

git/torch/install/bin/luajit: ...install/share/lua/5.1/fairseq/scripts/optimize-fconv.lua:25: "FConvModel" expected
stack traceback:
[C]: in function 'error'
...install/share/lua/5.1/fairseq/scripts/optimize-fconv.lua:25: in main chunk
[C]: in function 'require'
...install/lib/luarocks/rocks/fairseq-cpu/scm-1/bin/fairseq:17: in main chunk
[C]: at 0x00405d50

Here is my script for the training:
fairseq train -sourcelang ml \
        -targetlang en \
        -datadir  data/data-bin/mutlilingual \
        -model fconv \
        -nenclayer 4 \
        -nlayer 3 \
        -dropout 0.2 \
        -optim nag \
        -lr 0.25 \
        -clip 0.1 \
        -momentum 0.99 \
        -timeavg \
        -bptt 0 \
        -savedir trainings/fconv \
        -save_interval 100000 \
        -batchsize 32

Any help?


Removing old field from old users? Ignore schema when using $unset ?

Hello i'm performing some migrations on the database and i need to remove an old field from old users but when calling Meteor.users.update it says i can't $unset my field because it's not on the schema.
Is there a way of $unset a field while ignoring validation?

I just found the answer: https://github.com/aldeed/meteor-collection2#inserting-or-updating-without-validating


feat: assign filetype to background buffer

Summary
Assigns a filetype (zenmode-bg) to the background buffer created by zen-mode.

Why?

I use a utility to close floating windows. This utility interferes with zen-mode's background window so it helps if it has a filetype I can use to filter it out.

ty!

Thanks for merging!


Update Overlap screen to update when it comes into focus

Converted the Overlap class component to a functional component in
order to take advantage of hooks. The hooks allow us to hook into
the focus event so that we can update the map markers when the view
comes into focus. Previously the map would not update after the initial
mounting of the component.
Closes #253

looks good @thetimbanks but @tremblerz landed a large-ish overlap feature last night, so some conflicts here.

I figured I would need to update this after he got his PR merged. I’ll get it updated today.

@rparet Conflicts have been taken care of and made sure the tests were passing.

Thanks! @tremblerz can you look at this one? You've been doing the most work on Overlap, so would appreciate your check here.

looking at it


fix: flaky tests (WIP)

🎉 Thanks for sending this pull request! 🎉
Please make sure the title is clear and descriptive.
If you are fixing a typo or documentation, please skip these instructions.
Otherwise please fill in the sections below.
Which problem is this pull request solving?
Example: I'm always frustrated when [...]
List other issues or pull requests related to this problem
Example: This fixes #5012
Describe the solution you've chosen
Example: I've fixed this by [...]
Describe alternatives you've considered
Example: Another solution would be [...]
Checklist
Please add a x inside each checkbox:

[ ] I have read the contribution guidelines.
[ ] The status checks are successful (continuous integration). Those can be seen below.

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

Ok, Windows tests why are you now passing with 0 changes 😭

I think we can close this now that #139 landed.


Problem with backspace and alphanumeric caractere

I use SunEditor with ReactJS and I install SunEditor with npm install sundeditor --save
SunEditor version : 2.41.3.
Describe the bug
In SunEditor, when I write a sentence like "Hello world, I use SunEditor" and I press [Backspace] key, the full sentence is deleted and not just the "r".
But if I start the sentence with a special character like this "@ Hello [...] SunEditor" and  I press [Backspace] key so the last character is deleted and not the full sentence
Expected behavior
When [Backspace] key is pressed, only one character is deleted.

Desktop (please complete the following information):

OS: [Big Sur]
Browser: [chrome]

@vphu This issue is not reproduced.
Please explain in more detail.

Ok.
First, I create a component like this :
import React from "react";

import SunEditor from "suneditor-react";
import 'suneditor/dist/css/suneditor.min.css';

export default class Editor extends React.Component{

  render(){

        <SunEditor />

  }

}

In the React component that includes SunEditor, I write some sentence.
When I want to delete a character with the backspace key. It is the whole sentence which is deleted and not the desired character (as in the gif above).
But if the sentence starts with a character other than a letter or a number then I can delete my sentence character by character.
So, I tried with your code sample :
import React from 'react';
import SunEditor from 'suneditor-react';
import 'suneditor/dist/css/suneditor.min.css';

const MyComponent = props => {
  return (

 My Other Contents

      <SunEditor />

  );
};
export default MyComponent;

I have a same problem, I can't delete my sentence character by character. So it's a problem because if I want delete only one word, it's not possible and I have to rewrite my whole sentence.
I have this problem on Google Chrome [Version 93.0.4577.63], MacOs BigSur and High Sierra.
I tried on Safari but it's worse, I can't write a sentence longer than one character, the new character replaces the previous one.
Do you have enough details ?

Still can't reproduce the issue. :(
Does also have the same issue this link?

No I don't have this issue with this link.
I don't understand :/

@vphu No errors occur?
Do other functions work normally?

@JiHong88 yes, with your link, no error and other functions work normally.
I will try to reinstall SunEditor and try in a new project.
Thx


Altered pre_run_step to escape YYprojectDir env

As of 2.0.2, Dynamo inital setup sets the current users path into pre_project_step.bat
This creates two problems.

This sets the path to the current location. Meaning if you were to move the project, or use source control, it requires setting it all back up.
In regards to source control. While the intention behind this is to simply use .gitignore, this creates a problem where custom scripts must now be manually added in each time the repo is cloned.

This small change escapes the entire variable, passing through "as is". Meaning that Dynamo is ran at the project path, and not as a predetermined path.

Congrats on figuring which of the many branches to push into 😅


kube_metadata: Get metadata/labels once per namespace

What does this PR do?

Use cache when workloadmeta-kube_metadata gets namespace metadata.
Motivation

CONS-6403
Improving the kubemetadata.(*collector).parsePods() performance.
Currently, when workloadmeta-kube_metadata parses 100 Pods, the implementation results in 100 communications with the DCA to retrieve namespace metadata. Since the namespace metadata is the same for multiple Pods, there is no need to communicate with the DCA for each Pod.
https://github.com/DataDog/datadog-agent/blob/1248ec4f0bece8d46a33f20ff1447c7de7be6706/comp/core/workloadmeta/collectors/internal/kubemetadata/kubemetadata.go#L210-L293
Additional Notes

Cache expiration is same as DD_KUBERNETES_METADATA_TAG_UPDATE_FREQ (= 60s by default).

https://github.com/DataDog/datadog-agent/blob/1cf1cce4b5711e0929b4eca8d3fd1c51cf3b8a4c/pkg/config/setup/config.go#L1499

DCAClient doesn't use cache for the following functions.

https://github.com/DataDog/datadog-agent/blob/a9f51824ae913c875c8c39f187aa91b124b99de2/pkg/util/clusteragent/clusteragent.go#L405-L417
Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

@davidor Thanks for reviewing this PR! Use local map in the function instead of cache pkg.

/merge


trt_yolo.py: Assertion `d == a + length' failed.

I have no idea why I got this error.
python3: yolo_layer.cu:73: nvinfer1::YoloLayerPlugin::YoloLayerPlugin(const void*, size_t): Assertion `d == a + length' failed.
Btw, I am using Jetson Nano

This is most likely because you've built the TensorRT engine with an older version of the code (and tried to inference with the engine using a newer version of the code).
You could fix it by:

removing the TensorRT engine,
rebuilding the plugin (do make clean; make in the "plugins" subdirectory),
rebuilding the TensorRT engine (run "onnx_to_tensorrt.py")


Calling StateHasChanged anywhere on a page causes the MudAutoComplete to clear the user input

Bug type
Component
Component name
MudAutoComplete
What happened?
I have a MudAutocomplete on a page, and if I'm typing in it to find an item, and another area of the page triggers an update through some other async action (e.g., a callback from the server, or a timer, etc), when the StateHasChanged call happens, the autocomplete dropdown is dismissed and the AutoComplete input is cleared. This makes for a terrible UX. It didn't used to happen, so it's a recent change in the last 8-12 months.
Expected behavior
The autocomplete dropdown shouldn't clear and the autocomplete popover shouldn't be dismissed, if StateHasChanged is called anywhere on the page.
Reproduction link
https://try.mudblazor.com/snippet/cumREelCcZVFvROv
Reproduction steps

Run the sample
Start typing in the US State autocomplete
After 500ms, the background timer will fire, and StateHasChanged will be called, and any input will disappear and the autocomplete will be dismissed.
It's impossible to actually autocomplete an item

Relevant log output
No response
Version (bug)
6.2.0
Version (working)
Possibly 5.x
What browsers are you seeing the problem on?
Chrome
On what operating system are you experiencing the issue?
Windows, Mac OSX
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Update on this issue - I just upgraded to MudBlazor 7, and it's still a problem. The issue manifests itself slightly differently now - the auto-complete dropdown doesn't get dismissed when the StateHasChanged call happens elsewhere in the render tree, but the auto-complete box content gets cleared.

The MudAutoComplete rewrite is scheduled for version 7. However, I'm not sure if we can fit everything in, as there are already enough tasks for now. It requires using our ParameterState framework to get rid of these side effects.

Thanks for the update. Happy to wait!


Reset Passwords

Add a reset password feature for the manager in case someone forgets their password.
Either create a random password that the manager can see and share with the person that forgot their password or
have a default password that the manager resets it too.

The password reset functionality has been lost due to the Form being removed from the page.


[FIX] remove AnalogSignalGap

Reference issue
Fixes #12413.
What does this implement/fix
So far, to insert gap data (0's representing missing samples), the AnalogSignalGap class with a dummy load() method was defined and used to mirror the neo.AnalogSignalProxy object which loads recorded samples from disk.
As of v13, neo allows onlyAnalogSignal/AnalogSignalProxy types to be appended to Segment.analogsignals attribute. To fix this, we drop the over-engineered AnalogSignalGap class. Instead, we create zero-arrays directly with AnalogSignal and use a simple control statement to distinguish loading real data (via AnalogSignalProxy.load().magnitude) and padding 0's (via AnalogSignal.magnitude) when looping over segments and creating the output data array.

@larsoner See if it looks good? If yes, needs update from main and good on my end.


Index the wallet's UTxO set by asset type

Issue Number
ADP-633
Overview
This PR:

Adds a UTxOIndex type, which keeps track of which UTxO entries contain which assets.
Modifies the Wallet type to include a UTxOIndex instead of a simple UTxO set:

 data Wallet s = Wallet
-    { utxo :: UTxO
+    { utxo :: UTxOIndex
     , currentTip :: BlockHeader
     , getState :: s
     }

Context
The wallet currently uses the Random-Improve algorithm to generate the majority of its coin selections. This algorithm works by choosing entries at random from the wallet's entire UTxO set until there are enough entries to cover the ada value of each desired output, and then attempts to improve upon each selection until its total value is around twice the value of each output (so as to generate useful change outputs that match the wallet owner's typical spending pattern).
However, in a multi-asset context where we're attempting to cover some amount of a given token, it's imperative that we have a way to efficiently search a subset of a UTxO for entries containing a particular asset without having to traverse the entire set.

(Please do not merge this one before today's release cut)

(Please do not merge this one before today's release cut)

bors merge

bors merge

bors merge

bors merge

bors merge

bors merge

bors merge

bors merge


Update FAQs Page

Hi, you need to update the FAQ's page; https://thenewboston.com/faq based on the following design.
Here is the Figma link:
https://www.figma.com/file/C22D4Et2I0AIE0n8QwBDMm/FAQ?node-id=152%3A3508
Here is the FAQ content link:
https://docs.google.com/document/d/12DzwJnge7LaVabgtnkIjM9fpuVp1fNS4KdVsndiU4uc/edit?usp=sharing
If you have any query in this design, kindly let me know asap. Thank you!
Note: You can add this task in whichever sprint you want to.

Here is the Figma link:
https://www.figma.com/file/C22D4Et2I0AIE0n8QwBDMm/FAQ?node-id=152%3A3508

need permission to open link

@ThaDeveloper when you are done with this issue: https://github.com/thenewboston-developers/Website-API/issues/115 kindly let me know and let the frontend guys know as well by commenting on this issue as well. Thank you

@ma2b0043 This is now on dev


Does this SDK recognize development mode?

Does this SDK recognize development mode and separate related events? Otherwise, is there anything I should do to separate development mode and release mode?

Sorry, this isn't a use case that's important to my business so I haven't explored it. You'll have to test, I'm curious to hear results if you report back.


add to item page: link to map, link to timeline

as seen on CB-CDM

done f6214258a7d6782faa2db856f26da9b51b5c6e99


How address groups inside a role/ across vendors or countries

The samples don't show us how to address groups, for e.g. we have vendor groups that login. When they are inside our firewall we use Active Directory Groups, however inside Identity - grouping similar roles across regions to segment permissions is very hard.
In many cases a manager is a role, however they are segmented either by country or Vendor - so, the point is, while they may have the same role we need to lock their access to their group, i.e. managers in the same country or managers in the same vendor company,
This issue has not been addressed for a while, please share a solution to this - can be used in a web solution

Hello! Thanks for contacting us. This sounds like a general question about using ASP.NET Core. While we try to look at and respond to all issues, for questions like this we recommend posting to a community support forum or chat:

Stack Overflow (tag: asp.net-core
ASP.NET Core Slack Team
ASP.NET Gitter


bootstrap script should respond better to github errors

Currently if there is an error when trying to create a new github repository the bootstrap script just terminates (although it does at least say something went wrong with the git command).
This is particularly annoying if you happen to type your github password incorrectly.
Maybe it could respond better by looping and trying again?

This has been addressed in 09c540676543c095a910f1746605b3cab80e4ba7 and 3964a7d5b2d87429a03215bcfb5ec020931481f7


⚠ Bofrost.at has degraded performance

In dabfd36, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2407 ms

Resolved: Bofrost.at performance has improved in e20d0c1 after 15 minutes.


Refactor item page navbar

Blocked by #830.

Have rebased this but I'm not entirely sure I remember the point I was getting at when spending whatever Sunday in August I did on this -- will revisit at some point maybe


PointRCNN (and also the pretrained model) doesn't perform properly on the KITTI dataset

Hi,
I installed 1.0.0.rc2 and run the dataset generation for the KITTI dataset.
I evaluated the provided pre-trained model for PointRCNN over the KITTI dataset and it doesn't show the performance reported on the paper.
I assume it might be related to the refactoring of coordinate systems.

It should not be related to the refactoring of coordinate systems because PointRCNN is merged after the release of 1.0.0rc.
"couldn't get the reported performance" with training from scratch means similar performance with the one shown above? We can first try to reproduce your problem with the given pretrained model.

Yes. Even if I train the default config file over 8 GPUs, the reported mAP is similar to what I reported above. I tried it with the dataset generated with different rc0, rc1, and 0.18.1. I was not able to get the reported mAP? You're right it may not be related to the coordinate systems.

Hi, I have to fix it in https://github.com/open-mmlab/mmdetection3d/pull/1874. And I will update PointRCNN checkpoints in the model zoo later.


[OPENJDK-2903] [jlink] Ensure created objects are uniquely named

Hi,
This PR fixes OPENJDK-2903, adding a mandatory template parameter (APPNAME) that ensures all created resources are uniquely named.
https://issues.redhat.com/browse/OPENJDK-2903

@Josh-Matsuoka please add reference to APPNAME in templates/jlink/README.md too!


Integration Test - Release - 2022/04/25 03:41:55

Auto create for integration test. IssueCommandFlow

#pri1

Just type '#pri3'

#pri2

#pri3

#assign PRMergerTest1

#assign NonAllowedUser

The following users don't have valid GitHub IDs, or are not collaborators on this repo: nonalloweduser

#reassign PRMergerTest2

#reassign NonAllowedUser

The following users don't have valid GitHub IDs, or are not collaborators on this repo: nonalloweduser

GitHub user PRMergerTest1 has been removed from the MicrosoftDocs organization, so they were automatically removed as an assignee.

#please-close

#unassign PRMergerTest1

#unassign NonAllowedUser


Query array contains a certain value

How would I query the equivalent of this SQL?
select * from blog
where 'tech' = any(tags)

Where tags is text[]

You probably know this already, but just in case, one can always fall back to raw SQL which is still perfectly readable:
await db.blog.whereSql`${tag} = any(tags)`

Exactly what I currently do, just wanted to know if the orm supports it.

Added array type operators:

has, hasEvery, hasSome is based on what Prisma has;
containedIn to cover arrayContained from Drizzle;
length: you can filter for empty arrays with length: 0, and you can use numeric comparisons such as length: { gt: 3, lt: 5 }.


Replace skip/take with offset/limit which is relevant for sql based query.

This PR replaces the skip/take with offset/limit in mostly all the typeorm queries. So, basically skip/take is typeorm based and was not reflected back in the converted SQL query and we were not getting 50 user set for log query, which gave the 64k user limit issue #1286. This PR should resolve that as well. I tested the experiment list skip/take query after the update, but seems to not load the next set of experiments in paginated query at few times. Not sure why, might need to dig deeper. So, I have kept that skip/take for now in this query.
@bcb37 Meanwhile you can test the 64k issue by replicating it in your K6 test suite.

@bcb37 Thanks a lot for updating those test cases.

@danoswaltCL We can merge this now.


Added an xxLog function for each js module

Replaced Logger.log calls by xxLog calls
Resolves legalese/legalese.github.io#99

This has been rebased on the xmLog branch - Please accept https://github.com/legalese/legalese-google-app/pull/12 before this pull request.


unexpected error decoding rpc.Status when 401

Description of the bug
When the OTLP HTTP exporter receives 401 unauthorized from the collector:
E, [2022-07-26T16:48:17.742186 #13635] ERROR -- : OpenTelemetry error: unexpected error decoding rpc.Status in OTLP::Exporter#log_status - Error occurred during parsing

The problem seem to be caused by a feature introduced in  https://github.com/open-telemetry/opentelemetry-ruby/pull/1152
I don't understand why is Google::Rpc::Status being used to parse the response in that log method. There's only the HTTP protocol exporter, not the gRPC one?

The OTLP/HTTP part of the spec unambiguously states

Response body for all HTTP 4xx and HTTP 5xx responses MUST be a Protobuf-encoded Status message that describes the problem.

Originally, this was not implemented correctly in the OTLP collector. This was fixed in https://github.com/open-telemetry/opentelemetry-collector/pull/1788. Are you using an old version of the collector, or a 3rd-party collector?


Extension doesn't work in Linux/WSL

Hi!
I try to use this extension.  So far my observation is - it's not working from WSL/Linux. It's only working from Windows host OS.
There are no logs in the output showing the issue.
Could you please check that?
Thank you!

Hi,
You are right, but it's not an issue. BAS remote only works on the host machine (Windows/Mac/Linux), but will not work from a remote machine like WSL, remote container, etc;


rework nft logic in cart to happen in the reducer

[!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

#4313  👈
#4310
#4309
#4308
#4307
main

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

Join @wraeth-eth and the rest of your teammates on  Graphite


Delay config of listen options until server starts

Currently ListenOptions are configured immediately when Listen is called. We should delay this until we have determined to use the ListenOptions or IServerAddressesFeature since we may be throwing away the configured endpoints in https://github.com/aspnet/KestrelHttpServer/blob/99278c469161370b1015c67d18ae9388591a49af/src/Microsoft.AspNetCore.Server.Kestrel.Core/KestrelServer.cs#L119. This could potentially help when configuring HTTPS for Kestrel, see https://github.com/aspnet/KestrelHttpServer/issues/1290.
cc @glennc @danroth27 @muratg

Scenario: Kestrel is configured for https with a cert file for development, but then the app is published to IIS and that cert is not needed/appropriate. Even running the app in IIS Express would fail today if the self-host cert was not present, which is unnecessary.

Fixed by https://github.com/aspnet/KestrelHttpServer/pull/2003/files#diff-319ef184f2f31b202bc2f4566a710afe ?

Yes


snapweb: allow changing the client name

Love the Snapweb!
But would be awesome to have the
SOUND_DEVICE_NAME
And not an id(Bluetooth?)
But still thx for the webinterface :)

Hi,
What you see in the interface is the name of snapcast client. This name (according to recent push) should be
the device ID. Snapweb allow you to alias the name . In the interface select the edit icon and choose your name
and group.
This name should be persisted so you have to it only once.
hope this helps
EJ

Oh didn't see that - it helps - thx :)
But using the *_DEVICE_NAME would be... awesome ;)

Hi,
Note the name shown in the web interface is from the ID of the snapcast client which have no knowledge of *_DEVICE_NAME
EJ

Hi as @ejannett explained this is outside of our control as there is currently no way of specifying the snapclient name. There is a recently opened issue on SnapCast's repo (https://github.com/badaix/snapcast/issues/759) so maybe it'll gain traction and be implemented soon. Once that's done we can add it to balenaSound.

Default name is the hostname and you can change the name using the JSON RPC Api. Regarding the command line: Pull requests are welcome.

I'll look into updating the client. Love snapcast, glad to be able to try and give back


calamares: 3.2.1 -> 3.2.2

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools. This update was made based on information from https://repology.org/metapackage/calamares/versions.
meta.description for calamares is: '"Distribution-independent installer framework"'.

Checks done (click to expand)

built on NixOS
Warning: no invocation of /nix/store/hyfjysykjny1jn7x5z9i33w1ai2nl8xp-calamares-3.2.2/bin/calamares had a zero exit code or showed the expected version
0 of 1 passed binary check by having a zero exit code.
0 of 1 passed binary check by having the new version present in output.
found 3.2.2 with grep in /nix/store/hyfjysykjny1jn7x5z9i33w1ai2nl8xp-calamares-3.2.2
directory tree listing: https://gist.github.com/282dcb1aa17276e50e2d4a9487b816cc
du listing: https://gist.github.com/5f6b514990caee53199d3371e0b4ada6

Instructions to test this update (click to expand)

One-time optional setup to skip building using Cachix:

Install cachix from https://cachix.org/
Use r-ryantm's cache:cachix use r-ryantm

Test this update by entering a nix shell, seeing what is inside the
result, and if applicable, running some binaries:
nix-shell --pure -I nixpkgs=https://github.com/r-ryantm/nixpkgs/archive/ec0f3bc0fb3444f08c17daca8e66c2fc01ae2232.tar.gz -p calamares
ls -la /nix/store/hyfjysykjny1jn7x5z9i33w1ai2nl8xp-calamares-3.2.2

cc @manveru for testing.

Success on x86_64-linux (full log)
Attempted: calamares
Partial log (click to expand)
shrinking /nix/store/hyfjysykjny1jn7x5z9i33w1ai2nl8xp-calamares-3.2.2/lib/calamares/modules/finished/libcalamares_viewmodule_finished.so
shrinking /nix/store/hyfjysykjny1jn7x5z9i33w1ai2nl8xp-calamares-3.2.2/lib/calamares/modules/partition/libcalamares_viewmodule_partition.so
shrinking /nix/store/hyfjysykjny1jn7x5z9i33w1ai2nl8xp-calamares-3.2.2/bin/calamares
gzipping man pages under /nix/store/hyfjysykjny1jn7x5z9i33w1ai2nl8xp-calamares-3.2.2/share/man/
strip is /nix/store/h0lbngpv6ln56hjj59i6l77vxq25flbz-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/hyfjysykjny1jn7x5z9i33w1ai2nl8xp-calamares-3.2.2/lib  /nix/store/hyfjysykjny1jn7x5z9i33w1ai2nl8xp-calamares-3.2.2/bin
patching script interpreter paths in /nix/store/hyfjysykjny1jn7x5z9i33w1ai2nl8xp-calamares-3.2.2
checking for references to /build in /nix/store/hyfjysykjny1jn7x5z9i33w1ai2nl8xp-calamares-3.2.2...
postPatchMkspecs
/nix/store/hyfjysykjny1jn7x5z9i33w1ai2nl8xp-calamares-3.2.2

Success on aarch64-linux (full log)
Attempted: calamares
Partial log (click to expand)
shrinking /nix/store/8xzw1brn7ssazv4npb1q0fc7hqyib5vp-calamares-3.2.2/lib/calamares/modules/dracutlukscfg/libcalamares_job_dracutlukscfg.so
shrinking /nix/store/8xzw1brn7ssazv4npb1q0fc7hqyib5vp-calamares-3.2.2/lib/calamares/modules/contextualprocess/libcalamares_job_contextualprocess.so
shrinking /nix/store/8xzw1brn7ssazv4npb1q0fc7hqyib5vp-calamares-3.2.2/lib/libcalamares.so.3.2.2
gzipping man pages under /nix/store/8xzw1brn7ssazv4npb1q0fc7hqyib5vp-calamares-3.2.2/share/man/
strip is /nix/store/y4ymnvgxygpq05h03kyzbj572zmh6zla-binutils-2.30/bin/strip
stripping (with command strip and flags -S) in /nix/store/8xzw1brn7ssazv4npb1q0fc7hqyib5vp-calamares-3.2.2/lib  /nix/store/8xzw1brn7ssazv4npb1q0fc7hqyib5vp-calamares-3.2.2/bin
patching script interpreter paths in /nix/store/8xzw1brn7ssazv4npb1q0fc7hqyib5vp-calamares-3.2.2
checking for references to /build in /nix/store/8xzw1brn7ssazv4npb1q0fc7hqyib5vp-calamares-3.2.2...
postPatchMkspecs
/nix/store/8xzw1brn7ssazv4npb1q0fc7hqyib5vp-calamares-3.2.2

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: calamares
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.


How can I generate a list of classes?

I'm using Typedoc in a Typescript project where there's a large number of source files in a hierarchy, with each file having one main class (along perhaps one or two supporting classes). Unfortunately, in the resulting HTML tree the only way I can find to get to a class is through the "Globals" list of "modules" (each being one file), then choose a file, then choose the class. I can't find any way of seeing just a pure list of classes. (I've tried both the "default" and "minimal" themes.)
Am I missing something? Generated a list of classes for an object-oriented language seems like such a basic need that I'm surprised it doesn't happen by default.
Thanks,
Dan

And, I figured it out. The default "mode" value is "modules"; by changing this to "file" the index page generated a list of all the classes.
(I'll close this, but consider it as a request for better documentation.)

Is there any way to group them? I saw @categories notation in the docs by it doesn't seem to group anything


[RFE] Option to hide tags when viewing tiddlers

Request for Enhancement
What
I've come to rely on your shields and I don't necessarily need to see aaaallll the many tags I use for organizing and searching and linking things (like projects, etc), just those that are at a glance relevant when reading or working in the wiki.  As such I make shields for the most important ones to highlight them anyway.
This would work well with autogenerating shields https://github.com/NicolasPetton/Shields/issues/1
Examples
Currently, you see the user-defined tags and those that are used programmatically:

Would be good if it just hid the tags to reveal them on mouseover to create a cleaner UI

Even if there is a space left for them and the mouseover, it's still cleaner (some tiddlers are starting to build up a lot of tags that I just don't need to have visible...

I'm not planning to add that feature to Notebook, but you could add the following tiddler to hide tags, revealing them on mouseover:
tags: $:/tags/Stylesheet
title: hidetags.css
type: text/css

.tc-tags-wrapper {
  opacity: 0;
  transition: opacity .15s ease-in;
}
.tc-tags-wrapper:hover {
  opacity: 1;
}

Thank you!  Works like a charm :)


Idea: generate disallowX rule for each node type

Just an idea, not exactly sure how it would work.
We do have http://jscs.info/rule/disallowNodeTypes.html which covers it already. Or maybe it just needs better docs.

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/27077387-idea-generate-disallowx-rule-for-each-node-type?utm_campaign=plugin&utm_content=tracker%2F281640&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F281640&utm_medium=issues&utm_source=github).

Although thinking about it now it's probably not worth the effort haha.. unless someone else wants to implement this?


spi: spi_intel: Fix missing 'break' in case

Coverity scan found issue with a missing 'break' statement.  Fix
push_data by adding the break after handling the 1 byte case.
Coverity CID: 190978
Fixes #13842
Signed-off-by: Kumar Gala<EMAIL_ADDRESS>

Codecov Report

Merging #13899 into master will decrease coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #13899      +/-   ##
==========================================
- Coverage   52.26%   52.25%   -0.01%
==========================================
  Files         307      307
  Lines       45425    45425
  Branches    10506    10506
==========================================
- Hits        23740    23739       -1
  Misses      16895    16895
- Partials     4790     4791       +1

Impacted Files
Coverage Δ

arch/posix/core/posix_core.c
90.9% <0%> (-1.02%)
:arrow_down:

Continue to review full report at Codecov.

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

Dup of https://github.com/zephyrproject-rtos/zephyr/pull/13806 (but this one has better metadata as of now).


lollipopPlot2 error if there are not any mutation

Hi,
First of all congratulations because maftools is amazing. I could see that in lollipopPlot2 when you are comparing two different maf, there is an error if one of the maf has no mutations in the gene selected. In some cases, if you are comparing two maf files is because you would like show differences in the mutational phenotype between these two maf , showing that in one group there are mutations and are concentrated in the same domain and in the other group there are not any mutations. I recommend you to eliminate that error and generate only a warning.

Good point indeed. Thank you for the suggestion. I will fix it soon.

Hello,
Thanks again. I have fixed the issue. Could you please install again from GitHub and let me know if it works for you?
BiocManager::install("PoisonAlien/maftools")

Hi,
Yes it works perfectly thanks a lot!. Only a little thing, if there are not any mutations in a gene in both maf the error message is  'At least one of the maf should contain mutations'. I would print also the gene name, it would be useful when you call the function in a loop with a list of interesting genes in which you don't know the mutation status in the groups before, to eliminate it from the list.

It prints the gene name prior to plotting. Thanks for the suggestions. I am closing the issue now, please feel free to reopen if necessary.


Support for electron 4.0.0-beta

Hi,
Should electron-rebuild support the new 4.0.0-beta release of electron? I tried building my native module but am getting lots of unresolved symbol errors
node -v v10.12.0
node-gyp -v v3.8.0
electron --version v4.0.0-beta.2
electron-rebuild.cmd --version Electron Rebuild Version: 1.8.2

Errors:
NativeExtension.obj : error LNK2001: unresolved external symbol __imp_uv_mutex_unlock
NativeExtension.obj : error LNK2001: unresolved external symbol __imp_uv_queue_work
NativeExtension.obj : error LNK2001: unresolved external symbol __imp_uv_mutex_lock
NativeExtension.obj : error LNK2001: unresolved external symbol __imp_uv_close
NativeExtension.obj : error LNK2001: unresolved external symbol __imp_uv_mutex_destroy
NativeExtension.obj : error LNK2001: unresolved external symbol __imp_uv_mutex_init
NativeExtension.obj : error LNK2001: unresolved external symbol __imp_uv_async_send
NativeExtension.obj : error LNK2001: unresolved external symbol __imp_uv_async_init

Code is here if helps.
Thank you!

This is fixed on newer versions of the 4.0 beta

It looks like 4.0.0-beta.2 is the latest available beta (from electronjs.org ). Do you mean I should use the 4.0.0 nightlies instead?

The website can take up to 24 hours to update, there is a beta.3 release


Prepaid CC

HI, can the very small note about prepaid Credit Cards not being supported by highlighted please as a note at the top of the document?
"Azure doesn't support virtual or prepaid cards."
It is very small far down, and is not easily seen. Many customers try to use prepaid CCs for their Azure environments and then find errors. Please call out in a highlighted NOTE Azure does not support prepaid CCs.
Thank you
Em
[Enter feedback here]

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

ID: 1cf29e05-1e64-cbaf-c774-79d7fb4acf5b
Version Independent ID: 523e8db2-6428-6a51-b536-c3d531560aee
Content: Change your credit card for Azure
Content Source: articles/cost-management-billing/manage/change-credit-card.md
Service: cost-management-billing
Sub-service: billing
GitHub Login: @bandersmsft
Microsoft Alias: banders

@emilylait Thanks for your comment! We will review and provide an update as appropriate.

@emilylait Sure, I'll make the update as soon as I can.

https://github.com/MicrosoftDocs/azure-docs-pr/pull/211908

@emilylait I've updated the article as you requested. The article will get refreshed at 3:30 PM today PDT.

#please-close

Thank you 😊!!! Looks great.
Em
Emily Lait (she/her)
Account Technology Strategist
Higher Education, Microsoft
Level 28, 400 George Street
Brisbane QLD 4000 Australia
Phone:  +61 7 3291 6417
Mobile:  +61 408 253 049
<EMAIL_ADDRESS>Visit Microsoft Higher Educationhttps://www.microsoft.com/en-au/education/higher-education
@.***
We work flexibly at Microsoft. If you receive an email from me outside normal business hours, I am sending it at a time
that is convenient for me. I’m not expecting you to read or reply until your normal business hours.
Our mission is to empower every person and every organization on the planet to achieve more.
From: Bill Anderson @.>
Sent: Wednesday, September 21, 2022 4:45 AM
To: MicrosoftDocs/azure-docs @.>
Cc: Emily Lait @.>; Mention @.>
Subject: Re: [MicrosoftDocs/azure-docs] Prepaid CC (Issue #98548)
@emilylaithttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Femilylait&data=05|01|Emily.Lait%40microsoft.com|48907e7889344ea7d91b08da9b383ee2|72f988bf86f141af91ab2d7cd011db47|1|0|637992963160169018|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=joh88AXJrmS7vYOC2Nqd97%2Bx3D1Lu4dfPoJiOx%2FDgUE%3D&reserved=0 I've updated the article as you requested. The article will get refreshed at 3:30 PM today PDT.
—
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F98548%23issuecomment-1252768802&data=05|01|Emily.Lait%40microsoft.com|48907e7889344ea7d91b08da9b383ee2|72f988bf86f141af91ab2d7cd011db47|1|0|637992963160169018|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=MYoKGuJ2JZGDqKG2BkySpdqQfQVwxRU%2B0z2hvJICZYU%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAOQLEDSYB2GMK6J2BDAADETV7IA3LANCNFSM6AAAAAAQN3XKB4&data=05|01|Emily.Lait%40microsoft.com|48907e7889344ea7d91b08da9b383ee2|72f988bf86f141af91ab2d7cd011db47|1|0|637992963160169018|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=XZmk0fKvx7sc%2FxX9O%2FmP8ZgJ0bwnP%2FGX9QtyPMvqcEc%3D&reserved=0.
You are receiving this because you were mentioned.Message ID<EMAIL_ADDRESS>


Doesn't seem to install

At least on OSX 10.6.8 with rvm 1.9.2 - any version of ruby (well, 1.8.7-p352 and various flavours of 1.9.x)
Both 'gem install jaccard' and putting gem 'jaccard' (with or without version) in Gemfile and running bundle install results in an empty gem being installed (i.e<EMAIL_ADDRESS>gets created, but there's nothing in it).

He, nobody noticed before today. Thanks for this. I've updated the gemspec. Fix was implemented in 9c019c0b03844568a303ef9125b09a88709185d9


activerecord: Add #arel_table to AR::Relation

The ActiveRecord::Relation provides methods defined at AR::Base via delegation (see ActiveRecord::Delegation).
Therefore AR::Relation also provides #arel_table.  It is often used in the scope definition.

/merge


Fix workspace icon border radius is different than the overlay

Details

The workspace icon has the same value of border-radius with the +N overlay, but the actual radius shown is different.
Fixed Issues

$ https://github.com/Expensify/App/issues/23426
PROPOSAL: https://github.com/Expensify/App/issues/23426#issuecomment-1647134312
Tests

Same as QA Steps

[x] Verify that no errors appear in the JS console

Offline tests

Same as QA Steps
QA Steps

Prerequisite: have more than 4 workspaces

Open Settings
Hover/Long press over the Workspaces menu
Verify the border radius of the workspace icon and the overlay (+1, +2, etc.) is the same

You can zoom in to notice it easier

[x] Verify that no errors appear in the JS console

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is approved by marketing by adding the Waiting for Copy label for a copy review on the original GH to get the correct copy.
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If a new component is created I verified that:

[x] A similar component doesn't exist in the codebase
[x] All props are defined accurately and each prop has a /** comment above it */
[x] The file is named correctly
[x] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[x] The only data being stored in the state is data necessary for rendering and nothing else
[x] If we are not using the full Onyx data that we loaded, I've added the proper selector in order to ensure the component only re-renders when the data it is using changes
[x] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[x] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[x] All JSX used for rendering exists in the render method
[x] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(themeColors.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[x] I have checked off every checkbox in the PR author checklist, including those that don't apply to this PR.

Screenshots/Videos

Web

Mobile Web - Chrome

Mobile Web - Safari

Desktop

iOS

Android

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/danieldoglas in version: 1.3.49-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

FYI this caused this regression https://github.com/Expensify/App/issues/24069, I confirmed by reverting it locally

🚀 Deployed to staging by https://github.com/danieldoglas in version: 1.3.50-0 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/Beamanator in version: 1.3.49-3 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/puneetlath in version: 1.3.50-3 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


ACM-13036 Unable to use AWS EC2 instance type with hyphenated name

https://issues.redhat.com/browse/ACM-13036
Backport of https://github.com/stolostron/console/pull/3710

/hold for MCE 2.6.2

/cc @jeswanke

/test unit-tests-sonarcloud

/test unit-tests-sonarcloud

/unhold


Shuttle 1984

About the PR
Setting the alert level to red so the shuttle comes faster is cringe. There's a lot of powergaming behavior surrounding the shuttle right now that's beyond the scope of this to fix, but this is a good first step.
Media

[ ] I have added screenshots/videos to this PR showcasing its changes ingame
[X] This PR does not require an ingame showcase

Changelog

:cl:

tweak: The emergency shuttle will take 10 minutes to arrive regardless of alert level.
tweak: Thanks to subpar engines being installed as a cost-cutting measure, the emergency shuttle now takes 2 minutes to travel back to Centcom.

wouldn't it just make sense to let admins deal with this? Sometimes you need to leave quicker than 10 minutes.

Webedit, requiring screenshots.

Actually, i'ma just close this.
Make a non-webedit PR, continued refusal to test your PRs may result in removal of contribution rights.


Assembler exits with EXIT_FAILURE code if data segments were in source

Fixed in https://github.com/Nikita-bunikido/ROS/commit/90884011e2c18ebe92f0f2dd883d83b8325a944e


nuxt-icon module in parent page makes setup script runs twice when child's route changes.

Environment

Operating System: Windows_NT
Node Version:     v16.19.0
Nuxt Version:     3.1.1
Nitro Version:    2.1.0
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      runtimeConfig, app, modules, imports, css, proxy
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

Reproduction
https://stackblitz.com/edit/github-dgdjfn-hhmxav?file=pages/[username]/index/posts.vue
Describe the bug
Icon in parent page makes setup script runs twice when child route.
Additional context
No response
Logs
No response

Can you please open the issue on nuxt-modules/icon?


Try to guess AWS_REGION from ec2 instance metadata

Context / purpose
When running quickwit on an EC2, we still need to indicate the region AWS_REGION in order to properly use the IAM role associated with the EC2 machine.
This PR, leverage the ec2 instance metadata to get the AWS_REGION automatically when running on EC2.
Description
Describe the proposed changes made in this PR.
How was this PR tested?
We tested this PR on our EC2 instances that are running quickwit. We unset all the environment variables related to AWS and we try to Index data to S3
Checklist
Remove or complete this list if required.

[ X ] tested locally
[ ] added unit tests
[ ] included documentation

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Great. One remark though: what if a user wants to connect to.a bucket in a different region from the ec2 instance? We won't be able to choose the region from the env variable anymore from what I understand.
Currently the default region is like this:
impl Default for Region {
    fn default() -> Region {
        match std::env::var("AWS_DEFAULT_REGION").or_else(|_| std::env::var("AWS_REGION")) {
            Ok(ref v) => Region::from_str(v).unwrap_or(Region::UsEast1),
            Err(_) => match ProfileProvider::region() {
                Ok(Some(region)) => Region::from_str(&region).unwrap_or(Region::UsEast1),
                _ => Region::UsEast1,
            },
        }
    }
}

I think we should do the additional check on ec2 instance region after checking env variable and profile variable but before using the default value UsEast1.

I opened a PR in quickwit with an extra commit that makes the logic more documented and adds Francois request.
https://github.com/quickwit-inc/quickwit/pull/379


--fix fixes indentation even if it's ignored in .groovylintrc.json

--fix command is cool because it autofixes most issues in the codebase, however it also alters whitespaces / indentation even when i have specifically ignored those in the .groovylintrc.json

false alarm

Are you sure ?
I didn't find the time to investigate yet ^^

I tested and it seemed to work, I see some other kinds of white space changes in the diff after running --fix, but i think this particular issue is ok

There are lots of rules adding/removing whitespaces, you can disable or customize them :)
    "formatting.SpaceAfterCatch": {},
    "formatting.SpaceAfterClosingBrace": {},
    "formatting.SpaceAfterComma": {},
    "formatting.SpaceAfterFor": {},
    "formatting.SpaceAfterIf": {},
    "formatting.SpaceAfterOpeningBrace": {},
    "formatting.SpaceAfterSemicolon": {},
    "formatting.SpaceAfterSwitch": {},
    "formatting.SpaceAfterWhile": {},
    "formatting.SpaceAroundClosureArrow": {},
    "formatting.SpaceAroundMapEntryColon": {},
    "formatting.SpaceAroundOperator": {},
    "formatting.SpaceBeforeClosingBrace": {},
    "formatting.SpaceBeforeOpeningBrace": {},
    "formatting.TrailingWhitespace": {},


fix(resetSearchInput) multiple didn't clear after selecting

This will fix #1760
Still I don't have any tests @user378230  any thoughts on this.
Did try some stuff but didn't succeed.

Added some unit test to prevent problems in the future.
Still don't know if this is the best way to validate the resetSearch function, would love to see some feedback :smile:

1,226  test/select.spec.js
630 additions, 596 deletions not shown because the diff is too large. Please use a local Git client to view these changes.

Did you intend to make that many changes? I've not pulled down to review yet just wondering if maybe you broke some whitespacing or something.

@user378230 think I used auto format reverted the commit and added the new tests. These will check if the multiple ui-select is working with resetSearchInput.

Thanks for the fix, this was driving me crazy. @aaronroberson any chance this can be merged and a new release cut?

Merged. Will test to see if the travis hooks will update the version and publish to NPM.

Travis didn't run:
Skipping a deployment with the script provider because this is not a tagged commit
Skipping a deployment with the npm provider because this is not a tagged commit

@aaronroberson, thanks for merging this :smile:. I will update the wiki as the resetsearchinput is default false.

Thanks @timelf123 and @Jefiozie! v0.19.3 is now available :D

thanks @aaronroberson for the quick merge! can you git tag 0.19.3 && git push --tags?

We should really revert the default back to true

@Jefiozie Can you create a PR to revert the default to true?

@aaronroberson Sure no problem!


Update README.md

Bus address should start with sb for Azure Service Bus instead of rabbitmq.

Thanks again!


Fixed issue with circular buffer in _peer/connect events when sending to a peer

Fixes #261

Looks good.


HOTFIX: fix potentially hanging test shouldAddStateStoreToRegexDefine…

…dSource

@guozhangwang @mjsax
Updated the test in question.  I couldn't reproduce locally, but that could be because I'm running on a  Mac (10.10.5).  But at any rate, it's consistent now with other tests that have had no issues.

@mjsax thanks for the review, timeout increased.

Retest this please.

LGTM

Call for final review and merging @ijuma

retest this please.

retest this please.

retest this please

I guess it might be helpful to rebase this to get 3 green builds -- we got a lot of test stabilization fixes recently... WDYT?

@mjsax  agreed, I'll rebase this PR as soon as I have access to my machine.

rebased - let's see if this fixes the tests

retest this please

retest this please.

retest this please.

looks like this PR is still having issues, I'll rebase again and see if that helps

@guozhangwang @mjsax  finally 3 greens

@guozhangwang  comment addressed.

@bbejeck After thinking a bit more about it I think there is still a small risk, because a separate consumer being able to consume the message (meaning that the consumer has discovered the topic and hence being able to fetch from it) does not necessarily mean that the streams' embedded consumer, which is independent, would be able to discover the new topic in time as well. Though adding this one more line does reduce the timing issue by a lot.
So I think a safer fix would be 0) do not close Streams right after producing the data, 1) use TestUtils.waitForCondition to continuously check that stateStoreToSourceTopic = builder.stateStoreNameToSourceTopics(), until say 30 seconds has elapsed and then report the test as failed. 2) call streams.close() in a finally block.
WDYT?

@guozhangwang  yeah I think that is a better approach, thanks for the suggestion.  I'll make the corresponding updates.

@guozhangwang made changes per comments.
Also, looking at the test some more, I removed the part of publishing and consuming a message.  IMHO the test is really about making sure that we associate state stores with the discovered topics matching the regex.  That is once StreamPartitionsAssignor calls builder.updateSubscriptions, the right actions take place.   WDYT?

@bbejeck totally agree!


Cannot install Ubuntu 1804; WslRegisterDistribution failed with error: 0x80070003

Your Windows build number:  (Type ver at a Windows Command Prompt)

Microsoft Windows [Version 10.0.17134.829]

What you're doing and what's happening:  (Copy&paste the full set of specific command-line steps necessary to reproduce the behavior, and their output. Include screen shots if that helps demonstrate the problem.)

Downloaded Ubuntu 1804 via Powershell
Invoke-WebRequest -Uri https://aka.ms/wsl-ubuntu-1804 -OutFile Ubuntu.appx -UseBasicParsing
Installed it
Add-AppxPackage .\Ubuntu.appx
Now where I try launching Ubuntu 18.04 to initialize it I get this output
Installing, this may take a few minutes...
WslRegisterDistribution failed with error: 0x80070003
Error: 0x80070003 The system cannot find the path specified.

What's wrong / what should be happening instead:

Ubuntu should be installed

Are you by chance trying to install the .appx from a drive other than C:?

If possible upgrade Windows version.

@sirredbeard Yes, I was trying to install it on D:
But I also found out that WSL doesn't have access to the GPU so I had to switch to a dual-boot installation :)

@Devilmoon I am glad we worked it out but sorry it couldn't meet your needs. You should also know you can run Ubuntu on Hyper-V with PCI pass-through to your GPU and then ssh in from WSL.
@therealkenc This can be closed.

I got the same error on Windows 11 preview. Kindly help as I need it to work with Docker. I have tried installing it from both the store and command line still same error.

I got the same error on Windows 11 preview. Kindly help as I need it to work with Docker. I have tried installing it from both the store and command line still same error.


Could not open disk image: Permission denied

Full debug log is at https://gist.github.com/manveru/810b83da37478055bd98
I tried chmod 0777, chgown manveru:kvm, but every time i run veewee build, it ends up with:
xi github/jedi4ever/veewee % ls -la iso/ubuntu-13.04-server-amd64.iso
-rw-r--r-- 1 nobody kvm 735051776 Jun 24 10:08 iso/ubuntu-13.04-server-amd64.iso

So I tried locking permissions to manveru:kvm with chattr +i, but even then I get the same error.
I'm out of ideas why this is happening apart from digging through massive strace logs, any help would be appreciated.

For anybody still having this issue, throw the ISO file into /var/lib/libvirt/images and open from there.

@lacksfish I need it in another folder because of disk space

@billtang solution works fine for me in CentOS7,Thanks too much:
weiguo@weiguo:~$ ls -Z /home/weiguo/xp.iso
-rwxrwxrwx. root root system_u:object_r:virt_content_t:s0 /home/weiguo/xp.iso
weiguo@weiguo:~$ sudo -u qemu file /home/weiguo/xp.iso
[sudo] password for weiguo:
/home/weiguo/xp.iso: cannot open (Permission denied)
weiguo@weiguo:~$ sudo chmod ugo+rx /home/weiguo/xp.iso
weiguo@weiguo:~$ sudo -u qemu file /home/weiguo/xp.iso
/home/weiguo/xp.iso: cannot open (Permission denied)
weiguo@weiguo:~$ sudo chmod ugo+rx /home/weiguo
weiguo@weiguo:~$ sudo -u qemu file /home/weiguo/xp.iso
/home/weiguo/xp.iso: # ISO 9660 CD-ROM filesystem data 'GRTMPVOL_CN' (bootable)
weiguo@weiguo:~$

Unable to complete install: 'internal error: qemu unexpectedly closed the monitor: 2019-04-14 09:07:27.349+0000: Domain id=2 is tainted: high-privileges
2019-04-14T09:07:27.508170Z qemu-kvm: -drive file=/home/weiguo/xp.iso,format=raw,if=none,id=drive-ide0-0-1,readonly=on: could not open disk image /home/weiguo/xp.iso: Could not open '/home/weiguo/xp.iso': Permission denied'
Traceback (most recent call last):
File "/usr/share/virt-manager/virtManager/asyncjob.py", line 89, in cb_wrapper
callback(asyncjob, *args, **kwargs)
File "/usr/share/virt-manager/virtManager/create.py", line 2553, in _do_async_install
guest.start_install(meter=meter)
File "/usr/share/virt-manager/virtinst/guest.py", line 498, in start_install
doboot, transient)
File "/usr/share/virt-manager/virtinst/guest.py", line 434, in _create_guest
domain = self.conn.createXML(install_xml or final_xml, 0)
File "/usr/lib64/python2.7/site-packages/libvirt.py", line 3725, in createXML
if ret is None:raise libvirtError('virDomainCreateXML() failed', conn=self)
libvirtError: internal error: qemu unexpectedly closed the monitor: 2019-04-14 09:07:27.349+0000: Domain id=2 is tainted: high-privileges
2019-04-14T09:07:27.508170Z qemu-kvm: -drive file=/home/weiguo/xp.iso,format=raw,if=none,id=drive-ide0-0-1,readonly=on: could not open disk image /home/weiguo/xp.iso: Could not open '/home/weiguo/xp.iso': Permission denied


Problem when run npm run dev on Windows

Describe the bug
I'm trying to use next module federation with Next, but when trying to run the application I get the following error:
Error: Cannot find module 'C:\Users\x\DEV\x-x-nextjs\node_modules\next\dist\compiled\lib\util\makeSerializable'
Require stack:
this is my dependencies
 "dependencies": {
    "@module-federation/nextjs-mf": "^8.1.8",
    "next": "^14.1.0",
    "react": "^18",
    "react-dom": "^18",
    "tailwind-variants": "^0.1.19",
    "webpack": "^5.90.0"
  },

Reproduction
npm run dev on windows
Used Package Manager
npm
System Info
System:
    OS: Windows 11 10.0.22621
    CPU: (8) x64 Intel(R) Core(TM) i5-10310U CPU @ 1.70GHz
    Memory: 5.12 GB / 15.73 GB
  Binaries:
    Node: 18.19.0 - C:\Program Files\nodejs\node.EXE
    npm: 10.2.3 - C:\Program Files\nodejs\npm.CMD
  Browsers:
    Edge: Chromium (121.0.2277.83)
    Internet Explorer: 11.0.22621.1

Validations

[X] Read the docs.
[X] Read the common issues list.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Make sure this is a Module federation issue and not a framework-specific issue.
[X] The provided reproduction is a minimal reproducible example of the bug.

Update to version 8.1.10; for me, It works well.

thanks, this work.

De nada @LukNasc

Sorry, i still have the problem even updating the version

Okay, the problem is that I was using NEXT_PRIVATE_LOCAL_WEBPACK in an .env file, change it to "NEXT_PRIVATE_LOCAL_WEBPACK=true npm run dev" and it worked. My problem now is that it is not recognizing React hooks.

this is problem

No, im using Next, Angular and React.
Host: NextJS v14
Remote 1: React v17
Remote 2: Remote v15

About the useContext problem, I figured it out. I was passing React to Share from the MFE plugin, this caused an error because this package is already in the default share list. So every things work now.


Statistic of the statistics

What problem does this feature solve?
;) The user will be pleased to see how fast the report amount has increased over time.
What does the proposed API look like?
Add a real time upload amount indicator on the home page ;)

Duplicate of https://github.com/penguin-statistics/frontend-v2/issues/308 ;)


✨ Add pathlib.Path interface

The Firecrest class is good, but it lacks a simple, programmatic, way to interact with the file system, consistent with how you would interact with a local file system.
This PR adds the FcPath class, which directly mimics the https://docs.python.org/3/library/pathlib.html interface, to provide such an interface. For example:
In [1]: from firecrest.path import FcPath
In [2]: root = FcPath.from_env_variables("cluster", "/home/service-account-firecrest-sample")
In [3]: path = root / "test.txt"
In [4]: path
Out[4]: FcPath('http://localhost:8000/', 'cluster', '/home/service-account-firecrest-sample/test.txt')
In [5]: path.exists()
Out[5]: False
In [6]: path.write_text("Hallo!")
In [7]: path.is_file()
Out[7]: True
In [8]: path.read_text()
Out[8]: 'Hallo!'
In [9]: path.checksum()
Out[9]: '357a57fe73d6c63bb1923e970f79a924db3a3609eb9797547784a94ee38fc3df'
In [10]: list(root.iterdir())
Out[10]:
[FcPath('http://localhost:8000/', 'cluster', '/home/service-account-firecrest-sample/.bash_logout'),
 FcPath('http://localhost:8000/', 'cluster', '/home/service-account-firecrest-sample/.bash_profile'),
 FcPath('http://localhost:8000/', 'cluster', '/home/service-account-firecrest-sample/.bashrc'),
 FcPath('http://localhost:8000/', 'cluster', '/home/service-account-firecrest-sample/test.txt')]

In order to limit calls to the REST API, I have also added the concept of caching.
For example, when using list_files, the generated paths have the mode cached:
In [11]: for p in root.iterdir():
    ...:     print(p._cache)
    ...:
_Cache(st_mode=None, lst_mode=33188, link_target='')
_Cache(st_mode=None, lst_mode=33188, link_target='')
_Cache(st_mode=None, lst_mode=33188, link_target='')
_Cache(st_mode=None, lst_mode=33204, link_target='')
_Cache(st_mode=None, lst_mode=33204, link_target='')

You have to specifically enable use of the cache, then methods like is_file, is_dir will use this cached value.
For example, this requires only a single API call:
In [12]: root.cache_enabled = True
In [13]: for p in root.iterdir():
    ...:     print(p.is_file())
    ...:
True
True
True
True
True

I have also created https://pypi.org/project/virtual_glob/ which can work with this implementation to provide recursive globbing:
In [14]: from virtual_glob import glob
In [15]: root.cache_enabled = True
In [16]: for p in glob(path, "**/job.sh"):
    ...:     print(p.path)

I haven't added any pytest tests in the PR yet, but will do if you agree that this should go in to this package

Note, one pain point at the moment is https://github.com/eth-cscs/firecrest/issues/171, which means you can't currently get the file mode from the /utilities/stat API, and have to use /utilities/ls to workaround it.
Hopefully this will be fixed soon 🤞

Oh another thing I'd note, is that I added a dependency on typing_extensions for python < 3.11, to use https://peps.python.org/pep-0673/
Its not strictly necessary, but it works well with #42

Note, I made some fixes/improvements to this in https://github.com/aiidateam/aiida-firecrest/blob/main/aiida_firecrest/remote_path.py, so should get round to up-streaming them soon

Hi @chrisjsewell , this look good already! Do you have time to push the fixes/improvements upstream and add maybe some small unittests? The most important part is the fixes 😄 If not, I can try to finalize the PR myself and merge it and we can make new PRs for further improvements.

Heya, feel free to have a look at finalizing it yourself. I will try to have a look at some point, but can't promise I will have time right now 😅

No problem, thanks for your efforts anyway! We will try to merge it this week.