[WIP] Influx metrics reporter

This adds influx metrics reporter, it uses gokit wrapper added in https://github.com/jaegertracing/jaeger-lib/pull/23
I had to add a bunch of flags because (unlike prometheus/expvar) influx reporter requires at least url, credentials and database name. I also wanted to be able to add some predefined tags to be able to indicate the environment name.

@jpkrohling What should I do with this PR? Is it likely to get included? Or is the plugin approach the way to go?

Not "plugin" in the sense of storage plugins (which could be in a different process, for example), but the factory pattern used for storage https://github.com/jaegertracing/jaeger/blob/master/storage/factory.go
The main thing in that factory pattern is that CLI options do not show up unless a specific factory type is enabled.
Alternatively, is this heavy direct integration with influxDb the only way to report metrics to it? Isn't it possible to keep using Prometheus metrics in Jaeger and run a separate scraper that pulls the metrics from the process and converts then into influxDb?

+1 to what @yurishkuro wrote. It's clear by now that we'll not go with go plugins and this is what I had when I wrote the comment back in April.

I guess I can check how it would look with prometheus reporting to influx. IIRC I tried this a year ago and something was off with histograms.

I checked telegraf - it has an input plugin for Prometheus and an output plugin for Influx. Unfortunately it does not support histograms - Prometheus histogram buckets are sent to influx as-is (and fields of a metric). And it's not possible to make use of them (calculate e.g. 99th percentile) because there isn't anything in Influx which could behave like histogram_quantile() from Prometheus.
I didn't try to run Prometheus with Influx storage but I'm sure it would be the same case.
This basically means that latency percentiles wouldn't be available and that is not cool.

I didn't try to run Prometheus with Influx storage but I'm sure it would be the same case.

Why do you think so?  I expect that Prometheus stores latencies in the same format as it captures them, as a group of buckets with counts (which should be fine for Influx), and the percentiles are calculated by Prometheus server.

@mabn I am going to close this old PR, since it's unlikely that we want to support direct integration w/ InfluxDB for metrics. In the future we will likely upgrade Jaeger internals to use OpenTelemetry, and there probably will be some standard integration of OpenTelemetry with InfluxDB so that we won't need anything special in Jaeger.


[BUG] grouped bar chart with hover interaction and custom tooltip causes chart to only render one set of data

Expected Behavior

This is made with chart js and reactjs
In a grouped bar chart with multiple datasets and custom tooltip, when you hover over a group expect the tooltip to show up and the chart does not rerender
Or it should rerender with the same data
When we hover over a group the others bars dissapear and we only show the last sections' bars

Current Behavior

On the hover it renders only the 1 of my six bar chart data sets

Possible Solution

I noticed the redraw option for charts will destroy, recreate a new bar chart. In that moment the canvas context can be null. That maybe the source of the issue, possibll have a null check in that function.

Steps to Reproduce (for bugs)

1.Create a stacked bar chart with following data:
        datasets: [
          { data: [10, 20, 30, 50], label: "zz", backgroundColor: "#4286f4" },
          { data: [30, 20, 10, 40], label: "zz", backgroundColor: "#e5f94a" },
          { data: [0, 15, 20, 30], label: "zz", backgroundColor: "#f2264c" },
        ],
        labels: ["01/12", "02/12", "03/12", "04/12"],
      },

Have options with xAxes, yAxes stacked: false

hover: {
          onHover: (e: any) => {
            if (e && e.target) {
              e.target.style.cursor = 'pointer';
            }
          },
        },

tooltips: {enabled: false} <- okay since we are testing the hover interaction
3. have a hover action trigger different bar data to trigger a react rerender
4. hover over the grouped bar chart
5.  graph render is all wonky, renders only one section
Context

I want to draw a custom tooltip when I hover over the grouped barchart to show the points clearly.
It seems to be a weird state transition and rendering issue

Environment

Chart.js version: 2.7.3
react-chartjs version: 2.7.4
Browser name and version: latest chrome
Link to your project: https://codesandbox.io/s/98xm62929o

Yes ofcourse, added

I noticed the redraw option for charts will destroy, recreate a new bar chart.

That comes from your code (or the react-chartjs one?), not ours. I guess that calling this.setState() triggers a new render() of your component which destroy/recreate a new chart. So you may need to review your tooltip strategy in order to prevent this behavior but this is beyond the responsibility of this repository. You can still ask support on StackOverflow or the /reactjs/react-chartjs repository.


examples: fixing missing namespace

If we don't specify namespace while creating a ConfigMap to read keys from it, we can get errors like:

So, we can define what should be done and where to as a side note in example for user experience as in commit history.

Thank you for your reply and explanation 🙋🏻♂
Do you think that it is clearer than before if there is a Note comment for all examples ? Because I review all of them to discover Argo Workflows fully, I can do it for all.


Implemented ReadPipeScalar

PR for #108

Coverage increased (+0.08%) to 90.525% when pulling a021d3abde908fb380c62c1c66032adcbcb2fcec on SherifWaly:#108 into 05378e3a7498f14031586bc0e04c38d4682c13e6 on decorators-squad:master.

