ASIM WebSession schema parser with its sample data and tester files for Palo Alto Cortex Data Lake

Change(s):

Added ASIM WebSession schema parser for Palo Alto Cortex Data Lake based on CEF data

Reason for Change(s):

Initial version ASIM WebSession Parser

Version Updated:

Initial version

Testing Completed:

Yes

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

Yes

Hi @niralishah-crest,
Thank you for raising Pull Request with us! We will review the Pull Request and get back to you by shortly. Meanwhile can you please look into below KQL validation failure. Thanks

@v-atulyadav We have fixed the KQL validation failure.

Thanks @niralishah-crest.

Hi @niralishah-crest,
@vakohl has suggested some changes, please check them out. Thanks

Hi @niralishah-crest,
To follow up appropriately, we would appreciate some indication of the timeframe for these changes. Thanks

Hi @niralishah-crest,
There are some changes that Varun has suggested, please take a look at them. Thanks

Hi @niralishah-crest, There are some changes that Varun has suggested, please take a look at them. Thanks

@v-atulyadav We will update the changes by the end of this week.
Thanks.

Noted @niralishah-crest.Thanks

Hi @niralishah-crest, We look forward to hearing back from you. Thanks

Hi @niralishah-crest,
A few changes were suggested by @vakohl 3 weeks back. Could you please make those changes and update this request accordingly? Thanks

Hi @niralishah-crest,
We're waiting for an update on this. Thanks

Hi @niralishah-crest, please update on above pending comments. Thanks

Hi @niralishah-crest,
Please work on above comment.

We wanted to check on the status of PR #9127. PR is pending for more than 30 days. Please let us know if you need any assistance to review this PR. Per our standard operating procedures if no response is received in the next 7 business days, we will close this PR. Thank you for your cooperation.

Noted @jayeshprajapaticrest. Thanks

@v-atulyadav
Sorry for the inconvenience.
Due to a busy schedule, there is some delay in fixing the suggested changes.
But you can expect the fixes committed in the PR by Next Friday.
The same is informed to the reviewer @vakohl also.
Thanks.

Noted @niralishah-crest. Thanks

@v-atulyadav @vakohl
We have addressed all review comments.
Thanks

Thanks @niralishah-crest.

Hi @niralishah-crest, please update on above pending comment. Thanks

Hi @niralishah-crest,
There are some changes that Varun has suggested, please take a look at them. Thanks

Hi @niralishah-crest,
We're waiting for an update on this. Thanks

Hi @niralishah-crest, please update on above pending comments. Thanks

Hi @niralishah-crest,
There are some changes that Varun has suggested, please take a look at them. Thanks

Hi @niralishah-crest,
We're waiting for an update on this. Also look for branch conflicts Thanks


Roll Flutter from 3e6e996f9eff to 4e468f0122a9 (78 revisions)