Coverage increased (+0.08%) to 90.525% when pulling a021d3abde908fb380c62c1c66032adcbcb2fcec on SherifWaly:#108 into 05378e3a7498f14031586bc0e04c38d4682c13e6 on decorators-squad:master.

@amihaiemil Please check this commit :)

@SherifWaly If you spent the 30 min already, just add the toString() and leave a puzzle for the encapsulation stuff :D

@amihaiemil I will fix it now but I've made that as I thought I should indent first line only :D

@SherifWaly Nope, all the lines should be intended. Think of when you're printing the Yaml, you don't want to see it like this:
>
  first line
second line

You want to see it like this:
--->
  first line
  second line

Coverage increased (+0.02%) to 90.462% when pulling 4b45110529e024b47baff14ea491c2faf1cdda67 on SherifWaly:#108 into 05378e3a7498f14031586bc0e04c38d4682c13e6 on decorators-squad:master.

@amihaiemil I have changed value and indent methods and modified the unit test.

@SherifWaly Looks good! :)

@rultor merge please

@rultor merge please

@amihaiemil OK, I'll try to merge now. You can check the progress of the merge here

@rultor merge please

@amihaiemil Done! FYI, the full log is here (took me 1min)


Miscellaneous fixes for windows

Several fix for windows compilation

Add windows specific flags for skeleton subsystem
Remove the need for a semaphore in test-platform -> use a future&promise
Correct a unnecessary cast that was loosing precision

[x]   General Comment
<img border=0 src='https://avatars.githubusercontent.com/u/6862950?v=3' height=16 width=16'> @OznOg @dawagner please review

This pull request is now managed with CodeReviewHub. Refresh.

@OznOg @dawagner please review

:+1:

:+1:


check for solr.jetty.https.port porp to set https urlscheme

It seems clusterprops has very sophisticated mechanism in solr. Given the complexity its not worth. Closing this PR/ticket.


Change "editable" join tables to versioned records

Specifically the relevant tables are:

actor_categories
measure_categories
memberships
measure_resource
user_role

@parndt happy to do it myself if this is as easy as changing the first line in the respective model files from eg "class MeasureResource < ApplicationRecord" to "class MeasureResource < VersionedRecord". Thanks for confirming =)

@tmfrnz not quite that simple 😄 will open a PR

@tmfrnz it's a bit confusing though because these tables aren't editable
measure_categories CRD #27
actor_categories CRD #24
memberships CRD #19
measure_resources CRD #30
user_roles #36
So all of their policies etc won't allow this. See PR #92 for the implementation so far. It will need updating to change the policy files for each join table that can now allow updates, as well as extra test cases added to their controller specs if we want to make this change.  Let me know if you want that 😄


Bake times extremely slow, even with 25 dedicated cores / paid Azure account

Hi again,
Thank you for the assistance thus far with getting Project Acoustics up and running. I wanted to check back in as I have submitted a build over 24 hours ago, for a level featuring ~3,000 probes. I upgraded my account to a paid account, and see that there are 25 dedicated nodes currently active and running. I have requested a further increase to my cores, but the request has been denied. For the first hour or so of the build, progress was very quick, completing ~2,000/6000 tasks within an hour or two. Then it suddenly slowed to a crawl and has been limping along every since.
I'm just wondering if this is typical behavior. If not, would this be the correct forum to request assistance in resolving this issue? The project I'm working on features over 10 maps, several of which will likely have more probes than this (i.e. they are significantly larger/have more complex geometry) - I cannot imagine the team being satisfied with bake times of multiple days per level. Any assistance / advice would be highly appreciated.
Thank you!

The quick-slow behavior is as expected - Each simulation runs in two phases the first of which is quick.
A couple recommendations:

Use low-priority nodes as that will save you a lot of money (an integer factor). Our experience has been that there is not much difference in compute time although of course with low-pri you can get evicted. But good place to start for your first evaluations, as it is cost effective.
The wall clock time will go down linearly with number of nodes, dollar cost remains same. (The nodes are just baking volumetric probes separately, its embarrassingly parallel). So max out your node allocation - up to number of probes. With your current allocation this will take about 60 hours, with 100 nodes that comes down to 15 hrs, and so on.

Also - as a quick note, the bake cost is insensitive to geometric complexity/triangle counts. It scales (mostly) linearly with number of probes.

@nikunjragh  I think the local bake is a bit more logical here, baking .dat and .enc in pairs.  Also it would eliminate questions like these :D

Hi @jaegrover, we're happy to try to help, unfortunately the behavior you've seen is expected and the only solution is to get an increase in your Batch core quota. The default allocation is 25 cores which gets you only 1 16-core or 2 8-core nodes.
You mentioned that your quota increase request was denied. Unfortunately all I can suggest is to re-open the request and work with Batch support to address their reasons for denial. We have seen that requests for large quota increases (such as to thousands of cores) can require demonstrating the financial resources to cover large charges should the cores be occupied for a significant portion of a given billing period. The typical Project Acoustics use case only occupies the cores for a small period of time (one or a few hours) so such large charges are unlikely to result from using Project Acoustics.
@ashtat @NoelCross Can you share some info on the process to get a quota increase, or a suggested quota increase request size that we have seen succeed in the past?

Hi folks,
Thanks very much for your replies and support thus far. I have just noticed that the price for this initial bake is ~$500.00, which is admittedly higher than I had initially expected. Given the previous comments about cost scaling proportionally with the number of dedicated cores available, it now makes sense why my request for additional cores was denied.
It looks like I did not have the "use low priority nodes" option checked when initially submitting the bake - it also seems that my Azure account does not currently list any low-priority nodes associated with my account. I will reach out to Batch support again today to attempt to remedy that issue, and will try a smaller bake using the low-priority option if possible.
I guess I still feel like I'm missing something, and I apologize if I am asking anyone to repeat themselves. From the language in the PA instructional PDF, it seemed initially that one should expect a bake to take a few minutes - @kegodin, it also sounds like you're implying that PA bakes typically don't consume much cost / resources. Yet this bake, with what seems to me to be a relatively low dedicated node allocation and using one of our game's average-sized and less complex maps as a test case, is consuming several days, and several hundred dollars' worth of resources. I'm confused as to how this would be viable for smaller / indie developers - unless I've misinterpreted something, and this is a tool primarily aimed at projects with larger-scale / AAA production budgets, or I've simply missed something in my setup process.
Thanks again for taking the time to walk me through this - I hope we can figure something out!

Hi @jaegrover , thanks for sticking with us so far.
With a core quota increase your observed bake time (wall clock time) should decrease significantly. The workloads are parallelized by probe. With a core quota in the range of 2k cores, I'd expect you could complete a 3k probe bake in single-digit hours, much closer to the claims you see in our documentation.
As far as cost, as @nikunjragh mentioned above, cost scales linearly with probe count. We have generally seen that Azure Batch costs for low priority nodes are a fraction of regular prices; for example (https://azure.microsoft.com/en-us/pricing/details/batch/) in West US region the recommended F8 nodes have a low priority cost at 20% the cost of regular priority. Hypothetically, reducing the probe count to 1.5k and switching to low priority nodes would bring the cost to the $50 range.
To give you a sense of how we've calibrated the language in our documentation, 3k+ cores is among the largest maps we have heard of being baked with Project Acoustics, being larger than map sizes used in the AAA games that have shipped Project Acoustics so far. This map size is absolutely considered within the sizes we support. I am only stating this in the hope that it explains where our assumptions are coming from and the discrepancy between our documentation and your observations. Again, your use case is certainly supported by the tools.
Also, I'm not sure if I missed it or if you didn't mention - but if you're doing a 'fine' bake, one way to reduce cost and time significantly is to switch to a 'coarse' bake.

Also, let me clear our some potential confusions here @jaegrover
To summarize @kegodin's main point: With 3000 low-priority 8-core VMs (e.g., Standard_F8s_v2), both your USD cost and wall-clock bake time will reduce significantly.

Cost scaling

Given the previous comments about cost scaling proportionally with the number of dedicated cores available, it now makes sense why my request for additional cores was denied.
Cost of baking a given map does not scale with number of VMs allocated. It will cost much the same but will just go faster if you have more VMs. @kegodin was explaining why Batch requires paperwork to increase the quota.

Cores vs "nodes"
In analogy to how a PC is 1 machine with many cores, in Azure, a VM (Virtual Machine) or "node" is like a machine, and each VM has some number of cores. Our baking time scales down with VMs.

Probe counts
3000 probes is on the higher side for a single bake, as Keith noted - AAA games manage by typically splitting the map by splitting nav mesh, doing multiple bakes. But for this map at least, you could try the simpler set of increasing the max probe sample spacing from the default 3.5m to 4 meters and see how that affects your probe count. If you're interested, we can point you towards how to adjust this. Also, for iterating, using coarse bakes as @kegodin recommended is a good step.

Let me note overall that our wish is one day users don't have to worry about such details, but this is how far we've gotten till now.

Thank you kindly :) Especially having worked with the traditional volumes/rooms and portals setup on a previous FPS project, which was functional but very cumbersome, I very much appreciate the work you've done to streamline the whole process. I also appreciate your patience helping out a user who is both a n00b to PA and to Azure.
I will write back to Azure customer support about increasing my allocation of low priority nodes, as it seems I currently have no low priority nodes available on my Azure account.
I will also be sure to speak with members of the dev team regarding options for splitting up the maps and scaling geometry to reduce probe count for each individual bake.
Thanks again for all of your support thus far, and have a great day :)

In the meantime, @nikunjragh , if you could indeed point me to where to adjust the probe distancing / placement count, that would be awesome, at least to have it available for future experimentation. Thanks again!

Sure, its discussed in Issue #25, quoted below:

You can't place individual probes, but you can change the density. If you find the _AcousticsParamters asset in your hierarchy, you can change the density. For example, in the demo scene it is at Assets/AcousticsDemo/Editor/ProjectAcousticsDemo_AcousticsParameters.  If you change the Probe Horizontal Spacing min/max values then redo the probe calculation, you should see the resulting probe density change (before you do another bake).