Roll Flutter from 3e6e996f9eff to 4e468f0122a9 (78 revisions)
https://github.com/flutter/flutter/compare/3e6e996f9eff...4e468f0122a9
2022-01-18<EMAIL_ADDRESS>Revert "Roll Engine from fab19829949d to 83d99a58097b (31 revisions) (#96673)" (flutter/flutter#96777)
2022-01-15<EMAIL_ADDRESS>Roll Engine from fab19829949d to 83d99a58097b (31 revisions) (flutter/flutter#96673)
2022-01-15<EMAIL_ADDRESS>Skip flaky test: background_isolate_test.dart: Hot restart kills background isolates (flutter/flutter#96678)
2022-01-15<EMAIL_ADDRESS>Roll Plugins from ec80d3439adb to 07f48a6034b4 (1 revision) (flutter/flutter#96670)
2022-01-14<EMAIL_ADDRESS>Explicitly set verbose flag for customer testing. (flutter/flutter#96634)
2022-01-14<EMAIL_ADDRESS>Migrate assemble and integration_test_device to null safety (flutter/flutter#96630)
2022-01-14<EMAIL_ADDRESS>[framework] remove hashcode override for Element (flutter/flutter#96644)
2022-01-14<EMAIL_ADDRESS>feat: added custom padding in PopupMenuButton (flutter/flutter#96657)
2022-01-14<EMAIL_ADDRESS>Move examples from dart:ui ui/lib/text.dart to examples/api (flutter/flutter#96557)
2022-01-14<EMAIL_ADDRESS>Add debug informations on ios_module_test (flutter/flutter#96622)
2022-01-14<EMAIL_ADDRESS>Fix UNUSED_ELEMENT_PARAMETER for unused field formal parameter. (flutter/flutter#96684)
2022-01-14<EMAIL_ADDRESS>update adaptive slider docs (flutter/flutter#96599)
2022-01-14<EMAIL_ADDRESS>Document template CMake files (flutter/flutter#96534)
2022-01-14<EMAIL_ADDRESS>feat: Added more youtube refs to widgets docstrings (flutter/flutter#96484)
2022-01-14<EMAIL_ADDRESS>Disallow copy and cut when text field is obscured. (flutter/flutter#96309)
2022-01-14<EMAIL_ADDRESS>[framework] dont null assert in _debugVerifyIllFatedPopulation (flutter/flutter#96551)
2022-01-14<EMAIL_ADDRESS>Dynamically add integration_tests and screenshots to native iOS test results (flutter/flutter#95704)
2022-01-14<EMAIL_ADDRESS>Added TabBar.splashFactory, TabBarTheme.splashFactory,overlayColor (flutter/flutter#96252)
2022-01-14<EMAIL_ADDRESS>PopupMenu: add themeable mouse cursor v2 (flutter/flutter#96567)
2022-01-14<EMAIL_ADDRESS>Add example for CustomMultiChildLayout (flutter/flutter#96632)
2022-01-14<EMAIL_ADDRESS>Provide Appbar's foregroundColor to package license page titles (flutter/flutter#95685)
2022-01-14<EMAIL_ADDRESS>Revert "Roll Engine from fab19829949d to f121c1fe6843 (10 revisions) (#96631)" (flutter/flutter#96667)
2022-01-14<EMAIL_ADDRESS>Fix a [_ViewportElement] RenderObjectChild update bug (flutter/flutter#96377)
2022-01-14<EMAIL_ADDRESS>Land Set "basic" mouse cursor for disabled buttons, PR #89346 (flutter/flutter#96561)
2022-01-14<EMAIL_ADDRESS>Slider: add themeable mouse cursor v2 (flutter/flutter#96623)
2022-01-14<EMAIL_ADDRESS>Roll Engine from fab19829949d to f121c1fe6843 (10 revisions) (flutter/flutter#96631)
2022-01-14<EMAIL_ADDRESS>Modularize ReorderableListView auto scrolling logic (flutter/flutter#96563)
2022-01-14<EMAIL_ADDRESS>[flutter_tools] increase windows tool_integration_tests shard (flutter/flutter#95917)
2022-01-13<EMAIL_ADDRESS>Roll Plugins from 146b75179665 to ec80d3439adb (1 revision) (flutter/flutter#96621)
2022-01-13<EMAIL_ADDRESS>Roll Engine from e29aad31a30d to fab19829949d (5 revisions) (flutter/flutter#96620)
2022-01-13<EMAIL_ADDRESS>Mac cmd + shift + left/right (flutter/flutter#95948)
2022-01-13<EMAIL_ADDRESS>Ignore in _Location (flutter/flutter#96553)
2022-01-13<EMAIL_ADDRESS>Add Supported devices to the TapGestureRecognizer (flutter/flutter#96560)
2022-01-13<EMAIL_ADDRESS>Revert "Support Scribble Handwriting" (flutter/flutter#96615)
2022-01-13<EMAIL_ADDRESS>Roll Engine from<PHONE_NUMBER>29 to e29aad31a30d (1 revision) (flutter/flutter#96609)
2022-01-13<EMAIL_ADDRESS>Fix wrong documentation URL for adding desktop support to existing app (flutter/flutter#94399)
2022-01-13<EMAIL_ADDRESS>Run Mac_android run_release_test on presubmit (flutter/flutter#96539)
2022-01-13<EMAIL_ADDRESS>fix rsync output utf8decode bug (flutter/flutter#95881)
2022-01-13<EMAIL_ADDRESS>Date Picker year selectors announce as 'buttons' to accessibility framework. (flutter/flutter#96546)
2022-01-13<EMAIL_ADDRESS>Fully support Dart-only mobile and macOS plugins (flutter/flutter#96183)
2022-01-13<EMAIL_ADDRESS>Roll Engine from af8abcccbaa0 to<PHONE_NUMBER>29 (1 revision) (flutter/flutter#96594)
2022-01-13<EMAIL_ADDRESS>Roll Engine from cbf83cd6cc81 to af8abcccbaa0 (1 revision) (flutter/flutter#96587)
2022-01-13<EMAIL_ADDRESS>Roll Engine from c2d79b641ff0 to cbf83cd6cc81 (1 revision) (flutter/flutter#96585)
2022-01-13<EMAIL_ADDRESS>Roll Engine from fba23ea48548 to c2d79b641ff0 (14 revisions) (flutter/flutter#96577)
2022-01-13<EMAIL_ADDRESS>Add debug property for onPointerHover (flutter/flutter#96555)
2022-01-13<EMAIL_ADDRESS>MaterialStateProperty Docs (flutter/flutter#96532)
...

Commit queue failed; closing this roll.


JKBMS add allow to balance status

Describe the solution you'd like
I'd like to see, if balancing is active
Additional context
It would be nice to see, if the balancing on the BMS is switched on or off.

https://github.com/Louisvdw/dbus-serialbattery/pull/468


CMLFS/3-chroot/010-utmps: utmps Segmentation Fault

I have found the reason why utmps fails to build using Clang 12 from /llvmtools/bin. Since I am trying to build the end system without gcc or binutils, this was a pretty big issue for me.
After some short investigation, I found that somehow the static binaries generated by utmps were claiming to have an interpreter:
# file utmps-utmpd
utmps-utmpd: ELF 64-bit LSB executable, x86-64, version 1 (SYSV), statically linked, interpreter /lib/ld-musl-x86_64.so.1, not stripped

And when debugging with lldb:
* thread #1, name = 'utmps-utmpd', stop reason = signal SIGSEGV: invalid address (fault address: 0x0)
    frame #0: 0x00007ffff7fbe950 libc.so`decode_vec(v=0x0000000000000000, a=0x00007fffffffe118, cnt=32) at dynlink.c:198:10

But ldd correctly recognizes the binaries:
$ ldd utmps-utmpd
ldd: utmps-utmpd: Not a valid dynamic program

I found it extremely strange why a statically linked binary would have an interpreter listed by file, and why lldb showed the source of the Segmentation Fault was from within libc.so. So I did a bit more digging.
With CC=x86_64-cmlfs-linux-musl-clang, Clang seems to always pass -dynamic-linker, even for -static (for brevity the linked objects are replaced with ...):
"/llvmtools/bin/ld.lld" --sysroot=/usr --eh-frame-hdr -m elf_x86_64 -static -o utmps-utmpd /usr/lib/crt1.o /usr/lib/crti.o /llvmtools/lib/clang/12.0.0/lib/linux/clang_rt.crtbegin-x86_64.o -L/usr/lib -dynamic-linker /lib/ld-musl-x86_64.so.1 ... -lskarnet --verbose --start-group /llvmtools/lib/clang/12.0.0/lib/linux/libclang_rt.builtins-x86_64.a -l:libunwind.a -lc --end-group /llvmtools/lib/clang/12.0.0/lib/linux/clang_rt.crtend-x86_64.o /usr/lib/crtn.o

With CC=clang, the -dynamic-linker flag is no longer present, however some of the paths are broken, and I must manually pass -L/usr/lib:
"/llvmtools/bin/ld.lld" --sysroot=/llvmtools --eh-frame-hdr -m elf_x86_64 -static -o utmps-utmpd /llvmtools/bin/../lib/crt1.o /llvmtools/bin/../lib/crti.o /llvmtools/lib/clang/12.0.0/lib/linux/clang_rt.crtbegin-x86_64.o -L/usr/lib -L/llvmtools/bin/../lib -L/llvmtools/lib ... -lskarnet --verbose --start-group /llvmtools/lib/clang/12.0.0/lib/linux/libclang_rt.builtins-x86_64.a -l:libunwind.a -lc --end-group /llvmtools/lib/clang/12.0.0/lib/linux/clang_rt.crtend-x86_64.o /llvmtools/bin/../lib/crtn.o

Here's how those are symlinked:
# ls -lah `which x86_64-cmlfs-linux-musl-clang`
lrwxrwxrwx 1 root root 8 Jun 29 07:23 /llvmtools/bin/x86_64-cmlfs-linux-musl-clang -> clang-12
# ls -lah `which clang`
lrwxrwxrwx 1 root root 8 Jun 29 03:44 /llvmtools/bin/clang -> clang-12

Moreover, when building with CC=clang, the build not only succeeds, utmps-utmpd runs without issue:
# ./utmps-utmpd
utmps-utmpd: fatal: unable to get $IPCREMOTEEUID from environment

This seems to suggest to me that the out of tree cross compilation of LLVM for /llvmtools is somehow broken.
The host system is musl Void Linux running under WSL 2 on x86_64. The target system is also x86_64.

On further investigation, it seems this is the issue, from CMLFS/3-chroot/008-Adjust_Toolchain:
echo " --sysroot=/usr -Wl,-dynamic-linker /lib/ld-musl-$(uname -m).so.1" > \
     /llvmtools/bin/$(uname -m)-cmlfs-linux-musl.cfg

Here we are manually instructing clang to always insert -dynamic-linker, but clang can already build working dynamic libraries with the correct Interpreter:
# echo "int main(){}" > t.c
# clang -o t t.c --verbose -Wl,--verbose
"/llvmtools/bin/clang-12" -cc1 -triple x86_64-pc-linux-musl -emit-obj -mrelax-all --mrelax-relocations -disable-free -disable-llvm-verifier -discard-value-names -main-file-name t.c -mrelocation-model pic -pic-level 2 -pic-is-pie -mframe-pointer=all -fmath-errno -fno-rounding-math -mconstructor-aliases -munwind-tables -target-cpu x86-64 -tune-cpu generic -fno-split-dwarf-inlining -debugger-tuning=gdb -v -resource-dir /llvmtools/lib/clang/12.0.0 -isysroot /llvmtools -internal-isystem /llvmtools/usr/local/include -internal-externc-isystem /llvmtools/include -internal-externc-isystem /llvmtools/usr/include -internal-isystem /llvmtools/lib/clang/12.0.0/include -fdebug-compilation-dir /sources/src/utmps-system/temp -ferror-limit 19 -fgnuc-version=4.2.1 -fcolor-diagnostics -faddrsig -o /tmp/t-edcaf0.o -x c t.c
# file t
t: ELF 64-bit LSB pie executable, x86-64, version 1 (SYSV), dynamically linked, interpreter /lib/ld-musl-x86_64.so.1, not stripped

If I remove the -dynamic-linker section from the config file,
echo " --sysroot=/usr" > /llvmtools/bin/$(uname -m)-cmlfs-linux-musl.cfg

And rebuild:
# x86_64-cmlfs-linux-musl-clang -o t t.c --verbose -Wl,--verbose
Configuration file: /llvmtools/bin/x86_64-cmlfs-linux-musl.cfg
"/llvmtools/bin/clang-12" -cc1 -triple x86_64-cmlfs-linux-musl -emit-obj -mrelax-all --mrelax-relocations -disable-free -disable-llvm-verifier -discard-value-names -main-file-name t.c -mrelocation-model pic -pic-level 2 -pic-is-pie -mframe-pointer=all -fmath-errno -fno-rounding-math -mconstructor-aliases -munwind-tables -target-cpu x86-64 -tune-cpu generic -fno-split-dwarf-inlining -debugger-tuning=gdb -v -resource-dir /llvmtools/lib/clang/12.0.0 -isysroot /usr -internal-isystem /usr/usr/local/include -internal-externc-isystem /usr/include -internal-externc-isystem /usr/usr/include -internal-isystem /llvmtools/lib/clang/12.0.0/include -fdebug-compilation-dir /sources/src/utmps-system/temp -ferror-limit 19 -fgnuc-version=4.2.1 -fcolor-diagnostics -faddrsig -o /tmp/t-9e1d1a.o -x c t.c
# file t
t: ELF 64-bit LSB pie executable, x86-64, version 1 (SYSV), dynamically linked, interpreter /lib/ld-musl-x86_64.so.1, not stripped

Once this change has been made, with CC=x86_64-cmlfs-linux-musl-clang:
# file utmps-utmpd
utmps-utmpd: ELF 64-bit LSB executable, x86-64, version 1 (SYSV), statically linked, not stripped
# ./utmps-utmpd
utmps-utmpd: fatal: unable to get $IPCREMOTEEUID from environment

utmps succeeds to build, and works without Segmentation Fault

I created CMLFS/3-chroot/008-Adjust_Toolchain because the llvmtools toolchain produced binaries that looked for an interpreter in /llvmtools instead on the final system root.
LLVM docs don't seem to be much help when trying to build sandboxed toolchains.

I had trouble starting up a new build with LLVM-12.x and moved on to LLVM-15.0.5. Of course, there seems to be drastic changes in LLVM source from 12.x to 15.x which required me to rethink the method to cross compile clang.
After many painful days of failure, I got LLVM-15.0.5 to cross compile. By now, stage 1 clang [installed in llvmtools] links all binaries with -dynamic-linker /lib/ld-musl-$(uname -m).so.1 without a config file. So, stage 1 clang only needs to be configured to change sysroots from /llvmtools to /usr ... just as you have mentioned above

This has been addressed via commit 0bfc298e97abf3a13c1df0cf2bb120b1c8198e8b

commit bc471a8289ff19ff30934c62df0605cd4e7c1673 now recognizes this change

Issue still present.
Stage1 clang outputs compiled code that uses the dynamic linux loader in /llvmtools/lib. After the system libc [musl] is built, then stage1 clang needs to be reconfigured to produce compiled code that uses the dynamic linux loader in /lib
Either I add patchelf to llvmtools to manually patch shared binaries to use the dynamic linux loader in /lib [until the stage2 clang, the final system's compiler is built] OR .... compile stage1 clang twice, one per dynamic loader path.

Stage0 clang now builds the final system's default compiler which by default produces binaries that use the dynamic linker in /lib/ld-musl-$(uname m).so.1 as this change was done with commit 550cf2c3fc9990ea85f367d0215912214f7e0eb2
Now when utmps is built [in chroot], the final system's clang builds it with no cfg. Therefore, static binaries are not built with -dynamic-linker.
This change is present on the mussel-bootsrapped branch. Once I confirm this branch can indeed build a usable CMLFS system,  I will merge it into master and CMLFS will then be built with mussel.


refactor(auth): use Jetpack's Navigation Component

Part of #1217, this PR should introduce the Navigation Library to the auth module.
Changelist:

Turned most of the Activity classes into Fragments. The only one that is still an Activity is the PasswordlessActivity because it has an Intent Filter declared on the AndroidManifest.xml file.
Removed the snippets which were added to the snippets repo in firebase/snippets-android#287.
Deleted the test files, since they rely on Activities.

@rosariopfernandes this is perfect, thank you!


Get correct kubespray version in e2e

What type of PR is this?
/kind failing-test

What this PR does / why we need it:
Get correct kubespray version in e2e
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

/lgtm

/lgtm


bootstrap 4.2.1 toast delay does not 'restart' if the toast is shown again before the timeout ends

The title explains it well.
    clearTimeout(this._timeout)
}

The code above should be added in show() for the toast class at https://github.com/twbs/bootstrap/blob/9e4e94747bd698f4f61d48ed54c9c6d4d199bd32/js/src/toast.js#L80

Windows 10.0.17763 Build 17763
Version 71.0.3578.98 (Official Build) (64-bit)

Don't think there is need for a test case.

Hi @karimcambridge,
I tried to reproduce your issue but I'm not able to 😟
See my CodePen: https://codepen.io/Johann-S/pen/QYqrPw

https://codepen.io/karimcambridge/pen/RvLdGr
Try now, with a larger data-delay (5000).

Hmm ok now I understand 👍
For me in this particular use case, I would call manually show and hide method to avoid that behavior, but you're right we should fix that 👍

Thank you.

FYI for anyone still grappling with this, I worked around it by "disposing" of the toast before showing it again.

Same, but needs to be fixed.

Has this been fixed?


Request for Guidance on Achieving Dynamic Effects and Custom Dataset Integration

Hello, great work! I'd like to ask how to achieve the dynamic effects shown in the video demonstration. I’ve obtained numerous npy files in my log folder, but I’m not sure how to enable real-time changes. Also, if I want to use my own dataset instead of KITTI or nuScenes, which parameters should I modify? Thank you!

Thank you for your interest!
You can refer to main_lidar4d_sim.py and follow the instructions in README to enable simulation, including novel trajectory and configuration. The script will save the visualization results automatically.
If you have point cloud data and the corresponding poses, you could follow the code in the preprocess folder to convert them into rangeview files and the corresponding json files. Then the next step to load the data is basically the same as the current dataloader.


OTel Q&A with Dan Ravenstone - Create copy for social posts

[ ] create copy for promotion on OTel socials
[ ] promote on Slack channels ( #otel-announcements, #otel-sig-end-users, o11y-ambassadors) - schedule posts to go out on October 8th, 15th, and 17th

Related: #61, #63, #64, #65, #66

Hey @avillela , I got the following approved by the Q and A guest, Dan Ravenstone.
This copy has been posted to:
#otel-sig-end-user
#otel-announcements
#o11y-ambassadors

It’s time for another OTel Q and A session!
In this much-anticipated event, we will feature Dan Ravenstone, a staff engineer at Top Hat.
During his 25 years as an observability engineer, he had a front-row seat in observing the evolution of observability and the cultural hurdles that have persisted throughout the years.
Join us to hear about Dan’s experiences and challenges around OTel adoption.
WHEN: Thursday, October 17th, 1 PM - 2 PM EDT (GMT-4)
WHERE: https://zoom.us/j/94248708389?pwd=U4d4ezafHaq01KfOb3dB9jRPlUTeer.1
See you there!

@avillela , I just updated all the event links for the Slack posts that went out today.
These posts have been scheduled to go out on October 8th, 15th, and 17th at 11 AM EST on all three channels mentioned above.

Social copy for Twitter/Bluesky:
It's time for #OTel Q&A, featuring Dan Ravenstone. During his 25 years as an o11yneer, he's had a front-row seat in observing the evolution of o11y and the cultural hurdles that have persisted throughout the years.
WHEN: Oct 17th @ 13:00ET/10:00PT
JOIN: https://community.cncf.io/j/86u3h5bahrjz4/
Social copy for Mastodon/LinkedIn:
It's time for #OTel Q&A, featuring Dan Ravenstone, staff engineer at Top Hat. During his 25 years as an observability engineer, he has had a front-row seat in observing the evolution of observability and the cultural hurdles that have persisted throughout the years.
Join us to hear about Dan’s experiences and challenges around OTel adoption.
WHEN: Oct 17th @ 13:00ET/10:00PT
JOIN: https://community.cncf.io/j/86u3h5bahrjz4/
** Be sure to attach social card!!


Creating animation with all frames.

Please add support for creating animations for texture atlases with all frames without mentioning start and end frames.

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

Thank you for submitting this feature request. We have implemented this and the feature has been pushed to the master branch. It will be part of the next release. If you get time to build and test it for yourself we would appreciate that.


Maybe viewport position bug?

I have change the height of my browser window
http://picload.org/image/cwirral/colorpickerb.png
see blue border left/bottom then I click on input field and the colorPicker is over the input field. Its not much better to set position that colorPicker is above the input field?
Like http://picload.org/image/cwirrli/colorpickerc.png
the red rectangle is the new browser window, and set the box above the input field, when its possible.

I tried to keep colorPicker as small as possible and as open as possible. My intention was to keep the color picker inside the view port and it dynamically moves with the resize of the view port or the scrolling. Positioning like you said is another approach, also very good or even better, but I decided for the other one. To cover too many possibilities would blow up the size of the app.
Color picker is very open in it's possibilities that you can program this yourself: In your renderCallback you have access to toggled which delivers true for 'just opened', false for 'just closed' or something else if none of the two. As this gives you the instance with some information and $elm with the current input field, you can calculate the position of your color picker yourself (You might to turn off animation though and make your own or just drop it...)

I agree. Maybe, you can add this feature for jquer-ui users?: http://jqueryui.com/position/ ;)


Unable to use cloud bank due to merge error

I am receiving a merge error when the cloud service attempts to pull the git repo and launch Jupyter.
As a work around, I'm just using the info as provided on GitHub itself.
This is the output:

$ git fetch
$ git reset --mixed
$ git -c<EMAIL_ADDRESS>-c user.name=nbgitpuller merge -Xours origin/main
fatal: refusing to merge unrelated histories
Traceback (most recent call last):
File "/srv/conda/envs/notebook/lib/python3.11/threading.py", line 1045, in _bootstrap_inner
self.run()
File "/srv/conda/envs/notebook/lib/python3.11/threading.py", line 982, in run
self._target(*self._args, **self._kwargs)
File "/srv/conda/envs/notebook/lib/python3.11/site-packages/nbgitpuller/handlers.py", line 99, in pull
raise e
File "/srv/conda/envs/notebook/lib/python3.11/site-packages/nbgitpuller/handlers.py", line 93, in pull
for line in gp.pull():
File "/srv/conda/envs/notebook/lib/python3.11/site-packages/nbgitpuller/pull.py", line 144, in pull
yield from self.update()
File "/srv/conda/envs/notebook/lib/python3.11/site-packages/nbgitpuller/pull.py", line 344, in update
yield from self.merge()
File "/srv/conda/envs/notebook/lib/python3.11/site-packages/nbgitpuller/pull.py", line 273, in merge
for line in execute_cmd([
File "/srv/conda/envs/notebook/lib/python3.11/site-packages/nbgitpuller/pull.py", line 48, in execute_cmd
raise subprocess.CalledProcessError(ret, cmd)
subprocess.CalledProcessError: Command '['git', '-c'<EMAIL_ADDRESS>'-c', 'user.name=nbgitpuller', 'merge', '-Xours', 'origin/main']' returned non-zero exit status 128.

Thanks for posting this issue @ossiesf. I removed your materials-fa24 folder which had a conflict with the current version. You should be able to open the links from Canvas successfully now. Let me or your instructor know if you are still having an issue. Once, I know this is resolved, I'll close this issue.


Add a favicon

A favicon is always needed as browsers automatically request /favicon.ico.
I'd recommend following the favicon cheatsheet and build several sizes into one file and also offer alternatives for maximum browser support and also best experience on retina displays.
<link rel="shortcut icon" href="/favicon.ico"/>
<link rel="icon" sizes="16x16 32x32 48x48" href="/favicon.ico">
<link rel="apple-touch-icon-precomposed" href="/favicon-152.png">

Done in #21


add missing events

add missing TaskCandidateUserAddedEventEntity, TaskCandidateGroupAddedEventEntity, TaskCandidateUserRemovedEventEntity, TaskCandidateGroupRemovedEventEntity events

This is good but it doesn't cover every event type. For example https://github.com/Activiti/activiti-cloud-query-service/blob/develop/activiti-cloud-services-query/activiti-cloud-services-query-rest/src/main/java/org/activiti/cloud/services/query/events/AbstractProcessEngineEvent.java#L36
What we need for it to not error when there's a missing event type is https://github.com/Activiti/activiti-cloud-query-service/blob/develop/activiti-cloud-services-query/activiti-cloud-services-query-rest/src/main/java/org/activiti/cloud/services/query/events/AbstractProcessEngineEvent.java#L31

Oh, right, I'll add a default event implementation. Thanks.

After discussion with @erdemedeiros we think it's best to merge this and then make the further updates in another PR. We can discuss on standup


AdjustWindowRectExForDpi is missing

[DllImport("user32.dll", SetLastError = true)]
static extern IntPtr AdjustWindowRectExForDpi(
    ref NetCoreEx.Geometry.Rectangle lpRect,
    WindowStyles dwStyle,
    bool bMenu,
    WindowExStyles dwExStyle,
    uint dpi
);

Hi @CiriousJoker - would appreciate a PR for small additions like these. Thanks!

There are still open prs so I didn't bother so far. Maybe I'll make a PR for these


fix(deps): update terraform-module

This PR contains the following updates:

Package
Type
Update
Change

terraform-ibm-modules/event-notifications/ibm (source)
module
minor
1.3.4 -> 1.4.3

terraform-ibm-modules/icd-mysql/ibm (source)
module
patch
1.2.6 -> 1.2.7

terraform-ibm-modules/landing-zone-vpc/ibm (source)
module
patch
7.18.1 -> 7.18.2

terraform-ibm-modules/landing-zone/ibm (source)
module
minor
5.23.0 -> 5.24.3

terraform-ibm-modules/scc/ibm (source)
module
minor
1.4.2 -> 1.6.0

Release Notes

terraform-ibm-modules/terraform-ibm-event-notifications (terraform-ibm-modules/event-notifications/ibm)
v1.4.3
Compare Source
Bug Fixes

deps: update terraform-module (#162) (4adb8fa)

v1.4.2
Compare Source
Bug Fixes

deps: update terraform ibm to latest for the deployable architecture solution (#207) (7f9d725)

v1.4.1
Compare Source
Bug Fixes

add support_details to the DA (#205) (457182e)

v1.4.0
Compare Source
Features

add support to send failed events to a COS bucket (#157) (ed002fa)

v1.3.7
Compare Source
Bug Fixes

deps: update terraform ibm to latest for the deployable architecture solution (#198) (89fdbb6)

v1.3.6
Compare Source
Bug Fixes

add missing depends_on which can cause KMS integration to fail (#196) (f613570)

v1.3.5
Compare Source
Bug Fixes

various variable description and documentation updates (#184) (3c4688c)

terraform-ibm-modules/terraform-ibm-icd-mysql (terraform-ibm-modules/icd-mysql/ibm)
v1.2.7
Compare Source
Bug Fixes

deps: update terraform-module (#76) (4fd79e9)

terraform-ibm-modules/terraform-ibm-landing-zone-vpc (terraform-ibm-modules/landing-zone-vpc/ibm)
v7.18.2
Compare Source
Bug Fixes

vpc data lookup (#798) (fe27e6f)

terraform-ibm-modules/terraform-ibm-landing-zone (terraform-ibm-modules/landing-zone/ibm)
v5.24.3
Compare Source
Bug Fixes

landing zone configuration tool updates (#801) (d1c8359)

v5.24.2
Compare Source
Bug Fixes

deps: update landing-zone-vsi to 3.3.0 and landing-zone-vpc to 7.18.2 (#771) (5978ff9)

v5.24.1
Compare Source
Bug Fixes

update version.tf in roks quickstart to allow tf 1.6 (#805) (c4a7972)

v5.24.0
Compare Source
Features

expose DNS options in the vpc variable of the root level module (#795) (18af306)

terraform-ibm-modules/terraform-ibm-scc (terraform-ibm-modules/scc/ibm)
v1.6.0
Compare Source
Features

add support to create SCC CBR rules (#104)  - Added new input variable cbr_rules  - Added call to cbr module to create 1 or more supplied rules (d834b0a)

v1.5.0
Compare Source
Features

added support to pass the string latest for the profile_version input in the attachment submodule (#98) (369002d)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

:tada: This PR is included in version 1.6.4 :tada:
The release is available on:

GitHub release
v1.6.4

Your semantic-release bot :package::rocket:


CodeGen from PR 12124 in Azure/azure-rest-api-specs - Azure HealthBot

Required changes for Azure HealthBot swagger (#12124)

This checklist is used to make sure that common guidelines for a pull request are followed.

For new extensions:

[ ] My extension description/summary conforms to the Extension Summary Guidelines.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your PR is merged into master branch, a new PR will be created to update src/index.json automatically.
The precondition is to put your code inside this repo and upgrade the version in the PR but do not modify src/index.json.

healthbot

Please fix
/home/vsts/work/1/s/src/healthbot/azext_healthbot/tests/latest/example_steps.py:80:1: W391 blank line at end of file
/home/vsts/work/1/s/src/healthbot/azext_healthbot/tests/latest/test_healthbot_scenario.py:80:5: E303 too many blank lines (2)
/home/vsts/work/1/s/src/healthbot/azext_healthbot/tests/latest/test_healthbot_scenario.py:86:1: W391 blank line at end of file
1     E303 too many blank lines (2)
2     W391 blank line at end of file

Please fix
/home/vsts/work/1/s/src/healthbot/azext_healthbot/tests/latest/example_steps.py:80:1: W391 blank line at end of file
/home/vsts/work/1/s/src/healthbot/azext_healthbot/tests/latest/test_healthbot_scenario.py:80:5: E303 too many blank lines (2)
/home/vsts/work/1/s/src/healthbot/azext_healthbot/tests/latest/test_healthbot_scenario.py:86:1: W391 blank line at end of file
1     E303 too many blank lines (2)
2     W391 blank line at end of file

Thanks @qwordy , I'm unable to push changes to my branch, so I forked the repo and added your suggestions. A new PR with the fixes is available here:
https://github.com/Azure/azure-cli-extensions/pull/2965

Thanks @qwordy , I'm unable to push changes to my branch, so I forked the repo and added your suggestions. A new PR with the fixes is available here:
#2965

Let's close this one.


Help! Content-Length is missing or -1, but transfer encoding is set to chunked

Does this lib work with web servers that don't provide Content-Length, but respond with chunked encoding?
The content-length is either missing or set to -1, but transfer-encoding header comes with value "chunked".
Thanks

You can use the SimpleDownloadBuilder with chunked responses, but you won't be able to resume or multi-part download such a response with this library. My knowledge about chunked transfers isn't so deep so I'm not quite sure if it is possible support resumes as chunked downloads are usually used when the final size isn't known ahead.


TagBot trigger issue

This issue is used to trigger TagBot; feel free to unsubscribe.
If you haven't already, you should update your TagBot.yml to include issue comment triggers.
Please see this post on Discourse for instructions and more details.
If you'd like for me to do this for you, comment TagBot fix on this issue.
I'll open a PR within a few hours, please be patient!

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/74326

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/76390

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/76582

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/77153

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/79455

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/83136

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/85325

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/90800

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/92604

Triggering TagBot for merged registry pull request: https://github.com/JuliaRegistries/General/pull/96704


Dispose TerminalWordLinkDetector listeners

Fixes https://github.com/microsoft/vscode/issues/187082

@Tyriar it seems like the yarn.lock check won't run for some reason, blocking the merge?

Sometimes they get stuck


Adding to leaderboard interactivity and changing submission structure

To review, biggest change here is allowing people to submit a repo through a google form that includes their model params. In run.sh, we iterate through the form to evaluate the agents.  The different Rmarkdown files that generate the leaderboard are in the html/ directory -- currently I have only made submissions for climate and fishing gym so these are the only html's up.
We need to update the github workflow here then we can make more ground from there.

ha, I was too slow, cron job continues to update the files.
Can you again resolve merge conflict as before and re-commit?  Then feel free to merge this so I don't miss the boat again.
I guess in general it is easier on PRs if we don't include changes to automatically generated files in the PR but just wait until the action rebuilds them.  also probably we should turn off the cron-scheduled autobuild and stick with manual triggers while we are in early devel!


bug in bonfire-smallest-common-multiple

Challenge Bonfire: Smallest Common Multiple has an issue.
User Agent is: Mozilla/5.0 (X11; Ubuntu; Linux x86_64; rv:43.0) Gecko/20100101 Firefox/43.0.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:
function smallestCommons(arr) {

    var newArr = [];
    var n = 1;
    var found = false;
    //swap elements if first element is smaller than second
    if (arr[0] > arr[1]) {
        var temp = arr[0];
        arr[0] = arr[1];
        arr[1] = temp;
    }
    //find out all the numbers between the given arguments
    for (var i = arr[0]; i <= arr[1]; i++) {

        newArr.push(i);
    }

    //find out the smallest common multiple
    while (!found) {
        n++;
        for (var j = 0; j < newArr.length; j++) {
            if (n % newArr[j] !== 0) {
                found = false;
                console.log(n + " is " + found);
                break;
            } else {
                found = true;
            }
        }
    }
    return n;
}

smallestCommons([1,5]);

@velmuruganvelayutham thanks for posting this issue!
Please, provide the description of your issue, thanks!

Please review the Guidelines for Contributing, thank you!.

final test case is getting failed even after my solution is correct..

@velmuruganvelayutham Your code is too slow and is getting flagged by our infinite loop detector. You can either try coming up with a more efficient solution or put // noprotect in your code to submit as is (just be prepared for the page to lock for about 10 seconds after you submit your code). Thanks and happy coding!

thanks @ltegman . It works fine..


Update better downloader and audio convertion & summarization

The mp4 files streamed from pytube had to be converted to mp3 format with the correct codec, and a small adjustment was made to the file naming. If the video does not download, a more descriptive error message is provided. At the same time, simple logging messages are written to the screen for the download process.
In the demo in the readme file, the return command instead of print was forgotten and the attribute in the language option wasn't closed correctly.

I'm sorry for committing the text summerization commits on the same pull request.

I'm sorry for committing the text summerization commits on the same pull request.

Thank you for your contribution. I tested the model and the result is very good 🚀


Is it possible to use variable inside url?

I have several server instances in development environment. For example, one instance is http://localhost:3000 and the other is http://<IP_ADDRESS>:7899 and so on.
And I have many api request code in restbook. If I want to switch between servers, I had to change url parts one by one, which is not practical.
Is it possible to use variable inside url? I tried but it seems not working for now.
For example,
let api = http://<IP_ADDRESS>:7899
let api2 = http://localhost:3000
GET @api/users
GET @api2/users

you can use variables with $ in front.


(v2) Metadata isn't included in build

🐛 Bug Report
There is no metadata other than title showing in the build.
Have you read the Contributing Guidelines on issues?
Yes
To Reproduce

Install v2 alpha 61
Add metadata to docusuarus config
Run build

Expected behavior

I expected to see meta tags title, description, <meta og:..> in the source code of each page.
Actual Behavior

Only <title> is present.
Your Environment

Docusaurus version used: v2 alpha 61
Environment name and version (e.g. Chrome 78.0.3904.108, Node.js 10.17.0): Microsoft Edge (Chromium)
Operating system and version (desktop or mobile): Desktop

Reproducible Demo
View the source code of https://answers.equityforhire.com.
Other Tests
I first noticed this issue while attempting to share the URL above on Facebook. No metadata showed. Then I visited https://developers.facebook.com/tools/debug/ to scrape the data and no metadata was fetched.
Here is my docusaurus.config.js:

Then I attempted to add the <Head> component on my home page to force React Helmet as shown below:

Unfortunately, still no metadata.

Hey @martavis , on Docusaurus v2 website it works for me:

2 ways I can help you:

you submit a PR on d2 website so that I can see the issue in the deploy preview (we won't merge it)
have access to your site's source code

closing for now

I finally got a chance to check this out again. I had major issues breaking my builds.
It turns out that this was a user error. I have a higher-order component wrapping my pages to check for a public vs private page and it block the Head component from rendering. Everything works.


Support existing pvc as catalog question

Proposed changes
Support existing pvc as catalog question

Types of changes

New feature (non-breaking change which adds functionality)

Linked Issues
https://github.com/rancher/rancher/issues/17060

Further comments

There is no backend dependency as it is a new type for catalog question. We can merge it.


[WIP] Add passthrough for configuring keychain storage options

This will allow external developers the option to configure how the mnemonic is stored in the ios device keychain.

[x] Add some form of android support that ignores iOS options

see https://github.com/rally-dfs/rly-network-mobile-sdk/pull/97


Tests cleanup

Code organization changes:

Removed indirection when running test transactions
Simplified signature of test utils
Standardized the code formatting for many tests, so it's less distracting

I haven't looked at this PR too deeply: would it work fine with #735? I'm hoping we can merge #735 some time today - would it make sense to rebase this PR after that?

closing in favor of #735


Should be "preFilter" not "PreFilter"

In the Hybrid Search example "vectorFilterMode" field had a value as "PreFilter". This should be "preFilter".

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

ID: 3b5c8e78-fc1a-33f5-3378-a4e2d157f7eb
Version Independent ID: 71fe941a-6d91-85b0-1e34-531c14a5a84c
Content: Quickstart vector search - Azure AI Search
Content Source: articles/search/search-get-started-vector.md
Service: cognitive-search
GitHub Login: @HeidiSteen
Microsoft Alias: heidist

@jixjia
Thanks for your feedback! We will investigate and update as appropriate.

@jixjia, this has been fixed in a recent doc update. Thank you for providing your feedback.
#please-close

Awesome work, thanks team !


Adding GitHub Enterprise as a source provider

I also did a bit of refactoring since the implementation for GitHub, Bitbucket, and GitHub Enterprise were all identical except for one line.

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

Looks good.

@scottboehmer FYI.

@TingluoHuang - Thanks Ting!


[feat] support any code block language

Currently we require code languages manually, which won't work for all projects. Somehow we need to find a way to support dynamic languages.
https://highlightjs.org/ might be an option, however we'd have to ensure that the parsing of the language is carried out on the server (right now, it's on the client).

After some investigation, using highlight.js provides the following:
Pros:

Any language supported (server side parsing)
No client work required, simpler to manage

Cons:

No support for line-numbers or line-highlighting

Closes in #23


build clickhouse报错

Problem description
Directly Compile ClickHouse backend按照这个步骤编译clickhouse

System information
CentOS Linux release 7.8.2003
CMake log
No response

What's the result of under directory cpp-ch and execute command bash -x ../ep/build-clickhouse/src/build_clickhouse.sh?


different library for loading audio files

I read that PyTorch doesn't want to add complex number tensor types.  A lot of audio processing requires fourier transforms, specifically into MEL spectrograms.  It would be nice to use a library that already handles a lot of these transformations in numpy.  The most popular one seems to be librosa, which uses numpy arrays for easy conversion to torch tensors.
Is there any reason this library uses a cffi extension instead of a library such as librosa?
There are actually a lot more reasons to use librosa rather than the current solution, but it'd take too long to list them.  I wouldn't mind making a pull request to switch from the current libsox cffi way of doing things to the librosa/numpy way.

The reason that I started this library over librosa is because @jfsantos gave me feedback that there's no efficient way to load audio into a Tensor/ndarray in the python ecosystem. Almost all currently existing libraries (apparently) invoke a subprocess to ffmpeg or other equivalents and read/write through unix pipes, which is too heavy, when loading lots of small files in particular.
I'm happy to have this library shape up to be what everyone wants it to be.
In fact, I expected this lib to be something that just loads the files, and then folks would use scipy/librosa for actual pre-processing.

That makes a lot of sense.  My point is there seems to be a trade-off between efficiency and usability that is out of whack with this lib.  Using libsox directly makes the library more efficient, but because people get strange errors like #7 and #1 nobody really uses the library.

After I wrote my last comment I realized a lot of my gripes were due to errors that could easily be fixed by adding some checks in the load and save functions. Also, you are right, these functions load audio orders of magnitude faster than librosa.
I'll go ahead and add some error checking to those functions instead of whining about it. :-)


[type:fix]Add DumperOptions for yaml parsing

// Describe your PR here; eg. Fixes #issueNo

Make sure that:

[x] You have read the contribution guidelines.
[x] You submit test cases (unit or integration tests) that back your changes.
[x] Your local test passed mvn clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #2645 (32a970e) into master (f84d224) will decrease coverage by 0.56%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #2645      +/-   ##
============================================
- Coverage     61.20%   60.64%   -0.57%
+ Complexity     4330     4300      -30
============================================
  Files           660      660
  Lines         18981    18981
  Branches       1682     1682
============================================
- Hits          11618    11511     -107
- Misses         6407     6524     +117
+ Partials        956      946      -10

Impacted Files
Coverage Δ

...ache/shenyu/plugin/grpc/cache/GrpcClientCache.java
68.75% <0.00%> (-31.25%)
:arrow_down:

...n/alibaba/dubbo/cache/AlibabaDubboConfigCache.java
43.75% <0.00%> (-31.25%)
:arrow_down:

...gin/apache/dubbo/cache/ApacheDubboConfigCache.java
43.03% <0.00%> (-29.12%)
:arrow_down:

...henyu/plugin/grpc/resolver/ShenyuNameResolver.java
44.68% <0.00%> (-19.15%)
:arrow_down:

...a/org/apache/shenyu/common/utils/VersionUtils.java
67.85% <0.00%> (-10.72%)
:arrow_down:

.../apache/shenyu/admin/model/dto/BatchCommonDTO.java
50.00% <0.00%> (-7.15%)
:arrow_down:

...apache/shenyu/admin/model/dto/AuthPathWarpDTO.java
57.14% <0.00%> (-7.15%)
:arrow_down:

...org/apache/shenyu/admin/model/dto/AuthPathDTO.java
58.82% <0.00%> (-5.89%)
:arrow_down:

...che/shenyu/sync/data/http/HttpSyncDataService.java
68.31% <0.00%> (-4.96%)
:arrow_down:

...in/response/strategy/NettyClientMessageWriter.java
69.56% <0.00%> (-4.35%)
:arrow_down:

... and 23 more

Continue to review full report at Codecov.

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


Support environment variable OTEL_LOG_LEVEL

When the user provides a value to OTEL_LOG_LEVEL, the diag logger should be enabled without need for further code changes.
Original discussion: https://github.com/open-telemetry/opentelemetry-js-api/issues/102

My company is also very interested in being able to use OTEL_LOG_LEVEL.

I'm adding my voice here as someone that would like the logging to work out of the box. Or in the alternative, tell me that that's not the case so I can enable logging instead of setting the env var and wondering why it ain't working.

You can assign this to me if that's ready to implement


langchain[major]: Upgrade langchain to be pydantic 2 compatible

Upgrading the langchain package to be pydantic 2 compatible.
Had to remove some parts of unit tests in parsers that were relying on spying on methods since that fails with pydantic 2. The unit tests don't seem particularly good, so can be re-written at a future date.
Depends on: https://github.com/langchain-ai/langchain/pull/26057
Most of this PR was done using gritql for code mods, followed by some fixes done manually to account for changes made by pydantic

This PR was merged despite a few failing unit tests:

There is non-determinism in this unit test:

https://github.com/langchain-ai/langchain/blob/e19255bb3c7bfcfba492bb5b5b74a45281d8a4f7/libs/langchain/tests/unit_tests/retrievers/document_compressors/test_chain_filter.py#L24-L24

Extended tests still trigger some failing unit tests. We'll need to address those as well prior to release.


Improve username behaviour

Fixed the behaviour of validate_user function so that it now takes into account the numericals in the beginning of paramters passed when calling the function.
Fixes #1538

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

@googlebot I signed it!


Dont work atomic() in fastapi task

_GatheringFuture exception was never retrieved
future: <_GatheringFuture finished exception=RuntimeError('The transaction must run within a task')>
Traceback (most recent call last):
  File "C:\Users\User\Desktop\git\microservice-users\routes\roles.py", line 160, in m
    async with objects.atomic():
  File "C:\Users\User\AppData\Local\Programs\Python\Python312\Lib\site-packages\peewee_async.py", line 1273, in __aenter__
    return (await self._ctx.__aenter__())
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "C:\Users\User\AppData\Local\Programs\Python\Python312\Lib\site-packages\peewee_async.py", line 1202, in __aenter__
    raise RuntimeError("The transaction must run within a task")
RuntimeError: The transaction must run within a task

@router.patch("/roles")
@get_scopes('manage_roles')
async def role_patch(request: Request):

/// other code

	async def m():
		async with objects.atomic():
			for change in data_json:
				role_id = change.get("role_id")
				position = change.get("position")

				if role_id not in jpath.search("*.role_id", roles):
					raise HTTPException(400, "The role you have in JSON was not found")

				if position not in range(1, len(roles)+1):
					raise HTTPException(400, "Invalid positon role")

				role = await get_role(str(role_id))

				role.priority = 7 # value for example

				await objects.update(role)

	task = asyncio.create_task(m())
	asyncio.gather(task)

@kalombos Hi! Btw, the new aio_atomic() is in place, right? Maybe let's release it?

Hi @rudyryk  It is already released as beta and i use the version in my projects. I am going to make a prod-release after some time


start hub project

Pull Request Test Coverage Report for Build<PHONE_NUMBER>2
Details

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

Totals

Change from base Build<PHONE_NUMBER>9:
0.0%

Covered Lines:
571

Relevant Lines:
617

💛  - Coveralls


fix: ingress update event handler not filter by watching namespaces

Type of change:

[x] Bugfix
[ ] New feature provided
[ ] Improve performance
[ ] Backport patches

What this PR does / why we need it:

Fix https://github.com/apache/apisix-ingress-controller/issues/869
Since the ingress update event handler is not filtered according to the watching namespaces, and our e2e uses two ginkgo nodes, it will cause the lb status test case to be interfered with the ingress controller in other test cases. This is why e2e is unstable as described in https://github.com/apache/apisix-ingress-controller/issues/869.
Pre-submission checklist:

[x] Did you explain what problem does this PR solve? Or what new features have been added?
[ ] Have you added corresponding test cases?
[ ] Have you modified the corresponding document?
[x] Is this PR backward compatible? If it is not backward compatible, please discuss on the mailing list first

Thanks!

It's true that I can't reproduce it in my local environment, I didn't consider this reason before.  Let me think of a way to reproduce it


Request to Publish Extension on open-vsx.org

Hello,
Thank you for developing the intellicode extension. I noticed that the extension is available on the Microsoft Marketplace but not on open-vsx.org, which is an open-source alternative that serves the broader community, including VSCodium users.
Would you consider publishing your extension on open-vsx.org as well? It would benefit users who prefer or are required to use open-source software. The process is outlined here: Publishing Extensions.
Thank you for considering this request.
Regards,
Surya

Hello Team,
Any updates?
Thanks & Regards,
Surya


WICKET-6704 JavaSerializer.serialize causes the JVM crash !

Do not check instances of PropertyChangeSupport whether they are Serializable because PropertyChangeSupport#writeObject() adds extra fields which confuse CheckingObjectOutputStream

This PR fixes the problem in WICKET-6704 but there are more classes in Java which do the same as PropertyChangeSupport, e.g. java.security.BasicPermission, java.util.BitSet, and one dozen more. Should we whitelist them too ? Or someone sees how to fix the problem in CheckingObjectOutputStream ?


How to make an interactive call bot using Bot Framework?

Bot Info

SDK Platform: .NET
SDK Version: Latest
Active Channels: Thats the question
Deployment Environment:  Azure App Service

Issue Description
I want to make a calling bot, where the user can call, user's speech can be converted to text, analysed by an NLP and then bot processes this information and prepare a text response and post to user after converting into speech. Am not sure how to accomplish this, skype call project works more like an IVR and does not appear it can handle normal conversation. Please help.

Take a look at Bing Speech API  and LUIS you can accomplish what you are asking about with this services.


Make fork-bomb.js executable using node

That's a bash script, not a javascript program!

Heh, whoops. Thanks for the PR.


[Help] JSBI to BigInt in webpack

Hello;
I am not using babel, but webpack, to compile my TS code into JS. My total app size is gigantic due to how I have to create the entrypoints for my app. I am curious if not having to bundle JSBI would improve my bundle size. Is there a way to include the jsbi-to-bigint package in webpack?

Ah, yes, I should have inspected the output first. Sorry about that.
It looks like out of the 293.91KB from node_modules, 144.36KB is from @js-temporal/polyfill/dist/index.esm.js and 33.2KB is from jsbi/dist/jsbi-umd.js.
So it appears that Temporal + JSBI accounts for ~52% of my dependencies. Is this expected? I don't know how complex the polyfill is, but that seems excessive. Is there any advice for reducing the package size?

I don't know how complex the polyfill is

We do run the polyfill library source through an optimizer (Terser) before shipping it to try and reduce our bundle size, but:

this polyfill does aim to be faithful to the Temporal spec, which itself is large and complicated
even when publishing es modules, I don't believe that a bundler/optimizer is able to tree-shake away unused pieces due to the way the polyfill is structured - see  because many Temporal classes can create each other. I explained that somewhat here: https://github.com/js-temporal/temporal-polyfill/issues/235#issuecomment-1504040023.
One potentially large chunk of code that you aren't using is the non-ISO calendar support. The Temporal spec only mandates that there be calendar support for the ISO calendar, but we also ship implementations for various other calendars. We could potentially do better here.

If you don't need bigint compatibility, I would be interested to see what code-size numbers you get with the Babel plugin configured. That should drop code-size a fair bit both by removing JSBI from the bundle entirely but I'd expect the source attributed to Temporal to maybe drop in size a bit as well as the rewritten version of all the JSBI operations is likely smaller.

One potentially large chunk of code that you aren't using is the non-ISO calendar support.

Actually I am using an extremely small chunk of the library - basically just converting strings and numbers into PlainDates, doing a few comparisons, adding days, calling .toString, that's about it.

I would be interested to see what code-size numbers you get with the Babel plugin configured.

I am currently not using babel at all (only webpack + typescript with ts-loader), so I was trying to see if I could port the jsbi-babel plugin into a native ts-loader transformer, but it's looking to be way more complex than babel (plus I wasn't able to get ts-loader to actually see the files from /node_modules/). I've opened an issue at https://github.com/GoogleChromeLabs/babel-plugin-transform-jsbi-to-bigint/issues/15 but that repo is really low activity so I don't think I'll be able to do that any time soon.
I was playing around with the other polyfill library (https://github.com/fullcalendar/temporal) and they don't ship with JSBI, but instead rely on native BigInt support. With IE now officially deprecated, I don't know if I personally see a reason to ship with a BigInt polyfill since it appears that most major browsers support BigInt already, but I'm certainly not going to tell you how to ship your own library.
With how little of the library I'm using, at this point I'm content to use temporal-polyfill and wait to see if I can get some assistance for the ts-loader transformer. If there's anything else I can try (that won't take up hours and hours) I'd be more than glad to give it a shot.

I set up a barebones webpack project that can use the ts-loader for TS sources and Babel to run the JSBI plugin, and the important part seems to be adding
{
  test: /\.(js)$/,
  use: {
    loader: 'babel-loader',
    options: {
      plugins: ["transform-jsbi-to-bigint"]
    },
  }
}

to the webpack.config.js file. This configures the plugin to run over every JS file in the compilation - this could probably be refined if you wanted by adjusting the test (see https://webpack.js.org/configuration/module/#condition).
I'm not sure how ts-loader fits into this - are you also configuring the loader to look at node_modules JS files? The TS loader is designed to load TS sources and compile them with the typescript compiler, and it might be that it "can't see" the files in node_modules because those aren't TS sources anymore - they've already been converted to JS and typings. Note that we do publish our TS sources in our package but those would not be useful to pass to ts-loader as those aren't the sources you would actually be building against.
If you would rather not run Babel at all during development it should be relatively easy to configure that in the webpack config, only running this transform in production builds where the code-size and runtime benefits would be worth the build setup cost.
We are also considering ways to ship a bundle that is more modern that might help here, which would have the JSBI plugin run over the project sources at build time, potentially via a separate export path. See https://github.com/js-temporal/temporal-polyfill/issues/155. If configuring this sort of a dual-package system is something you have expertise in, we would appreciate your help over there.

Wow, awesome, thank you! I really don't mean for you to "do it for me", but I absolutely appreciate the assistance! I'll put my webpack.config.js down below.

are you also configuring the loader to look at node_modules JS files?

Not explicitly? See in the wbpack config, ts-loader is only loading .ts files, but we do tell webpack to resolve .js files too. I'm no webpack expert, but our output does build and run, so I'm not sure if that's a satisfactory answer for you.

If you would rather not run Babel at all during development...

Ideally we'd have our local and prod builds be as similar as possible, but for sure we can exclude this if we need to.

If configuring this sort of a dual-package system is something you have expertise in, we would appreciate your help over there.

Unfortunately I do not. I see there's a #167 already; something like that would be perfect for my use-case.
As for my webpack setup, here's the weird part. It doesn't matter include babel or not, the entire temporal polyfill gets included twice, but the JSBI dependency gets put into the 'polyfill' chunk... Again, not a webpack expert, but that seems very strange. Maybe we don't have it set up correctly, idk.
When I do include the babel plugin, I can see that the this library gets split into its constituent pieces, probably due to the modifications to the source files. It looks like I save about 3KB in the polyfill itself, plus the ~33KB from excluding JSBI, so that's awesome.

export default async (env, options) => {
  const devMode = options.mode === "development";

  const entry = {
    polyfill: ["core-js/stable", "regenerator-runtime/runtime"],
    taskpane: "./src/taskpane/taskpane.ts",
    functions: "./src/functions/functions.ts", // put this last so that hot reloading works
  };

  const additionalEntryPoints = await getAdditionalEntryPoints();

  additionalEntryPoints.forEach(({ name, path }) => {
    entry[name] = path;
  });

  return {
    devtool: devMode ? "eval-source-map" : "source-map",
    entry: entry,
    resolve: {
      extensions: [".ts", ".html", ".js"],
    },
    module: {
      rules: [
        // {
        //   test: /\.js$/,
        //   use: {
        //     loader: "babel-loader",
        //     options: {
        //       plugins: ["transform-jsbi-to-bigint"],
        //     },
        //   },
        // },
        {
          test: /\.ts$/,
          loader: "ts-loader",
          options: {
            getCustomTransformers: () => ({ before: [tsNameof] }),
          },
        },
        {
          test: /\.html$/,
          loader: "html-loader",
          options: {
            preprocessor: (content) => (devMode ? content : transform(content)),
            sources: {
              //ignore 'src' attributes pointing to anything in the 'assets/' folder
              urlFilter: (attribute, value) => attribute !== "src" || !/^\/assets\//.test(value),
            },
          },
        },
        {
          test: /\.css$/,
          type: "asset/resource",
        },
      ],
    },
    output: {
      path: resolve(__dirname, "dist"),
      publicPath: "/",
      clean: true,
    },
    plugins: [
      new CustomFunctionsMetadataPlugin({
        input: "./src/functions/functions.ts",
        output: "functions.json",
      }),
      new TypedocWebpackPlugin({
        functionsFilename: "src/functions/functions.ts",
        tsConfigFilename: "./tsconfig.json",
        outputFilename: "assets/functions-typedoc.json",
      }),
      new HtmlWebpackPlugin({
        template: "./src/taskpane/taskpane.html",
        filename: "taskpane.html",
        chunks: ["polyfill", "taskpane", "functions"],
        minify: "auto",
        xhtml: true,
      }),
      ...additionalEntryPoints
        .filter(({ hasHtml }) => hasHtml)
        .map(
          ({ name, htmlName, htmlPath }) =>
            new HtmlWebpackPlugin({
              template: htmlPath,
              filename: htmlName,
              chunks: ["polyfill", name],
              minify: "auto",
              xhtml: true,
            })
        ),
      new CopyWebpackPlugin({
        patterns: [
          {
            from: "assets/*",
            //don't copy anything from the SVG folder
            filter: (filePath) => !filePath.includes("/svg/"),
            //don't transform when in development or if it's not in the list
            transform: (content, fullPath) => (devMode || !assetsToTransform.includes(basename(fullPath)) ? content : transform(content)),
          },
          {
            from: "manifest.xml",
            //don't transform when in development
            transform: (content) => (devMode ? content : transform(content)),
          },
          {
            from: "favicon.ico",
          },
        ],
      }),
    ],
    devServer: await getDevServerOptions(devMode),
    optimization: {
      splitChunks: {
        chunks: "all",
      },
    },
  };
};

WITHOUT BABEL PLUGIN:

WITH BABEL PLUGIN:

but the size on disk is larger with JSBI removed.

That is very unexpected. I'll try and set some time aside this week to look at the repo I cobbled together to see if that is also true for my setup.
Thanks for posting your webpack config - that helps a lot. Is the project itself opensource? If so, I'd be happy to fork a branch with your work in progress stuff and take a more direct look when I have time, but if not no worries. I'll try and replicate something similar based on what I see here.
I didn't expect that the representation of this polyfill in the source map explorer output to change so drastically between the two configurations... it sort of seems like in the "no Babel plugin" case the sourcemaps from our dist files (that point back to the TS files) are being ignored. There also appears to be two entire copies of the polyfill in there, but the sources referenced are identical? That also seems very odd to me.

Is the project itself opensource?

Unfortunately not, but I can tell you it's an Excel add-in which contains a taskpane, custom functions, and ribbon commands. The details of the app itself aren't important - I can try to strip down our code and see if I can include something you can look at. Maybe by whittling down I can find somewhere where it's being duplicated.
In both cases (babel and no babel) they are identical. So I'm not sure what's causing that - I don't think it happens for any other package but every other package is also much smaller. Maybe if it were tree-shakable it would be better? I don't know.

A minimal repro of your build setup including the tsconfig.json, webpack config files, package.json (with other libraries removed is fine), commands/scripts run, etc would be greatly helpful.

I've found the problem already. Our app works by defining the 'taskpane.html' and any dialog html pages. If I exclude all of the dialogs, then only one copy is produced. If I include a dialog that does not reference Temporal, only one copy is produced. When I include one or more dialog window pages, then a second copy is produced.
I can't imagine this is a js-temporal problem, but the other Temporal polyfill library (fullcalendar/temporal) does not exhibit this problem, nor any other npm package (that I can tell).
I'm not sure if that helps, but I will keep digging. I didn't have time last night, but I'll try to get a minimal repo for you soon.

That is very interesting indeed. My guess is that the way the dialog code is written is causing bundlers to think you need both the ESM and (either CJS or UMD) versions of Temporal at the same time.
Does https://github.com/js-temporal/temporal-polyfill/blob/95a131ddb47b7bef192cb0768d8986c97e0f6f78/package.json#L8 ever show up in your build somehow?

I can force all node_modules into a single chunk with the splitChunks.cacheGroups option, but is that really the best way?
    optimization: {
      splitChunks: {
        chunks: "all",
        //solves duplicate package problem but at what cost?
        cacheGroups: {
          vendor: {
            test: /[\\/]node_modules[\\/]/,
            name: "vendors",
            chunks: "all",
          },
        },
      },
    },

it might be that it "can't see" the files in node_modules

Hi, I managed to pick and choose from the npm-packaged typescript modules by applying this change #283. By transpiling them in my stack I could also apply the JSBI babel plugin to further reduce bundle-size.


Add Integration Tests for the Log components

Description:
This PR adds the below test cases on top of the POC to test the components CWLogs Exporter, Filelog Receiver.

log4j
syslog rfc5424
json

Testing:
Local Testing

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

We need to find a way to catch build time failures of this new test bed at PR time. Are build time failures of this test bed caught already by existing PR workflows?

Added PR build check for the build purpose


Block Legacy Authentication

Very crucial security mechanism that is not mentioned in the text above is to block legacy authentication. See blog post of Alex Simons: https://techcommunity.microsoft.com/t5/Azure-Active-Directory-Identity/Azure-AD-Conditional-Access-support-for-blocking-legacy-auth-is/ba-p/245417

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

ID: 826a3309-03a5-60b4-aa4c-63f385150455
Version Independent ID: 1d2e2f1c-2805-5cda-a4ad-76fedbc3f525
Content: Best practices for secure admin access - Azure Active Directory
Content Source: articles/active-directory/users-groups-roles/directory-admin-roles-secure.md
Service: active-directory
GitHub Login: @curtand
Microsoft Alias: curtand

@raffiloch Thank you for your feedback. We will investigate and update the thread.

@raffiloch We investigated your recommendation however it is already mentioned in the section where we advise to determine the exposure to password based sign-in protocols if the privileged admin accounts are using exchange online for emails. I have made slight modification to reference it to related articles as pointed out by you . However , the statement given also pertains to the same thing as point out by you , albeit not directly though . I have added links to the relevant pages for clarity of any reader. Thank you for pointing this out . Once it passes the review process it will be updated in the main doc.
We will now proceed to close this issue. Should you have any queries feel free to tag me to your reply and we can continue the conversation.
Thank you .


RSS support

Thank you for your contributions to the work.
I wonder whether there would be an RSS link so we can easily follow any website updates.
Thank you!

Thank you for your suggestion, we will support RSS in the future.
As this issue is a duplication of #27, we will close this issue, please watch #27 for subsequent updates.

RSS 已上线，地址为：https://zotero-chinese.com/feed.xml
该地址仅包含所有文档，日期为最后更新日期，不包含插件商店、CSL 商店和转换器商店的更新。


OpenBSD "Unknown OS, Assuming Linux"

When running rock under OpenBSD, it prints out Unknown operating system, assuming Linux... and then attempts to link to a non-existent .libs/linux32/libgc.a library.

Aha! I knew there was at least one BSD we hadn't covered :)
I'll try to set up an OpenBSD VM and play with getting this working.

Hi,
I no longer use "rock", but you can see how I had ported it to FreeBSD.
https://github.com/nddrylliog/rock/commits/master?page=6 (Jul 05, 2011)
Nicolas.


Refactor/ios

Lots of small improvements to the iOS codebase.

Moved a lot of public properties to private properties since there was no reason to keep them public.
Cleaned up header files
Moved data classes (event & batch) to a Models group
Renamed methods to be more descriptive
Added some documentation

ios/sdk/O2MC.xcodeproj/project.pbxproj looks like it's generated, if this is the case, would it be better to exclude it from git by placing its name in .gitignore?

The ios/sdk/O2MC.xcodeproj/project.pbxproj is indeed generated but should not be ignored since the project can't be compiled otherwise. You can compare it to a Makefile.


feat: Secure communication with redis

What type of PR is this?
/kind enhancement
What does this PR do / why we need it:
This PR adds support to watch for changes to the argocd-operator-redis-tls secret, which should contain the TLS configuration (cert/key combination) that the argocd components should use to communicate with the redis server. If a change is detected, the appropriate workloads will be restarted.  Note that this secret is something defined by the operator and not by Argo CD itself.
The PR adds two new settings

.spec.redis.disableTLSVerification: <bool>: if set to true, the application controller, the repo server and the api server will not validate the TLS certificate used by the redis server (by setting the --redis-insecure-skip-tls-verify flag when starting these components).
.spec.redis.autotls: <string>: Can be set to a provider which does automatic provisioning of the argocd-operator-redis-tls secret. Currently, the only provider supported is openshift

If the argocd-operator-redis-tls secret was created using the AutoTLS setting, no further configuration will be necessary. If the secret was manually added to the namespace, it will require an annotation argocds.argoproj.io/name: <instance_name> to be successfully mapped back to the managing ArgoCD instance.
OpenShift Container Platform provides a cluster-internal CA to issue service certificates, as described here. If .spec.redis.autotls: openshift is specified, the controller will add the appropriate annotation to the redis servers Service in the instance's namespace, which indicates that a TLS certificate for this service should be issued by the service CA.
Have you updated the necessary documentation?

[x] Documentation update is required by this PR.
[x] Documentation has been updated.

How to test changes / Special notes to the reviewer:
To test the enablement of TLS for Redis when autotls is disabled

Create an Argo CD CR with default values

apiVersion: argoproj.io/v1alpha1
 kind: ArgoCD
 metadata:
   name: argocd

Wait until reconciliation is complete and workloads became available
Create a self-signed certificate for the redis server. The cert will be stored in /tmp/redis.crt, and the key will be stored in /tmp/redis.key.  Set the environment variable NAMESPACE to the namespace where argocd is running.

openssl req -new -x509 -sha256 \
  -subj "/C=XX/ST=XX/O=Testing/CN=redis" \
  -reqexts SAN -extensions SAN \
  -config <(printf "\n[SAN]\nsubjectAltName=DNS:argocd-redis.$NAMESPACE.svc.cluster.local\n[req]\ndistinguished_name=req") \
  -keyout /tmp/redis.key \
  -out /tmp/redis.crt \
  -newkey rsa:4096 \
  -nodes \
  -sha256 \
  -days 10

Create the argocd-operator-redis-tls secret

kubectl create secret tls argocd-operator-redis-tls --key=/tmp/redis.key --cert=/tmp/redis.crt

Check that the pods for the argocd-server, argocd-repo-server, argocd-redis and argocd-application-controller were not restarted yet
Annotate the secret to indicate it belongs to the argocd operand

kubectl annotate secret argocd-operator-redis-tls argocds.argoproj.io/name=argocd

Check that the operator reconciled and the argocd-server, argocd-repo-server, argocd-redis and argocd-application-controller have been restarted.

Check that the redis server has been started with the correct arguments.  Run

kubectl get deployments.apps argocd-redis -o jsonpath='{.spec.template.spec.containers[0].args}'

The arguments should include

"--tls-port","6379"
"--port","0"
"--tls-cert-file","/app/config/redis/tls/tls.crt"
"--tls-key-file","/app/config/redis/tls/tls.key"
"--tls-auth-clients","no"

Check that the repo server has been started with the correct arguments. Run

kubectl get deployments.apps argocd-repo-server -o jsonpath='{.spec.template.spec.containers[0].command}'

The arguments should include

"--redis-use-tls"
"--redis-ca-certificate","/app/config/reposerver/tls/redis/tls.crt"

Check that the api server has been started with the correct arguments. Run

kubectl get deployments.apps argocd-server -o jsonpath='{.spec.template.spec.containers[0].command}'

The arguments should include

"--redis-use-tls"
"--redis-ca-certificate","/app/config/server/tls/redis/tls.crt"

Check that the application controller has been started with the correct arguments. Run

kubectl get statefulsets.apps argocd-application-controller -o jsonpath='{.spec.template.spec.containers[0].command}'

The arguments should include

"--redis-use-tls"
"--redis-ca-certificate","/app/config/controller/tls/redis/tls.crt"

To test the enablement of TLS for Redis when HA is enabled and autotls is disabled
Note that this requires running on a cluster with at least three worker nodes.

Create an Argo CD CR with which enables HA

apiVersion: argoproj.io/v1alpha1
 kind: ArgoCD
 metadata:
   name: argocd
spec:
  ha:
    enabled: true

Wait until reconciliation is complete and workloads became available.  This may take many minutes
Create a self-signed certificate for the redis server. Note that this cert is different from the one for non-HA mode. The cert will be stored in /tmp/redis-ha.crt, and the key will be stored in /tmp/redis-ha.key.  Set the environment variable NAMESPACE to the namespace where argocd is running.

openssl req -new -x509 -sha256 \
  -subj "/C=XX/ST=XX/O=Testing/CN=redis" \
  -reqexts SAN -extensions SAN \
  -config <(printf "\n[SAN]\nsubjectAltName=DNS:argocd-redis-ha-haproxy.$NAMESPACE.svc.cluster.local\n[req]\ndistinguished_name=req") \
  -keyout /tmp/redis-ha.key \
  -out /tmp/redis-ha.crt \
  -newkey rsa:4096 \
  -nodes \
  -sha256 \
  -days 10

Create the argocd-operator-redis-tls secret

kubectl create secret tls argocd-operator-redis-tls --key=/tmp/redis-ha.key --cert=/tmp/redis-ha.crt

Check that the pods for the argocd-server, argocd-repo-server, argocd-redis-ha-server-0, argocd-redis-ha-server-1, argocd-redis-ha-server-2, argocd-redis-ha-haproxy and argocd-application-controller were not restarted yet
Annotate the secret to indicate it belongs to the argocd operand

kubectl annotate secret argocd-operator-redis-tls argocds.argoproj.io/name=argocd

Check that the operator reconciled and the argocd-server, argocd-repo-server, argocd-redis-ha-server-0, argocd-redis-ha-server-1, argocd-redis-ha-server-2, argocd-redis-ha-haproxy and argocd-application-controller pods have restarted. It may take many minutes for the pods to restart.

Check that the configuration for the redis servers is correct. Run

kubectl exec -ti argocd-redis-ha-server-0 -c redis -- sh

Examine the redis.conf file.  Run less conf/redis.conf
The file should include the following lines

port 0
tls-port 6379
tls-cert-file /app/config/redis/tls/tls.crt
tls-ca-cert-file /app/config/redis/tls/tls.crt
tls-key-file /app/config/redis/tls/tls.key
tls-replication yes
tls-auth-clients no

Examine the sentinel.conf file.  Run less conf/sentinel.conf
The file should include the following lines

port 0
tls-port 26379
tls-cert-file "/app/config/redis/tls/tls.crt"
tls-ca-cert-file "/app/config/redis/tls/tls.crt"
tls-key-file "/app/config/redis/tls/tls.key"
tls-replication yes
tls-auth-clients no

Check that the repo server has been started with the correct arguments. Run

kubectl get deployments.apps argocd-repo-server -o jsonpath='{.spec.template.spec.containers[0].command}'

The arguments should include

"--redis-use-tls"
"--redis-ca-certificate","/app/config/reposerver/tls/redis/tls.crt"

Check that the api server has been started with the correct arguments. Run

kubectl get deployments.apps argocd-server -o jsonpath='{.spec.template.spec.containers[0].command}'

The arguments should include

"--redis-use-tls"
"--redis-ca-certificate","/app/config/server/tls/redis/tls.crt"

Check that the application controller has been started with the correct arguments. Run

kubectl get statefulsets.apps argocd-application-controller -o jsonpath='{.spec.template.spec.containers[0].command}'

The arguments should include

"--redis-use-tls"
"--redis-ca-certificate","/app/config/controller/tls/redis/tls.crt"

To test the enablement of TLS for Redis when autotls is enabled (only works on OpenShift)

Create an Argo CD CR with default values

apiVersion: argoproj.io/v1alpha1
 kind: ArgoCD
 metadata:
   name: argocd

Wait until reconciliation is complete and workloads became available
Patch the ArgoCD CR to set .spec.redis.autotls to openshift

kubectl patch argocds.argoproj.io argocd --type=merge -p '{"spec":{"redis":{"autotls":"openshift"}}}'

Check that a secret called argocd-operator-redis-tls was created by the CA's controller.

kubectl get secrets argocd-operator-redis-tls

The secret should be of type kubernetes.io/tls and should have a size of 2
Check that the operator reconciled and the argocd-server, argocd-repo-server, argocd-redis and argocd-application-controller have been restarted.
For each of the argocd-server, argocd-repo-server, argocd-redis and argocd-application-controller pods, do a kubectl exec -it <podname> -- sh and look for the following files to exist there:

argocd-server: /app/config/server/tls/redis/tls.crt
argocd-repo-server: /app/config/reposerver/tls/redis/tls.crt
argocd-redis-server: /app/config/redis/tls/tls.crt
argocd-application-controller: /app/config/controller/tls/redis/tls.crt

To test the enablement of TLS for Redis when HA is enabled and autotls is enabled (only works on OpenShift)
Note that this requires running on a cluster with at least three worker nodes.

Create an Argo CD CR with which enables HA

apiVersion: argoproj.io/v1alpha1
 kind: ArgoCD
 metadata:
   name: argocd
spec:
  ha:
    enabled: true

Wait until reconciliation is complete and workloads became available.  This may take many minutes
Patch the ArgoCD CR to set .spec.redis.autotls to openshift

kubectl patch argocds.argoproj.io argocd --type=merge -p '{"spec":{"redis":{"autotls":"openshift"}}}'

Check that a secret called argocd-operator-redis-tls was created by the CA's controller.

kubectl get secrets argocd-operator-redis-tls

The secret should be of type kubernetes.io/tls and should have a size of 2
Check that the operator reconciled and the argocd-server, argocd-repo-server, argocd-redis-ha-server-0, argocd-redis-ha-server-1, argocd-redis-ha-server-2, argocd-redis-ha-haproxy and argocd-application-controller pods have restarted. It may take many minutes for the pods to restart.

Hi @jannfis I've made the changes you suggested.  Please take a look.


make the HTTP settings available to the client

This is needed to correctly implement WebTransport. The client has to check that the server sent the WebTransport setting in SETTINGS, for example.

How about adding a new option in http3.RoundTripOpt, for example WaitForSettings, which indicates the client to not send the request until peer's settings frame is received. The http.RoundTripper struct can have a SettingsHandler callback, which will be passed into roundTripperOpts like StreamHijacker, to decide wether to accept the peer's settings.
type RoundTripOpt struct {
    ...
    // If set, the client will not send the request until the settings frame is received on the control stream.
    WaitForSettings bool
}

type RoundTripper struct {
    ...
    // If non-nil and WaitForSettings is set, it will be called once the settings frame is received.
    // If a non-nil error is returned, the request will not be sent and the client will close the QUIC
    // connection with an ErrCodeSettingsError.
    SettingsHandler(*settingsFrame) error
    ...
}

Inside the client, I still want to have the peerSettingsHandler implemented in #4168. The client can block on PeerSettingsReceived() <- chan struct{} and req.Context().Done() before sending the request.

@marten-seemann  could you share your view on this? really want to get this done and make the work on http/3 datagram and WebTransport move forward.

Currently on vacation, so it will be some time until I can comment here.
I don’t think this issue is needed for datagram support in WebTransport though, since datagrams are a requirement for the protocol anyway.


Glob pattern to cover both .ts and .tsx files

I'm trying to have apollo-codegen extract queries from both my .ts and .tsx files by using a glob pattern of '**/*.ts{,x} like this:
$ apollo-codegen generate '**/*.ts{,x}' --schema schema.json --target typescript --output schema.ts

I get this error:
.../node_modules/apollo-utilities/src/getFromAST.ts: Syntax Error /XXX/node_modules/apollo-utilities/src/getFromAST.ts (1:1) Unexpected )

1: );
   ^
2:   }

Not sure if my pattern is wrong or unsupported?

@jonaskello Unfortunately that is not supported at the moment. Here is what I've been using in my projects.
apollo-codegen generate $(find src -iname \"*.ts\" -o -iname \"*.tsx\") --target typescript --output src/schema.ts

I do agree that Apollo-codegen should natively support globs.

@jonaskello I may have answered too soon. It appears that globbing is ideally done by your shell, however only ZSH and bash v4+ when configured to do so will do the double-star glob. Even still I had issues getting that working in npm scripts which I assume you're trying to do too.
Here's what I ended up doing. schnittstabil/globstar is a package that lets you use globstar patterns in npm scripts in a cross-platform manner. Here is what my final command is. Note that double quotes are only required on windows, but I'd rather keep it generic.
"globstar --node -- apollo-codegen generate \"src/**/*.ts{,?}\" --target typescript --output src/schema.ts"

The double star seems to work for me out of the box (the examples in the readme also has double stars). It is just the .ts{,x} part that is not working. I also tried **/*.{ts,tsx} but it did not work either. The **/*.ts{,x} does works for me in mocha so I think it is a valid pattern.

I use zsh and have been running the following command without issue:
apollo-codegen generate src/**/*.ts src/**/*.tsx --schema schema.json --target typescript --output src/types/schema.ts

Nice workaround 👍 I didn't know that you could specify multiple file names.
If the intention is that you should be able to use glob pattern I think it is still broken, but this workaround is very useful.

This definitely works for me when using it as an npm script task inside package.json

"scripts": {
    "apollo:codegen": "apollo-codegen generate **/*.{ts,tsx} --schema ...",
}

In the prettier docs I found this comment:

Don't forget the quotes around the globs! The quotes make sure that Prettier expands the globs rather than your shell, for cross-platform usage. The glob syntax from the glob module is used.

Seems like they use " rahter than '. May be relevant here too since codegen also uses the glob package AFAIK. I'll close this for now until I have a chance to test that theory.


Feature Request: TLS 1.3 0-RTT

Add Feature for TLS 1.3 0-RTT in Netty.

@hyperxpro  Hi, is 0-RTT supported in netty with boringssl?


Native Dependabot breaks when transferring a repo

Package manage/ecosystem
javascript
What you expected to see, versus what you actually saw
So, I recently switched from depenabot-preview to native and it was working fine. A couple of days after that I moved my repo from a personal account to organization (from MrBartusek/ReplacementBot to ReplacementBot/ReplacementBot) After that, even when I have depnabot.yaml depenabot is not starting and prompting me to create a depenabot file
Images of the diff or a link to the PR, issue or logs

EDIT: Updating file fixes issue https://github.com/ReplacementBot/ReplacementBot/commit/13d54b70ccd607e9389e05e63ca01bfc029b2398

Sorry about this! Glad it's fixed though.

Today is finally the day that my oldest issue is closed, after 2 years and 2 months 🥳 Thanks @jeffwidman ❤


[MB-6832] Add Fuel Surcharge pricer params

Description
This PR adds pricing params to the Fuel Surcharge (FSC) service item's pricer.
Reviewer Notes

There is a migration included to switch the FSCPriceDifferenceInCents key to be of type DECIMAL instead of INTEGER because I discovered that fuel price data can be in tenths of a cent (which we do use in the calculation).
I added some additional tests to increase test coverage from 70-ish percent to 100%.
The front end UI for the FSC calculation needs some adjustments and a separate story is being created to track that.

Setup

make server_run and make client_run
In the customer UI, create a new move.
As a TOO in the office UI, find the move.  Update the missing fields for the orders, then approve the move so it's released to the prime.
Run prime-api-demo <locator> local and run through the steps until just after uploading proof of service docs.
As a TIO in the office UI, review/approve the payment request created by prime-api-demo (which should include FSC)
You should now be able to see the details of the FSC calculation (example below).

You can also do a prime-api-client --insecure fetch-mto-updates to verify the list of param key/values for the FSC item.
Code Review Verification Steps

Any new migrations/schema changes:

[ ] Follow our guidelines for zero-downtime deploys (see Zero-Downtime Deploys)
[ ] Have been communicated to #g-database
[ ] Secure migrations have been tested following the instructions in our docs

[ ] Request review from a member of a different team.
[ ] Have the Jira acceptance criteria been met for this change?

References

Jira story for this change

Screenshots

Messages

:book:
:link: MB-6832

  Generated by :no_entry_sign: dangerJS against 74b9a9383b50fe1f7f4f1270a25138f9f6b10789


⚠ Jenkins Plugins Artifactory has degraded performance

In a26ee58, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1376 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 34ee4f2 after 15 minutes.


objid_t: tracing infrastructure should use objid_t

Original issue: https://charm.cs.illinois.edu/redmine/issues/176

Once the basic objid_t scaffolding is available (#170), the impacts on the tracing infrastructure and projections logs should be audited. Specifically, Tracing routines on the message send and delivery paths might only have an objid_t available at that point in the code. For eg look at getProjectionID(). Hence, the benefits of managing all the tracing using objid_t should be evaluated. Implementation decisions should be made accordingly.

Original date: 2013-04-30 17:42:54

xi-symbol.C: generated tracing code uses ForArrayEltMsg explicitly, which is fine for now (so the tracing infrastructure won't get confused), but should eventually distinguish the ForIDedObjMsg case.

Original date: 2013-10-07 19:05:52

This won't be ready for 6.6, so push it back.

Original date: 2017-10-11 20:23:47

This is implicitly dependent on 64-bit ID, which is somewhat unstable according to Eric Mikida, so it's been waiting for that. It seems that it could be added for array elements now, but the intention was to wait until the ID stuff was stable for all kinds of chares.


artifactory_remote_maven_repository always updating password in plan/apply

Describe the bug
A clear and concise description of what the bug is.
Requirements for and issue

[x] A fully functioning terraform snippet that can be copy&pasted (no outside files or ENV vars unless that's part of the issue)
[x] Your version of artifactory (you can curl it at $host/artifactory/api/system/version
[x] Your version of terraform

Expected behavior
A clear and concise description of what you expected to happen.
Additional context
Add any other context about the problem here.
Terraform v1.1.7
on linux_amd64

Installing hashicorp/aws v4.3.0...
Installing hashicorp/helm v2.4.1...
Installing hashicorp/kubernetes v2.8.0...
Installing hashicorp/random v3.1.0...
Installing jfrog/artifactory v2.21.0...
Installing hashicorp/vault v3.3.1...

resource "artifactory_remote_maven_repository" "mulesoft-ee" {
  key                            = "mulesoft-ee"
  repo_layout_ref                = "maven-2-default"
  url                            = data.vault_generic_secret.mulesoft.data["url"]
  handle_releases                = true
  handle_snapshots               = true
  includes_pattern               = "**/*"
  username                       = data.vault_generic_secret.mulesoft.data["username"]
  password                       = data.vault_generic_secret.mulesoft.data["password"]
  socket_timeout_millis          = "5000"
  store_artifacts_locally        = true
  block_mismatching_mime_types   = true
  retrieval_cache_period_seconds = "6000"
  missed_cache_period_seconds    = "1800"
  project_environments           = []
  property_sets                  = []
}

Terraform Plan:
Artifactory 107.33.12
Terraform will perform the following actions:
  # artifactory_remote_maven_repository.mulesoft-ee will be updated in-place
  ~ resource "artifactory_remote_maven_repository" "mulesoft-ee" {
        id                                    = "mulesoft-ee"
      ~ password                              = (sensitive value)
        # (43 unchanged attributes hidden)

        # (1 unchanged block hidden)
    }

Plan: 0 to add, 1 to change, 0 to destroy.

After applying it will continue to state it will update in place. This happens whether password is hard coded or a data lookup.
Terraform state shows:
# artifactory_remote_maven_repository.mulesoft-ee:
resource "artifactory_remote_maven_repository" "mulesoft-ee" {
    allow_any_host_auth                   = false
...
    password                              = (sensitive value)

...
   unused_artifacts_cleanup_period_hours = 0
    url                                   = (sensitive)
    username                              = (sensitive)
    xray_index                            = false

There is a long, tortured story about passwords. We are working on a solution and welcome your commentary:

@alexhung - This may not be at all related to general password issues

@chb0github After worked on #383 I think this issue may be similar with state or unpacking or omitempty. Will take a quick look today.

Depends on #380

Confirmed that this issue is caused by the hashing of the password for TF state, as well the unpacking not ignoring password field on read().


refactor(app): Move /settings endpoints to redux-observable

overview
This PR continues the work of #3395 by refactoring the settings endpoints (/settings and /settings/pipettes) to use the new observable-based state management.
changelog

refactor(app): Move /settings endpoints to redux-observable

review requests
test plan

[ ] Changing robot-level advanced settings still work
[ ] Changing pipette-level settings still work
[ ] Errors during pipette settings changes display error message

e.g. You can induce a failure by turning your wifi off before you hit submit, then turning it back on quickly before the app disappears the robot for being unconnectable

Merged to allow for continued refactoring. @IanLondon and @b-cooper feel free to review and flag anything for followup changes in future PRs


[Android] Blank map on physical device

Hey all, I've read all of the blank map threads I can find and tried many of the solutions. However, I cannot get this map to show. Here's a screenshot, some relevant code, and a list of things I've tried.
I have doubts as to this being a style issue as the map is loading with a pale background and Google logo.

JSX

import React from 'react';
import { StyleSheet, Dimensions, View } from 'react-native';
import { Actions } from 'react-native-router-flux';
import MapView from 'react-native-maps';

export default () => (
  <View style={styles.container}>
    <MapView
      style={styles.map}
      initialRegion={{
        latitude: 37.78825,
        longitude: -122.4324,
        latitudeDelta: 150,
        longitudeDelta: 150
      }}
    />
  </View>
);

const styles = StyleSheet.create({
  container: {
    ...StyleSheet.absoluteFillObject,
    justifyContent: 'flex-end',
    alignItems: 'center'
  },
  map: {
    ...StyleSheet.absoluteFillObject
  }
});

android/app/build.gradle

...
dependencies {
    compile(project(':react-native-maps')){
        exclude group: 'com.google.android.gms', module: 'play-services-base'
        exclude group: 'com.google.android.gms', module: 'play-services.maps'
    }
    compile 'com.google.android.gms:play-services-base:10.0.1'
    compile 'com.google.android.gms:play-services-maps:10.0.1'
    compile fileTree(dir: "libs", include: ["*.jar"])
    compile "com.android.support:appcompat-v7:23.0.1"
    compile "com.facebook.react:react-native:+"  // From node_modules
}
...

android/settings.gradle

...
include ':react-native-maps'
project(':react-native-maps').projectDir = new File(rootProject.projectDir, '../node_modules/react-native-maps/android')
...

android/app/src/java//MainApplication.java

...
import com.airbnb.android.react.maps.MapsPackage;
...
    @Override
    protected List<ReactPackage> getPackages() {
      return Arrays.<ReactPackage>asList(
          new MainReactPackage(),
          new MapsPackage()
      );
    }
...

I've tried installing manually and with react-native link. I've tried changing my play services version to match my phone (12.2.38) which apparently doesn't exist - perhaps I missed something there.
Any help would be appreciated.

I was having the same issue than you, it was weird, so i took a look in the log on Android Studio and detected this:

I haven't enabled Google Maps Android API in my project, so you have to check that.
Go to your project's Control Panel and check that Google Maps Android API appears in the list. If it doesn't, enable the api by click the  button marked with red (It should say Enable API, my lang is Spanish as you can see).

Finally just make sure that you have this in your AndroidManifest
<meta-data android:name="com.google.android.geo.API_KEY" android:value="Your api key"/>

After that, the maps worked like a charm.
Hope it helps!

Thank you @eard7, it works!

That was it! Thank you very much, @eard7!


Assets load() function doesn't work with Array values

Per the user guide: https://straker.github.io/kontra/api/assets it seems to indicate that arrays are a valid input value for the load() function. But it errors out every time I try to use an array value for parameter, i.e.: load(array).then().....
The error is:
main-module.js:768 TypeError: url.replace is not a function     at joinPath (kontra-module.js:374)     at kontra-module.js:583     at new Promise (<anonymous>)     at loadImage (kontra-module.js:580)     at kontra-module.js:765     at Array.map (<anonymous>)     at Module.load (kontra-module.js:760)
If I hand feed it the file names, example: load("img1.png","text1.txt","audio1.wav").then()... with the same data it works as expected.

That is correct. The laod function takes a list of strings and not an array. It does accept an array for audio type files to support multiple audio types based on browser support.

Parameters
...urls
A list of Strings. Comma separated list of asset urls to load.


Adversarial, local, relative optima

While distribution on terms is optimized to minimize mismatch cost, we seek distribution on types to maximize this (but with a cost for generation).
This is across a space based on weights of various cost terms.
Cost terms include final entropies, and variables include coefficients as in #258 .
We look for, and store, local minima. These typically leave many statements unproved.
Also look for optima with relative costs : credit types with new unsolved/hard to solve problems and terms with improvement of mismatch.

It is safer to parametrize by separate costs, such as entropy of generating set and that of the final set.

Not clear this is useful given the Bot (and in the longer run deep learning) based plan.


How do we handle organizations for which we no longer have good contact information?

per #17

I think this is a duplicate of #21 and can be closed.

Yep, this is a dupe (but I have no /close magic powers).
cc: @anajsana


The 'send_confirmation_email' in sandbox gateway app, render must be a dict not context issue with Django 1.11

There is the same error in other project. Django 1.11 Must be a dict not context issue.

Thanks!