Tagging @MikeChemi who can comment if there were some other details to changing spacing in Unity. I'm suggesting changing the max spacing parameter, increase it from 3.5m.

Nope, that's the useful one. Just want to stress that you should verify the correct spacing in-editor before baking again. I've personally adjusted the wrong parameter many times before and got stuck wondering why spacing didn't do the right thing.

Also, we're continually optimizing our bake tools internally, @KyleStorck can tell you how to switch to new build for our bake tools that is about 1.4x faster. Fresh off the presses though, but its been working well in our tests.

Hi @jaegrover. You can try out the faster bake tools by changing the "Toolset Version" to "mcr.microsoft.com/acoustics/baketools:1.2_Optimized"


[CAMEL-9869] Create Apache Flink Component

Hello @davsclaus ,
I have enhanced the camel-flink producer implementation to support Flink datastream.
Please review.

Hello @davsclaus ,
Thanks for your review & comments.
I have made all the necessary changes.

standard for keys in camel is maintained.
all checkstyle issues are fixed.
all empty lines are removed after the license.

Please review now.


Menus have no keyboard selection indicators - but only on no-italics theme (+ possible fix?)

This happens on every menu that isn't the main Cmd + Shift + P menu.
An example is the recents menu (^ + R / Ctrl + R on macOS. Not sure about the equivalent shortcut on Windows).
Try pressing up / down on the keyboard to move around the menu -- the selection indicators are missing. Mousing over them makes the text bold which is a good indicator, but only selecting them with the keyboard does nothing.
This is only on the no-italics version of the theme. The regular theme has it just fine -- so I'm guessing this is a key that's on the regular theme but hasn't been copied to the no-italics theme.
It might be "list.focusBackground" but I'm not sure.
GIF showing the issue:

Duplicate of https://github.com/sdras/night-owl-vscode-theme/issues/139

Yes, this should be fixed with that PR :) I'll close it out in the meantime but feel free to reopen if after that PR is in, it's still not right. Thanks for reporting!


How to add a bash script to the Dockerfile so that it becomes launched when the container starts ?

I have adapted the bash script asterisk_reg.sh and would like to run this script when the docker container becomes started.
I can launch the adapted script manually by connecting to the docker container and executing below command (that is working fine)
bash asterisk_reg.sh

but I would like that this script becomes started automatically when the container starts.
How can I best do that ?
FYI here below my Dockerfile.
from tiredofit/freepbx:15-5.0.6

ADD ./asterisk_reg.sh /usr/local/bin/asterisk_reg.sh

You are almost there.
Since I don't have custom script support added into the image (I will add), your easiest bet is to do this:
Add a file called 20-asterisk_reg to /etc/cont-init.d with the following content:
#!/usr/bin/with-contenv bash

source /assets/functions/00-container
prepare_service single
# shellcheck disable=SC2034
PROCESS_NAME="asterisk_reg"

### PUT CUSTOM HERE

### END CUSTOM HERE

liftoff

This will run the script after all the FreePBX and Asterisk initialization has completed. Let me know if this works, and I'll see about adding custom script support into this one which will be as simple as mounting a folder called /assets/custom-scripts/ and calling your scripts 01,02-whatever to have them execute in order.

Thanks for the very fast response.
I have implemented the changes and the container is properly starting my asterisk_reg script but as this script runs forever I see that it doesn't launch the things that should come after it (see log file below).  I don't think that this is the purpose.  Should the script not be added to /etc/services.d instead of /etc/cont-init.d/ ?
[INFO] ** [mariadb] No embedded database detected, skip..
[cont-init.d] 09-mariadb: exited 0.
[cont-init.d] 10-freepbx: executing...
[NOTICE] ** [freepbx] Setting file permissions
[NOTICE] ** [freepbx] Setting configuration
[NOTICE] ** [freepbx] Setting RTP ports - start: '18000' finish: '18100'
[INFO] ** [freepbx] Starting Asterisk 17.6.0
[NOTICE] ** [freepbx] Enabling SSL
[WARN] ** [freepbx] No SSL certs found, autogenerating self-signed - WebRTC will not work with a self-signed certificate!
[INFO] ** [freepbx] Starting operator panel
[INFO] ** [freepbx] Web server started - container initialization completed - visit your http(s)://.../admin to administer
[cont-init.d] 10-freepbx: exited 0.
[cont-init.d] 15-socat: executing...
[cont-init.d] 15-socat: exited 0.
[cont-init.d] 20-asterisk_reg: executing...

I suppose it all depends on your script - If you had it in services.d it would execute again if the process finished which is maybe what you want. In that case, I would put it in services.d.
Again though hard to say what you are trying to accomplish.
For what it's worth though, the new image I pushed this morning has custom-scripts suppoer . Drop your 01-xxx.sh script in /assets/custom-scripts -- You could even get creative and write to /var/run/s6/services/20-asterisk-reg/run (essentially /etc/services.d but while a container is running) and create a mock service that way without having to create another Docker image.

The custom-scripts functionality is not working for me.
What I did:
I have upgraded to image 15-5.0.7 (before I was using 15-5.0.6).
I have copied my script to folder /assets/custom-scripts
The contents of my Dockerfile:
from tiredofit/freepbx:15-5.0.7

ADD ./01-asterisk_reg.sh /assets/custom-scripts/01-asterisk_reg.sh
RUN chmod a+x            /assets/custom-scripts/01-asterisk_reg.sh

The first part of my script 01-asterisk_reg.sh:
#!/usr/bin/with-contenv bash
pidof -o %PPID -x $0 >/dev/null && echo "ERROR: Script $0 already running" && exit 1

DB_TABLE="pushdb_pushkeys"

tail -Fn0 /var/log/asterisk/full | \
while read line ;
 do
        echo "$line" | grep "Registered SIP" > /dev/null
        if [ $? = 0 ]
        then
...

Which issues are noticed:
Above changes broke my freepbx installation:

my linphone is no longer able to connect to asterisk.
freepbx web console is no longer responding
...

I also see that my script 01-asterisk_reg.sh is running twice:

This is what I see in the log file:
...
[cont-init.d] 10-freepbx: executing...
[NOTICE] ** [freepbx] Setting file permissions
[NOTICE] ** [freepbx] Setting configuration
[NOTICE] ** [freepbx] Setting RTP ports - start: '18000' finish: '18100'
[INFO] ** [freepbx] Starting Asterisk 17.9.3
[WARN] ** [freepbx] Found Custom Scripts to Execute
[WARN] ** [freepbx] Running Script /assets/custom-scripts/01-asterisk_reg.sh

FYI this is what I see in my log file when not using a custom script:
...
[cont-init.d] 10-freepbx: executing...
[NOTICE] ** [freepbx] Setting file permissions
[NOTICE] ** [freepbx] Setting configuration
[NOTICE] ** [freepbx] Setting RTP ports - start: '18000' finish: '18100'
[INFO] ** [freepbx] Starting Asterisk 17.9.3
[NOTICE] ** [freepbx] Enabling SSL
[WARN] ** [freepbx] No SSL certs found, autogenerating self-signed - WebRTC will not work with a self-signed certificate!
[INFO] ** [freepbx] Starting operator panel
[INFO] ** [freepbx] Web server started - container initialization completed - visit your http(s)://.../admin to administer
[cont-init.d] 10-freepbx: exited 0.
[cont-init.d] 15-socat: executing...
[cont-init.d] 15-socat: exited 0.
[cont-init.d] 99-container: executing...
[cont-init.d] 99-container: exited 0.
[cont-init.d] done.
[services.d] starting services
[services.d] done.
[INFO] ** [zabbix] Starting Zabbix Agent
[INFO] ** [cron] Starting cron

So based on comparing the log files with and without custom script I see that in case of with custom script it is not executing/launching everything.

Yes, makes sense with your while statement in your script. The custom script functionality runs before everything starts (I use this to overwrite certain files, or do some modifications).
Try this in your script:
#!/usr/bin/with-contenv bash

source /assets/functions/00-container
prepare_service single
PROCESS_NAME="asterisk_reg_builder"

mkdir -p /var/run/s6/sevices/21-asterisk_reg/run
cat <<EOF > /var/run/s6/services/21-asterisk_reg
#!/usr/bin/with-contenv bash
source /assets/functions/00-container
prepare_service defaults 10-freepbx
PROCESS_NAME="asterisk_reg_builder"
check_container_initialized
liftoff
## Your script goes in here - I don't believe you'll need the pidof statement with the way this will work
## Be very careful when with this script and make sure you escape any dollar signs like how I have below

#pidof -o %PPID -x $0 >/dev/null && echo "ERROR: Script \$0 already running" && exit 1

DB_TABLE="pushdb_pushkeys"

tail -Fn0 /var/log/asterisk/full | \
while read line ;
 do
        echo "\$line" | grep "Registered SIP" > /dev/null
        if [ \$? = 0 ]
        then
...
EOF

chmod +x /var/run/s6s/services/21-asterisk_reg

What you'll get out of this is if you watch /tmp/state/21-asterisk-reg you'll get an entry every time the process "restarts" with a datestamp.
Let me know if this gets you going.

Thanks it is working now but  the log file is reporting following error

s6-svc: fatal: unable to control /var/run/s6/services/21-asterisk_reg: supervisor not listening

I am not sure if it is important:
My log file contents:
...
[INFO] ** [freepbx] Starting Asterisk 17.9.3
[WARN] ** [freepbx] Found Custom Scripts to Execute
[WARN] ** [freepbx] Running Script /assets/custom-scripts/21-asterisk_reg_builder.sh
s6-svc: fatal: unable to control /var/run/s6/services/21-asterisk_reg: supervisor not listening
[NOTICE] ** [freepbx] Enabling SSL
[WARN] ** [freepbx] No SSL certs found, autogenerating self-signed - WebRTC will not work with a self-signed certificate!
[INFO] ** [freepbx] Starting operator panel
[INFO] ** [freepbx] Web server started - container initialization completed - visit your http(s)://.../admin to administer
[cont-init.d] 10-freepbx: exited 0.
[cont-init.d] 15-socat: executing...
[cont-init.d] 15-socat: exited 0.
[cont-init.d] 99-container: executing...
[cont-init.d] 99-container: exited 0.
[cont-init.d] done.
[services.d] starting services
[services.d] done.
[INFO] ** [zabbix] Starting Zabbix Agent
[INFO] ** [cron] Starting cron

FYI I did something different from what you proposed in previous comment (I think it should give the same result).
My dockerfile:
from tiredofit/freepbx:15-5.0.7

ADD ./21-asterisk_reg.sh         /root/21-asterisk_reg.sh
RUN chmod a+x                    /root/21-asterisk_reg.sh

ADD ./21-asterisk_reg_builder.sh /assets/custom-scripts/21-asterisk_reg_builder.sh
RUN chmod a+x                    /assets/custom-scripts/21-asterisk_reg_builder.sh

My service build script : 21-asterisk_reg_builder.sh
#!/usr/bin/with-contenv bash

source /assets/functions/00-container
prepare_service 10-freepbx
PROCESS_NAME="asterisk_reg_builder"

mkdir -p                            /var/run/s6/services/21-asterisk_reg
cp /root/21-asterisk_reg.sh         /var/run/s6/services/21-asterisk_reg/run
s6-svc -u                           /var/run/s6/services/21-asterisk_reg

The actual service script : 21-asterisk_reg.sh
#!/usr/bin/with-contenv bash

source /assets/functions/00-container
prepare_service defaults 10-freepbx
PROCESS_NAME="asterisk_reg"
check_container_initialized
liftoff

DB_TABLE="pushdb_pushkeys"

tail -Fn0 /var/log/asterisk/full | \
while read line ;
 do
        echo "$line" | grep "Registered SIP" > /dev/null
        if [ $? = 0 ]
        then

...

screenshot of the processes running (command ps aufx) :

Sorry about this - this is getting way more complicated than it needs to be.
Instead of sending your secondary script to /var/run/s6/services could you try to send to /etc/services.d/ ? .
Last but not least if all else fails. On the first script:
echo "* * * * * /scriptlocation.sh" >>/assets/cron/crontab.txt
This will make it execute every minute - which is probably not what you want, but if you had the pidof statement in an earlier script it would limit it from running multiple times.


Unable Undo

Unable Undo

Hello, which action are you unable to undo ?

delete  node

The issue is fixed in 014dfd75527bf893fd7dd97ed892732ef22a0dfe.
Thanks for the bug report


Support for AddDirectory(string path, isRecursive = true) to BundleBase

Added support for this call as it's more discoverable and added new feature to support non-recursive (recursive is still default). Did not change behavior of Add().

Probably should have made it optional when we added directory support, but wasn't sure anyone would need it.  Thanks for proving me wrong in the kindest way possible :)

I don't know if anyone else needs it to be options, but I did.


Domain activation is Failed - [BRIEF DESCRIPTION]

App Id: d1lxf1wbkd51eu
Region: eu-west-1
Step: Domain activation
Status: Failed

Note: Do not include information that is sensitive in nature such as your domain name, company etc.

Issue/question
Not able to activate the domain
Error message
Activation failed/ Screenshot attaching with the case
Additional information
It was pointed to cloud front distribution and was a s3 hosting . we have moved the hosting to amplify. While adding the custom domain I am getting this issue

Refer to our documentation to troubleshoot domain setup.
https://docs.aws.amazon.com/amplify/latest/userguide/custom-domain-troubleshoot-guide.html
https://docs.aws.amazon.com/amplify/latest/userguide/custom-domain-troubleshoot-guide.html#i-get-a-cnamealreadyexistsexception-error
Let us know if you're still facing issues.

Refer to our documentation to troubleshoot domain setup.
https://docs.aws.amazon.com/amplify/latest/userguide/custom-domain-troubleshoot-guide.html
https://docs.aws.amazon.com/amplify/latest/userguide/custom-domain-troubleshoot-guide.html#i-get-a-cnamealreadyexistsexception-error
Let us know if you're still facing issues.

We are going to close this issue.  Please let us know if you need further assistance.

We are going to close this issue.  Please let us know if you need further assistance.


Test Databricks / Spark type acceleration into SQLite, DuckDB, Postgres Accelerators

Also test the ODBC connector with Databricks

For ODBC
tested with odbc:samples.tpch.customer, working well for all accelerators.
tested a lineitem dataset, works well
2024-06-03T04:18:05.093985Z  INFO runtime: Registered dataset lineitem
2024-06-03T04:18:05.094186Z  INFO runtime::accelerated_table::refresh: Loading data for dataset lineitem
2024-06-03T04:21:07.669949Z  INFO runtime::accelerated_table::refresh: Loaded 29,999,795 rows (145.96 GiB) for dataset lineitem in 3m 2s 574ms.


🚀 [Feature] Frontend: Use Stories from the Storybook to create the first page of the website

🚀 Feature
We have a link between Figma and Chromatic to create Stories in the Storybook.
Please use them to create the TopBar, the first page and the footer.
🤖 ToDo

[ ] Use the TopBar Storie in the PageShell.vue
[ ] Create the first page of the one pager
[ ] Use the Footer Storie in the PageShell.vue

This is split into several small issues assigned to and discussed with mackes


Doc Feedback: Instruction for port offset doesn't work

Location : https://apim.docs.wso2.com/en/latest/SetupAndInstall/DeployingWSO2APIManager/ThirdPartyKeyManager/configuring-wso2-identity-server-as-a-key-manager/
When trying to change offset in carbon.xml, it's getting rewritten to default '0' when server is started

This has been fixed. Please refer to https://apim.docs.wso2.com/en/latest/InstallAndSetup/DeployingWSO2APIManager/DistributedDeployment/configuring-wso2-identity-server-as-a-key-manager/#step-2-optionally-configure-port-offset-for-wso2-is


Add offset to Limit plan node and operator

This functionality supports queries like SELECT ... OFFSET n LIMIT m

Replaced with #375


[ENG-1417] Waffle so EGAP registrations are visible to admins

Purpose
Adds a group and flag that give EGAP admins access to the EGAP Registration schema
Changes

adds new manager method that allows users in the egap admins group in
adds a management command that adds appropriately configures flags and groups
removes some old waffling that was making it hard to maintain tests.

QA Notes
Should be dev test only. But QA/admin should remember to remove the old ENABLE_INACTIVE_SCHEMA switch.
Documentation
Docstrings, but no user facing docs.
Side Effects
Removes old waffling for ENABLE_INACTIVE_SCHEMAS and changes some test behavior.
Ticket
https://openscience.atlassian.net/browse/ENG-1417

Oh, one more thing: could you add testing notes to verify that the changes to only_active=false don't cause any regressions? That's a potential side effect.


Theme overriding: asset hash is not refreshed

Issue #7350 identified two issues with theme overriding that weren't working properly. Unfortunately, I've found a 3rd 😞
The asset helper adds a query param and asset hash to the end of all assets, allowing assets to be cached. However, when you override the currently active theme, the assetHash is not renewed, meaning the assets are cached as the previous version.
Overriding the theme needs to result in a new assetHash.
Due to the enormous and truly beautiful 😍  refactor of our config in Ghost alpha, we need 2 slightly different fixes for this - one for master and one for lts.
I already have the fix located and tested for the lts branch.
The master fix should be nicer, PRs incoming 😸

There's yet another piece to this puzzle :( We also need to refresh the asset hash when a theme is activated.


Firefox not display svg background-image

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[x ] Bug report
[ ] Feature request
[ ] Documentation issue or request

Current behavior
background-image styles that use SVG are not displayed on Firefox.
Expected behavior
background-image styles that use SVG should be displayed on Firefox.
Minimal reproduction of the problem with instructions
Open:
https://stackblitz.com/edit/nb-choices
on Firefox.
Select some item in Multiple select or Multiple select with checkbox.
Selected items are not display remove icon.
Environment
Browser:

[ ] Chrome (desktop) version XX
[ ] Chrome (Android) version XX
[ ] Chrome (iOS) version XX
[ x] Firefox version 59.0.1 (64-bit) on Ubuntu, Window and Mac
[ ] Safari (desktop) version XX
[ ] Safari (iOS) version XX
[ ] IE version XX
[ ] Edge version XX

Others:
I think the issue reason is here:
https://stackoverflow.com/questions/30676125/svg-as-data-uri-triggers-xml-parsing-error-in-firefox

Can you suggest a fix, please?

Because in the SVG image we use colorIndex as a parameter so it is hard to continue with SVG. So at the moment, I use font-awesome to solve the issue, but I think you could use 'x' text to solve this issue.
Sorry for could not give you a better solution.


[1.16.5] No sound and occasional crash

Mod Version:
DynamicSurroundings-1.16.4-<IP_ADDRESS>
Forge Version:
36.0.10
Link to client log:
https://gist.github.com/ocram9/84061eac149df445ebf558c580aa4df1
Link to crash log:
https://gist.github.com/ocram9/813218b36dabcb7527243898b3473177
Description:
After the update to java 15 and forge 36.0.10 there are no dynamic surroundings sounds and the game crashes on occasion. Nohting else changed besides the fact this mod updated to a new java version. It was working fine before.

What type of server config do you have?  The log shows it connecting to an integrated server, you losing connection, and then connecting to a vanilla server.

What type of server config do you have?  The log shows it connecting to an integrated server, you losing connection, and then connecting to a vanilla server.

I don't have any experience nor knowledge of Minecraft modding but if I remember and read the log right, it was just me joining and leaving a singleplayer test world to check something and then reconnecting to the server. I will add that after that crash I played for another hour or so without it crashing so I'm not sure if it was a one-off. I still don't have any sound though.

I don't have any experience nor knowledge of Minecraft modding but if I remember and read the log right, it was just me joining and leaving a singleplayer test world to check something and then reconnecting to the server. I will add that after that crash I played for another hour or so without it crashing so I'm not sure if it was a one-off. I still don't have any sound though.

So the connect to the local single player world worked, and then you tried connecting to a vanilla server running Paper and the crash happened.  What else is running on the server?
What I think may be happening is that during a normal Forge handshake with a server it generates some events, and those events trigger my mod to initialize.  What I think may be happening is those events are not being sent thus my mod does not initialize and it gets into a not-so-good state.

So the connect to the local single player world worked, and then you tried connecting to a vanilla server running Paper and the crash happened.  What else is running on the server?
What I think may be happening is that during a normal Forge handshake with a server it generates some events, and those events trigger my mod to initialize.  What I think may be happening is those events are not being sent thus my mod does not initialize and it gets into a not-so-good state.

I pushed <IP_ADDRESS> to CurseForge with a change that should fix this immediate issue of the crash.  I think the "no sound" part is because I do not get the event triggers that I am expecting.  I need to do some digging on this one.

I pushed <IP_ADDRESS> to CurseForge with a change that should fix this immediate issue of the crash.  I think the "no sound" part is because I do not get the event triggers that I am expecting.  I need to do some digging on this one.

So the connect to the local single player world worked, and then you tried connecting to a vanilla server running Paper and the crash happened. What else is running on the server?
What I think may be happening is that during a normal Forge handshake with a server it generates some events, and those events trigger my mod to initialize. What I think may be happening is those events are not being sent thus my mod does not initialize and it gets into a not-so-good state.

Correct, though I could connect to the server, after a few seconds it crashed. It only happened twice so it's going to be difficult to test if the fix worked. No problems so far though. I have also found out that in singleplayer the sounds are working, just not on my server. Might be a paper compat issue?

So the connect to the local single player world worked, and then you tried connecting to a vanilla server running Paper and the crash happened. What else is running on the server?
What I think may be happening is that during a normal Forge handshake with a server it generates some events, and those events trigger my mod to initialize. What I think may be happening is those events are not being sent thus my mod does not initialize and it gets into a not-so-good state.

Correct, though I could connect to the server, after a few seconds it crashed. It only happened twice so it's going to be difficult to test if the fix worked. No problems so far though. I have also found out that in singleplayer the sounds are working, just not on my server. Might be a paper compat issue?

Actually, I think I know what the problem is. The server doesn't run java 15 I think, that would explain the difference between singleplayer and the server. The downside is is that I can't fix that because I'm not running the java, the host is.

Actually, I think I know what the problem is. The server doesn't run java 15 I think, that would explain the difference between singleplayer and the server. The downside is is that I can't fix that because I'm not running the java, the host is.

OK.  Its related to Paper, not Java.  I made a tweak to optimize processing during server connect, and apparently when connecting to a Paper server it doesn't work.  It does work with a Forge server.  I am going to be making another change to deoptimize until I can figure out a better way to handle.

OK.  Its related to Paper, not Java.  I made a tweak to optimize processing during server connect, and apparently when connecting to a Paper server it doesn't work.  It does work with a Forge server.  I am going to be making another change to deoptimize until I can figure out a better way to handle.

Pushed <IP_ADDRESS> to CurseForge.

Pushed <IP_ADDRESS> to CurseForge.


Update membership link

Changing link from .html to .md
Type of change

Bug fix

Description
Reading the document I see the link, I clicked ..., and said wow this link don't works, why? And going to the folder I see a .md file and not a .html file and I resolve to change the link. And I changed...
Additional details
Nop
Related issues
Nop

Thank you for the change. We require a DCO signoff on all commits, you will need to checkout your branch locally and do a git commit --signoff --amend --no-edit && git push origin patch-1 -f

I'll also add, aside from reading the doc locally, it's also no meant to be read in GitHub, there are many things that will only render on ReadTheDocs or locally after building with tox


[Question]  firebase_messaging aar missing

[READ] For Firebase Unity SDK question, please report to Firebase Unity Sample
Once you've read this section and determined that your issue is appropriate for this repository, please delete this section.
[REQUIRED] Please fill in the following fields:

Pre-built SDK from the website or open-source from this repo: (https://firebase.google.com/download/cpp)
Firebase C++ SDK version: 8.8.0
Main Firebase Components in concern: General
Other Firebase Components in use: General
Platform you are using the C++ SDK on: Windows/Linux
Platform you are targeting: Android

[REQUIRED] Please describe the question here:
Hi,
I'm trying to add messaging on my android project but just realised the aar files are not generated for firebase_messaging. The messaging module is enabled in the CMake configurations. The static libraries (*.a) for messaging are correctly built but no trace of aar files. Am I missing something ?
Note all other modules are working correctly only messaging is making the app crash at boot due to missing aar in apk.

So, the messaging aar is built through gradle, specifically this file: https://github.com/firebase/firebase-cpp-sdk/blob/main/messaging/messaging_java/build.gradle
We define a dependency on it here: https://github.com/firebase/firebase-cpp-sdk/blob/main/messaging/build.gradle#L87, which should trigger that build.  How are you hooking into the build?


Add Ecole 19

s19 is a new addition to the school 42 (www.42.fr ; #102) family in Brussels
Students email are in the format<EMAIL_ADDRESS>

@Williambraecky Pull request merged. Thank you!


Create master database

Database for components, pcbs, projects, software, to help with version control.

Done.