Improved otel_metrics_source documentation

Is your feature request related to a problem? Please describe.
In Data Prepper v1.5.1 the otel_metrics_source cannot be used with the string_converter plugin.
Describe the solution you'd like
Documentation on plugin/feature compatibility.
Describe alternatives you've considered (Optional)
Enhance pipeline validation on startup to detect incomatibility issues.
Additional context
Related community forum question
pipelines.yaml
entry-pipeline:
  source:
    otel_metrics_source:
      ssl: false
  sink:
    - stdout:

otel-collector-config.yaml
receivers:
  prometheus:
    config:
      scrape_configs:
        - job_name: 'data-prepper'
          scrape_interval: 10s
          metrics_path: "/metrics/prometheus"
          static_configs:
            - targets: ['host.docker.internal:4900']

exporters:
  otlp/2:
    endpoint: data-prepper:21891
    tls:
      insecure: true

service:
  pipelines:
    metrics:
      receivers: [prometheus]
      exporters: [otlp/2]

prometheus.yaml

docker-compose.yaml
version: "3.9"

services:
  data-prepper:
    restart: unless-stopped
    image: opensearch-data-prepper:2.0.0-SNAPSHOT
    working_dir: /usr/share/data-prepper/
    volumes:
      - ./data-prepper/config/data-prepper-config.yaml:/usr/share/data-prepper/config/data-prepper-config.yaml
      - ./data-prepper/config/pipelines.yaml:/usr/share/data-prepper/pipelines.yaml
      - ../../shared-config/log4j2.properties:/usr/share/data-prepper/config/log4j.properties
    ports:
      - "2021:2021"
  otel-collector:
    restart: unless-stopped
    image: otel/opentelemetry-collector:0.40.0
    command: ["--config=/etc/otel-collector-config.yml"]
    volumes:
      - ./opentelemetry-collector/otel-collector-config.yml:/etc/otel-collector-config.yml
    depends_on:
      - data-prepper
    ports:
      - "4317:4317"
  prometheus:
    restart: unless-stopped
    image: prom/prometheus
    volumes:
      - ./prometheus/prometheus.yaml:/etc/prometheus/prometheus.yaml
    ports:
      - "9000:9000"
    depends_on:
      - otel-collector

I recommend using the smoke-tests docker-compose.yaml as a base for setup.

@sbayer55 the copy_values processor also facing the same problem. Do you have workarounds for creating a custom field and assign value from existing field?


Hovering over  in .table-hover only highlights first spanned row

If i have a table with rowspan in some td i get a strange behavior in the hover, only the first row is highlighted.
You can see the behavier in this image where i am hovering over the first cell in the first row

The code i am using is here https://jsfiddle.net/tja6gtz1/
If i am missing something, let me know. Thanks in advance

This is simply how the tr selector works in CSS. It affects the children of the tr rather than the members of the "logical rows" established by using rowspan. AFAIK, there's no non-super-hacky way to select such "logical rows" using CSS, so this is the best we can do.
CC: @mdo for confirmation

Yup, that sounds right to me.

hi guy,
please ref this url https://codepen.io/paulofsis/pen/xfHtC


Problem using fit route with specified format

Hello,
we are trying to use Nik4 for exporting map pdfs witih our custom map style. Using fit route together with a specified format (in this example Din A4) we are not getting the correct mearurements:
nik4.py -a 4 --padding 10 --fit route --add-layers route /home/osm/gs_sommer_web/project.xml /var/www/osm/test.pdf --fonts /usr/share/fonts
This resulted in a pdf with the size 374,3 x 264,6,instead of 297 mm x 210 mm (landscape)
Can you help us with this issue? Is there any parameter missing?
Thank you so much for your assistance

Thanks for reporting this. Alas, it's a problem in underlying Cairo/pdf renderer, and I cannot adjust pdf page size afterwards with this script, as I do for SVG with svg-resize. Proportions should be correct though: if you scale the pdf to A4, you would get 10 mm padding and all that.


Monotonic ID generation randomness increment variability

The randomness increment code specifies a RANDOM_MAX and doesn't let the random value exceed this value.  However, this means that in a given millisecond, there are more or fewer available IDs depending on where in the range of random values was generated (eg. if the range of random values was 1 - 1000, and you generated the first ID that millisecond at 999, you would only have 1 more ID available before running out).
My suggestion is that the random incrementer should just wrap around to 0 in this case.  There's not really a need to worry about collision unless 80 bits-worth of IDs are requested in a given millisecond, in which case we'd have an issue regardless of the algorithm because the whole space would already be allocated.

Thank you, fixed in v0.2.1.


Temporary solution for hardcoded className

This is a temporary solution for hardcoded className in keyboard handler.

Previously, className have to only contain phone_keyboard and nothing else. So if the element contained also other classes, it won't be handled.
It now checks if className includes phone_keyboard so element can now also use other classes.

A complete solution would be to completely drop className check and instead check if the element is a child element of screen container. This would enable handling all elements in the screen container.
This can be done with screen_element.contains(e.target). The reason why I didn't use this in this PR is that screen_element isn't available in the keyboard adapter. To fix this, adapters need to be modified to accept and use V86Starter options which contain screen element.

Note: The failing travis-ci build doesn't appear to be related to any changes in this PR, but rather the new version of closure-compiler not liking something that's already in master.

From travis-ci build logs
src/browser/speaker.js:19: ERROR - [JSC_INEXISTENT_PROPERTY] Property webkitAudioContext never defined on Window
    if(!window.AudioContext && !window.webkitAudioContext)
                                       ^^^^^^^^^^^^^^^^^^

Note: The failing travis-ci build doesn't appear to be related to any changes in this PR, but rather the new version of closure-compiler not liking something that's already in master.

I pushed a fix for this to master.


Updated the reusables

Why:

This pull request fixes part of #2919
Closes [issue link]

What's being changed:
The write up is being changed to a new route

Check off the following:

[x] I have reviewed my changes in staging. (look for the deploy-to-heroku link in your pull request, then click View deployment)
[x] For content changes, I have reviewed the localization checklist
[x] For content changes, I have reviewed the Content style guide for GitHub Docs.

I just wanted to send this in first before I fix the remaining routes because I am not sure the screenshot I used is valid.
cc: @janiceilene and @megbird

👋  @lumenCodes thank you so much for opening this PR! I added a few clarifying points over in this comment https://github.com/github/docs/issues/2919#issuecomment-784556744 if you want to take a look and update your PR 😄  let me know if you have any questions.


[COM-35] Management of VPCs with System Initiative

We need to be able to build systems on top of VPCs rather than always relying on the default vpc. To do this, there are a number of core resources that we need to be able to build:

[ ] VPC
[ ] Subnet
[ ] Internet Gateway
[ ] RouteTable
[ ] Route
[ ] RouteTableAssociation (these will not be actual resources but will be the edges between the components RouteTable and Route)
[ ] NatGateway
[ ] VpcEndpoint
[ ] FlowLog
[ ] Iam Role / Policy
[ ] Cloudwatch Log Group

Ideally we should be able to create a VPC that has 3 public and 3 private Subnets and is able to route the traffic for the private subnets through the natgateway to the internet
COM-35

oooh. one step closer to transit gateways ;)

I will ✅ these off as I finish and test the modules and add them to our module-index


mWeb/Safari -Change password - Cursor does not autofocus Change password page

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to https://staging.new.expensify.com/

Navigate to settings by clicking on the user avatar over LHN

Click on Security > Change password

Expected Result:
The cursor must be present in section Change password
Actual Result:
Cursor is not autofocused  in section Change password
Workaround:
unknown
Platform:

Where is this issue occurring?

Mobile Web - iOS Safari

Version Number: 1.2.23-7
Reproducible in staging?: y
Reproducible in production?: y
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:
https://user-images.githubusercontent.com/43996225/199808618-27b80060-f0af-4ca2-899a-bb5b6cc93d93.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause internal team
Slack conversation:
View all open jobs on GitHub

This one is already a known issue for IOS safari and has been discussed before and i guess decided not to do anything about it or remove focus on mWeb.
cc: @parasharrajat

Correct both of you. @kavimuru Could you please mention this issue on https://github.com/Expensify/App/issues/10414 and close this one. Thanks.

Closing this issue as this can be linked with #10414


Fonticon color matching theme

I try to use superqt.fonticon in napari widget. I observe that icons are dark when enabled. It does not look best when selecting a dark theme.
Is there a way to such an icon automatically match the text color for the current qss? Or does the QSS for the theme need to be updated to handle icons?

I think you'd want to use the setTextIcon function in that case (rather than using the icon() -> Qicon function).

Color works...

The problem is caused because the widget has not yet been added to the layout.
Of course, I could workaround it by using a hardcode size of the font, but there should be a better way.

hey @Czaki, I'm thinking about this a bit more, for both fonticon and iconify QIcons.  One thing we can do here is to install an event filter on the object that listens for PaletteChange events (and, optionally, parent change events) and then recolors the icon.  Fortunately, if you set the stylesheet on a widget to change the background color (like napari's theme for example) it will emit a palette change event and change the palette background in most cases.  So this is likely compatible with napari's stylesheet theming approach too.
Stripped of all the error catching conditionals it would look something like this:
class PaletteEventFilter(QObject):
    def eventFilter(self, obj: QObject, event: QEvent) -> bool:
        """Change icon color when palette changes."""
        if event.type() == QEvent.Type.PaletteChange:
            pal = (
                obj.palette() if hasattr(obj, "palette") else QGuiApplication.palette()
            )
            new_color = pal.color(QPalette.ColorRole.ButtonText)
            new_icon = self.getNewIcon(obj, new_color)
            obj.setIcon(new_icon)
        return False

    def getNewIcon(self, obj: QObject, color: QColor) -> QIcon | None:
        """Return an instance of QIcon suitable for obj using `color`."""

That event filter could be installed on any QObject that supports icon(), setIcon().  The getNewIcon method would depend on the object and exactly how the icon should be recolored.
Want me to put together a more complete proposal like that?

It looks promising. I do not know about this event.


Enable integration with Google Analytics

To enable tracking usage... and get data on traffic, generate ideas on what other content needs to be added.

Note that the Docsy theme has a hook for adding a Google Analytics ID. See issue #291.

/area docs-ux

/assign

/close


SQL Server Json is documented, but does not exist

Confusion around about our Json support for SQL Server:
prisma.io/docs/reference/api-reference/prisma-schema-reference#json
prisma.io/docs/reference/api-reference/prisma-schema-reference#microsoft-sql-server-7
But:
https://github.com/prisma/prisma/issues/7417 (I confirmed this manually)
https://github.com/prisma/prisma/issues/6843
It seems to not actually be implemented.
Why do we document otherwise?
(Our SQL Server page is not helpful here: prisma.io/docs/concepts/database-connectors/sql-server#type-mapping-between-microsoft-sql-server-to-prisma-schema)

@janpio as JSON is natively supported for a while now (back to MSSQL 2016), will there be an update to Prisma?
https://learn.microsoft.com/en-us/sql/relational-databases/json/json-data-sql-server?view=sql-server-ver16

@itpropro hey there! This is an issue around our documentation, not the engine of Prisma ORM. The issue around support for JSON in Prisma ORM can be found here: https://github.com/prisma/prisma/issues/7417

@itpropro hey there! This is an issue around our documentation, not the engine of Prisma ORM. The issue around support for JSON in Prisma ORM can be found here: prisma/prisma#7417

Thanks for your reply @jharrell, but that issue is already open for over 3 years and there doesn't seem to be a status on that. Any chance you or someone else from the prisma team can give a statement on the two most important MSSQL issues, which are Managed Identity/Token support (#13853) and JSON support (#7417)?
Thank you!

@itpropro hi again!
Unfortunately, I don't have much information on those issues. I do not think they are planned for at least the coming quarter, perhaps longer.

@itpropro hi again!
Unfortunately, I don't have much information on those issues. I do not think they are planned for at least the coming quarter, perhaps longer.

That's quite sad to hear, but thanks for coming back anyway!


Enhancement: Add placement type of overlay

It would be nice to specify the placement to be an overlay of the target dom element placed in the top left corner.
I am using it as a popover to edit a piece of information and would like it place right over the element I requested.
See the small edit dialog section of: [https://material.io/guidelines/components/data-tables.html#data-tables-interaction] for an example of what I mean.

Have you tried the inner modifier?


FEATURE REQUEST: make it optional to interact with the mouse outside the focus-trap

We want to keep de keyboard tabs within the focus-trap but when a notification pops up the user should be able to interact with the notification with the mouse.
We don't have the need to have the clicks blocked outside the focus-trap. Would be nice if this is optional

Hi @PieterSwitten. This has already been discussed a number of times, and there have been a few abandoned PRs. Please search the issues and PRs to find the backstory, then open a new issue if you have a new idea or a PR if you're able to implement something. Thanks.

I have a dropdown component, which, when opened, will attach a "click away" listener to the document to close the dropdown when the user clicks outsides of the dropdown.  The FocusTrap seems to be blocking this click event from happening.  How can I implement this behavior without an option to disable the FocusTrap from interfering with the click events?

I just did some more digging and found the clickOutsideDeactivates option, that works, nevermind!
<FocusTrap
  active={isOpen}
  focusTrapOptions={{ clickOutsideDeactivates: true }}
>


Create feature: add new comment

Implement the "add new comment" form on the Comments pop up according to the wireframe:

Hi @bhushan354 ,
Your project is complete! There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉

Highlights
✅ Correct use of Github workflow
✅ No linter error
✅ Popup displaying
✅ Comments adding
Optional suggestions
N/A
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


Getting Started with Multi-user Isolation

https://www.kubeflow.org/docs/components/multi-tenancy/getting-started/
kustomize/centraldashboard/base/parama.env should be common/config-kubeflow/params.env

Activity


🐛Bug: For tooltip, ignore delays when on a mobile device

Describe the bug
When on a mobile device, tapping the tooltip waits the specified delay, but should show it immediately.
To Reproduce
Steps to reproduce the behavior:

Go to https://www.melt-ui.com/docs/builders/tooltip
Tap on the example
Tooltip is shown after the delay.

Expected behavior
Tooltip should be visible immediately when tapped by a user on a mobile device.
Smartphone (please complete the following information):

OS: iOS16
Browser: safari
Version unknown

Actually, tooltips should not work on mobile devices. For those, Popover should be used.

@TGlide
I would encourage you think on this more though, and not follow Radix here.
1.) Responsive websites are the de facto standard, 2.) it's not particularly useful as an element with that intentional limitation, 3.) it's current behavior is almost perfect for desktop and mobile already--just needs the delay tweaked on mobile, and 4.) having gotchas in Melt elements that make them inconsistent from other elements will hinder the DX--at the very least, it needs a warning "Use only on non-touch devices", otherwise devs aren't going to anticipate this intentional limitation unless it's noted.

@TGlide
I would encourage you think on this more though, and not follow Radix here.
1.) Responsive websites are the de facto standard, 2.) it's not particularly useful as an element with that intentional limitation, 3.) it's current behavior is almost perfect for desktop and mobile already--just needs the delay tweaked on mobile, and 4.) having gotchas in Melt elements that make them inconsistent from other elements will hinder the DX--at the very least, it needs a warning "Use only on non-touch devices", otherwise devs aren't going to anticipate this intentional limitation unless it's noted.

It's an accessibility concern, rather than just following Radix and other libraries.
From MDN:

The use of the ARIA tooltip role is a supplement to the normal browser tooltip behavior. An example of a native browser tooltip is the way some browsers display an element's title attribute on long mouse hover. One cannot activate this feature through either keyboard focus or through touch interaction, making this feature inaccessible. If the information is important enough to include as a tooltip or title, consider including it in visible text.

I agree though that the docs should clearly specifiy this limitation!

An example of a native browser tooltip is the way some browsers display an element's title attribute on long mouse hover. One cannot activate this feature through either keyboard focus or through touch interaction, making this feature inaccessible

Your quote refers to the title="foo" attribute behavior. The default is inaccessible on mobile.
But Melt's tooltip is not implemented as merely a title="foo" attribute, and consequently it does not have this limitation. In fact, it is accessible to mobile users already, which is great. We can just refine the behavior for them by eliminating the delay.

An example of a native browser tooltip is the way some browsers display an element's title attribute on long mouse hover. One cannot activate this feature through either keyboard focus or through touch interaction, making this feature inaccessible

Your quote refers to the HTML title="foo" attribute behavior, which yes, is inaccessible on mobile.
But Melt's tooltip is not implemented as merely a title="foo" attribute, and consequently it does not have this limitation, and doesn't need to be artificially limited to the limited behavior of a native title="foo" attribute. In fact, it is accessible to mobile users already, which is great; we can just refine the behavior for them by eliminating the delay on tap.

By reading more into what I quoted, it even goes against me. It says the title attribute is inaccessible, not that the tooltip being activated by touch is.
The WAI-ARIA guidelines do not say anything about touch, but do say about receiving focus... I think you've convinced me. Will think a bit on this.

Linking the #279 PR as a resolution.

@TGlide No preference for tooltip vs popover. My goal was to help test MeltUI for DX/UX and avoid unexpected behaviors.

In this issue, the button is both a tooltip trigger and a menu trigger. Which means, you'll never be able to see the tooltip in mobile, due to it corresponding to an action that moves focus outside of the tooltip trigger.

I researched this more and it seems super unclear.

I can see how you'd read MDN to say tooltip should use hover and popover should require intentional interaction.
W3C Aria's tooltip pattern "is work in progress; it does not yet have task force consensus". But discussions are captured here, which includes  a link to 3 janky examples including one that is a button and tooltip--theirs triggers the button and then the tooltip occurs after it's dismissed haha. Their basic example is touch accessible on mobile.

For practical purposes:

Bootstrap's tooltip is accessible on desktop via hover and mobile via tap. To me, that seems intuitive.
Imo, the dev's approach in the linked issue of a button with a tooltip is problematic and not best practice for responsive web design--b/c  it overlays an element with a hover event (tap on mobile) on top of an element with click (tap on mobile) event, this will inherently cause issues for mobile users.

TDLR No clear answer. I lean toward the practical approach primarily b/c seems more intuitive DX/UX, and secondarily b/c W3C's example is touch accessible, but maybe go with whichever you feel is best since it's unclear. and just document the foot guns on the tooltip docs.

Hey @jasongitmail, thanks for your feedback!
I agree that it can be a bit confusing! It did take a lot of research for sure.
Please note we still have an option to have the tooltip work on mobile. By setting closeOnPointerDown to false, if the trigger is focusable, the tooltip will appear, similar to Bootstrap. If you set openDelay to 0, it will work exactly the same as Bootstrap's example.
We'll be also dispatching custom event listeners for every listener we currently have, as well as controlled usage, so the ignorable delays should be achievable. It's just not something I think should be on by default. I thought of having it configurable, but I think the event listeners I mentioned should be more than enough.


$http Angular 1.6 compatability

$http.success() is deprecated, use $http.then(onSuccess(response), onError(response))

Thank you for PR.


Should commands fail-fast like default GitHub action behavior?

Describe the bug
Depending on your perspective, this may not be a bug.  However, if it's not then I think some documentation to make the expected behavior clearer would help users.
Default GitHub behavior for steps with a run command is to fail-fast when supported.  This behavior exists for bash/sh/powershell scripts.
I discovered the retry action while attempting to troubleshoot a step of my GitHub workflow that's susceptible to an external race condition with the Azure CLI where retrying a specific step a few times is a simple way to deal with that race condition.  After reading over the documentation, it seemed like close to a drop-in replacement for my existing run step.  However, the action would never retry when the script encountered the expected failure because the last line of my command was a simple command that always succeeded.
az account set --subscription <some-sub-id>
az command_that_sometimes_fails
az account set --subscription <original-sub-id>

It's not hard to workaround this by adding set -e at the start of my command.  But I was not expecting to have to do this and went through several rounds of adding debug calls and enabling debug logging until I realized it wasn't retrying because it was executing the final line even when the previous line failed.
Expected behavior
IMO, it'd be great if the retry action supported the default GitHub run behavior (even if it's optional / opt-in) where the command fails-fast.  However, if you disagree, then it's probably worth documenting this behavior so that people don't assume it behaves the same as a run step.
Screenshots
n/a
Logs
n/a

You could simply chain the commands with &&.
Though i don't fully disagree, if there is command splitting logic (i saw something like that in the code, just glancing at it though), that is probably the nices behaviour.
Then again, just chaining with && or || depending on your needs should be fine


if condition default value is empty

Describe the bug
the default value of if condition is empty. It should be true or false at least. And I think most of users want to set an expression as condition. So it is better that the condition input box is expression as default

Version

Browser

[ ] Chrome
[ ] Safari
[ ] Firefox
[ ] Edge

OS

[ ] macOS
[ ] Windows
[ ] Ubuntu

To Reproduce
Steps to reproduce the behavior:

add a if condition
check the default condition value

Expected behavior

Screenshots

Additional context

@vishwacsena, @tomlm this is a P0 that's been sitting for 12 days. How can we drive this forward?

added default : "true" for conditions


chore: Remove link locales

@nschonni : Thanks for your contribution! The author, @KarlErickson, has been notified to review your proposed change.

Verified typo fixes, ready to merge.


Polish failure message from DaemonClient

When we fail to send a message to the newly started daemon, explicitly say that this is unexpected and adjust message to identify long, technical output as "diagnostics" instead of "pid".
Reviewing cheatsheet
Before merging the PR, comments starting with

❌ ❓must be fixed
🤔 💅 should be fixed
💭 may be fixed
🎉 celebrate happy things

@bot-gradle merge


[Feature Request] Build Carpe installers on Github Actions

🚀 Feature Request

Motivation
Accelerate the process of publishing new releases.
Pitch
Today it is only possible to build installers on a Mac OS.
Linux and Windows developers should be notified if building installers are failing asap.
Adding this task to GitHub Actions would be great!

[x] Windows
[x] Mac
[ ] Ubuntu

done on 1.0.0


Message deletions not synchronized on server

Expected behaviour
When the "Sync server deletions" setting is checked and the "When I delete a message" setting is set to "Delete from server", I would expect messaged that are deleted through K-9 Mail to be also deleted on the server.
Actual behaviour
Messages deleted through K-9 Mail still appear in the server's inbox.
Steps to reproduce

Create an IMAP account
Make sure the  "Sync server deletions" setting is active
Make sure the "When I delete a message" setting is set to "Delete from server"
Delete a message from K-9 Mail
Check to verify if it was also removed from the server through another mail client (ie. Thunderbird) or webmail

Environment
K-9 Mail version:5.008
Android version:6.0.1
Account type (IMAP, POP3, WebDAV/Exchange): IMAP
Log message
SSLException: Write error: ssl=0x977e7680: I/O error during system call, Broken pipe
K9-Mail version: 5.008
Device make: LGE
Device model: Nexus 5
Android version: 6.0.1
com.fsck.k9.mail.MessagingException: IO Error
at com.fsck.k9.mail.store.ImapStore$ImapFolder.ioExceptionHandler(ImapStore.java:2208)
at com.fsck.k9.mail.store.ImapStore$ImapFolder.appendMessages(ImapStore.java:2072)
at com.fsck.k9.controller.MessagingController.processPendingAppend(MessagingController.java:2117)
at com.fsck.k9.controller.MessagingController.processPendingCommandsSynchronous(MessagingController.java:1987)
at com.fsck.k9.controller.MessagingController.synchronizeMailboxSynchronous(MessagingController.java:975)
at com.fsck.k9.controller.MessagingController.access$400(MessagingController.java:111)
at com.fsck.k9.controller.MessagingController$8.run(MessagingController.java:934)
at com.fsck.k9.controller.MessagingController.run(MessagingController.java:435)
at
java.lang.Thread.run(Thread.java:818)
Caused by: javax.net.ssl.SSLException: Write error: ssl=0x977e7680: I/O error during system call, Broken pipe
at com.android.org.conscrypt.NativeCrypto.SSL_write(Native Method)
at com.android.org.conscrypt.OpenSSLSocketImpl$SSLOutputStream.write(OpenSSLSocketImpl.java:771)
at java.io.BufferedOutputStream.write(BufferedOutputStream.java:174)
at java.io.FilterOutputStream.write(FilterOutputStream.java:129)
at com.fsck.k9.mail.filter.EOLConvertingOutputStream.write(EOLConvertingOutputStream.java:21)
at java.io.FilterOutputStream.write(FilterOutputStream.java:114)
at com.fsck.k9.mail.filter.Base64OutputStream.flush(Base64OutputStream.java:149)
at
com.fsck.k9.mail.filter.Base64OutputStream.flush(Base64OutputStream.java:165)
at com.fsck.k9.mail.filter.Base64OutputStream.close(Base64OutputStream.java:180)
at com.fsck.k9.mail.store.LocalStore$BinaryAttachmentBody.writeTo(LocalStore.java:4082)
at com.fsck.k9.mail.internet.MimeBodyPart.writeTo(MimeBodyPart.java:141)
at com.fsck.k9.mail.internet.MimeMultipart.writeTo(MimeMultipart.java:89)
at com.fsck.k9.mail.internet.MimeMessage.writeTo(MimeMessage.java:458)
at com.fsck.k9.mail.store.LocalStore$LocalMessage.writeTo(LocalStore.java:3616)
at com.fsck.k9.mail.store.ImapStore$ImapFolder.appendMessages(ImapStore.java:2016)
... 7 more

From the trace this looks like IMAP, not WebDAV/Exchange
I think we need to be slightly more intelligent and less prone to erroring out on basic IO disconnection behaviour - poor mobile signal is not a K-9 error.

Thank you. This is indeed an IMAP account on an Exchange server. I rectified this mistake in my original post.
The errors also occur on a good WiFi connection, so I do not think this is a connectivity issue.

 It's possible the server is closing the connection once it realizes the message is larger than allowed. I believe we're currently not handling that case very well and it could lead to this error.
If an upload is failing all other commands in the queue won't be sent to the server. That would explain why messages are not deleted from the server.

No, most of the messages are simple ones with no attachements. It happens on every message on that account when I delete them one by one or in a bulk.
I have other IMAP accounts on which those errors do not occur.

We should probably be a bit better at logging information about the message that failed to synchronize.

Sorry, I guess I should have explained better. The error message you posted shows that a message upload fails. Messages are only uploaded in two cases:

when saving a draft
when uploading a copy of a sent message to the 'Sent' folder

It is that operation that fails. And when that happens all other operations in the 'command queue', e.g. marking a message as read or deleting a message, are skipped. So what you're seeing is only a symptom of the upload issue. Chances are deleting messages works just fine, it's only that K-9 Mail doesn't get to the delete commands because there's a failing upload command stuck in the command queue.
What would help us are answers to the following questions:

Are there any messages in K-9 Mail's 'Drafts' folder that are not yet on the server?
Are there any messages in K-9 Mail's 'Sent' folder that are not yet on the server?

If so, please check if there's anything odd about them. Large attachments, missing recipient, etc.

Thank you for your help.
I have no messages in the 'Drafts' folder, but I have way too many messages in the 'Sent' folder to be able to easily check if they are all on the server.
Is there a way to know what is in the command queue to simplify the search for the stuck command ?

If you look at the verbose logging you might get some info, but having seen this for other mails it's pretty impenetrable. However, we need to fix #1124 to help really.

Thank you philipwhiuk and cketti for your help.
The problem seems to have vanished with no particular action on my end. It thus seems to have been a command stuck in the queue as suggested by cketti.
I restarted my phone a couple of times for another separate issue unrelated to K-9, which might be the reason behind the fix.
Thank you again for all your help.
Best regards.

Please create a new issue if this is still a problem.


Missing position absolute in minified js

The minified js is missing position absolute for the div wrapper of the icon (in line 99 of the unminified version).

icon labels are not placed correctly when i use minified version

I get some odd positioning of the labels inside. The main SVG vector paths such as the SQUARE_PIN is positioned well, but all labels inside appear in a straight vertical line.
Checked CSS, apart from box-sizing: border-box, there's no other CSS overriding it at all.

I get the same vertical align error with the minified version as well. Even after building


Add social media sharing buttons and popover to Changelog

Add additional social share buttons to the Changelog to match the social icons that will be available in the Migration Advisor.

Viewable: https://developer-portal-git-social-s-2c454a-sitecoretechnicalmarketing.vercel.app/changelog
How Has This Been Tested?
Manually tested
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[X] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation update (non-breaking change; modified files are limited to the /data directory or other markdown files)

Checklist:

[X] I have read the Contributing guide.
[X] My code/comments/docs fully adhere to the Code of Conduct.
[X] My change is a code change.
[ ] My change is a documentation change and there are NO other updates required.
[ ] My change has new or updated images which are stored in the /public/images folder that need to be migrated to Sitecore DAM

Approved in Teams messages.


Your code is not worthy of being called "God"

By the way, my "God" is an Edward Witten inspired, Supersymmetric learning algorithm.

Gods ought to be learning algorithms, not basic code as you currently have in your repo.
Notably I am an atheist, so I observe that Gods as defined within the realm of artificial general intelligence ought not to be worshipped and ought to be atheism compatible!!

I also urge you to watch this video https://www.youtube.com/watch?v=LKP-PUAI96U , which is based on the same grounds and  following the same ideas as expressed in 1) and 2).

No human-made coding project is worthy of being called "God". There is only one God, which is the uncreated creator of everything that exists, and He is worthy of all praise.


[ORM] MySQLiConnector does not support connection_collation

Affected Version
SS 4.4.1
PHP 7.3
MySQL 5.6
Description
SilverStripe\ORM\Connect\MySQLiConnector does not support switching connection_collation, which results in Illegal mix of collations error for prepared statements with literal arguments (strings).
Unfortunately, I couldn't find an easy fix for that. I've been trying to play with set names ... collate ..., but that's not working (potentially that may require specific server side settings set to start working).
Here's some people in the internet solving that problem: https://www.php.net/manual/en/mysqli.set-charset.php#121647
Please note, that I set the following variables on the server:

character-set-client-handshake = FALSE or 0
skip-character-set-client-handshake = TRUE or 1

Steps to Reproduce
See tests in https://github.com/silverstripe/silverstripe-framework/pull/9161

@dnsl48  Fixed in https://github.com/silverstripe/silverstripe-framework/pull/9605

It turns out the problem is not with the connection collation, but with what mysqli->get_charset()->collation returns.
For some reason (bug?) it always returns a default collation.
The tests proving the actual connection collation works correctly: https://github.com/silverstripe/silverstripe-framework/pull/9651

I encountered this issue recently when using the AddExistingAutoCompleter on chinese pages.
For my case, it is because I had both the Title of the pages searchable, as well as a Date dbfield searchable.
Since the collation cannot be set on the date field, this throws an error about mismatched collation with '='
I had to remove the date field from searchable as a workaround


Fix common definition pack->channel reference error

The pack AP_00010016 should refer to Side-Left and Side-Right, not Side-Right and Top-Centre as it does currently.
The error is also present in the excel sheet attached to BS.2094-1, but not the BS.2094-1 document itself.

Codecov Report

Merging #53 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #53   +/-   ##
=======================================
  Coverage   88.15%   88.15%
=======================================
  Files         115      115
  Lines        5287     5287
=======================================
  Hits         4661     4661
  Misses        626      626

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 c46570a...8577264. Read the comment docs.

Looks good; I'd rather keep this the same as in other places, but that's unambiguously wrong.


Cannot show full file path for long symbol signature in Go To Symbol

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.63.2
OS Version: Windows 10 Enterprise, Version 2019, OS Build 18363.1801

Steps to Reproduce:

Press Ctrl+T, type something to find symbols, it triggers a drop-down list. Each entry on the list has two parts, left is a matched symbol, right is the file path.
If the symbol part on the left is short enough,  there will be some room on the right to show the file path (usually just incomplete file path). We hover the mouse over the file path, it'll display a pop up to show the full file path as below

If the symbol's signature is too long, there won't be room on the right to show the file path at all. So cannot hover the mouse over the file path, thus no way to tell which one is which as seen below

There should be a way to show  the full file path, for example, change to be able to hover the mouse over anywhere of the entry to trigger the pop up to show the full file path

Notes to myself:
Perhaps once we have https://github.com/microsoft/vscode/issues/138617, then we could have a nice rich visual hover of these symbols?
cc @bpasero @Tyriar

I get a native hover, isn't that sufficient?

I get a native hover, isn't that sufficient?

I don't think the label gets a hover... only detail and description.
Easy fix: give label a hover
More interesting fix: use a custom hover and provide some nice rich visual (I'm thinking similar to the signatureHelp)p


Update README.md color func

Guessing this should match?

Awesome that I missed that. 😂

Ohhh, if I had a nickel for the number of times I've done that...


on_construct and on_destroy for context variables

Is there a way to attach on_construct and on_destroy handlers for registry context variables?

Actually no. What you would expect to receive in the callback exactly?
Just an empty notification that something has been added or deleted?

Yeah that's what I'd expect....
... if it weren't for the fact that constructors/destructors were called. You're too fast for me @skypjack ;)
Existing functionality is fine, idk why I thought you didn't call the constructors. Sorry for the noise <3

Don't worry. You're welcome. :wink:

Hello.
I'm interested in such kind of functionality.
For example, I'd like to store scene-wide physics settings (gravity etc) in a context variable and make my physics system observe changes in the context variable to push it to PhysX.
How can I observe changes in contexts without on_update signal?

@alexander-v your context variable could for example inherit from entt:emitter, so that you can update and publish from that.


Fix storage config file path for store-gateway and compactor

The path for the storage specified while running store-gateway and compactor is not correct.
The file exists in current directory but is expected under /etc/edtor. This results in the containers
quietly failing in the background, which store not showing up in the Query UI

Came here to propose the same fix.
Are these tutorials still updated?


🔌 Plugin: Upgrade Google Analytics plugin for GA4

🔖 Summary
Current Google Analytics plugins uses ReactGA implementation which has been archived by the owner on Jul 10, 2023. Currently Google Analytics (GA4) provides a different/newer JS library (gtag.js) which was introduces after August 2017.
🌐 Project website (if applicable)

ReactGA impl: https://www.npmjs.com/package/react-ga
ReactGA4 impl: https://github.com/codler/react-ga4

✌ Context
Upgrade plugging implementation to use GA4
👀 Have you spent some time to check if this plugin request has been raised before?

[X] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Are you willing to submit PR?
No, I don't have time to work on this right now

Hi @rafaeltuelho, I'm very confused is this not using GA4? https://github.com/backstage/backstage/tree/master/plugins/analytics-module-ga4
It's using react-ga4: https://github.com/backstage/backstage/blob/4dbaf95080aad2b9358839f3e0a613e9b0507830/plugins/analytics-module-ga4/package.json#L40

Hi @awanlin
Oh, I was referring to this one https://github.com/backstage/backstage/blob/master/plugins/analytics-module-ga
I was not aware of the new implementation you just pointed out. So, it seems Backstage has two implementations for the Google Analytics API. In this case, it is worth adding a note in the old readme pointing to the GA4 impl plugin. What do you think?

Sure, if you want to submit a PR to add that feel free 👍


test(payment): INT-4855 increase code coverage on CheckoutcomCustomFo…

What?  INT-4855
Increase CheckoutcomCustomFormFields code coverage
Why?
To increase code coverage
Testing / Proof
Before

### Now

@bigcommerce/checkout. @bigcommerce/apex-integrations

The only things left to test are

optional chaining which i cant test because cant undefined them
not defined _selectedIssuer but it wont happened as handleClick is only trigger when you click on an option so that find will never return an empty object but find returns  T | undefined so thats why its left

@bigcommerce/checkout code owner review please ✨


feat: Introduce EdgeNetwork

PiperOrigin-RevId: 582128758
see b/310749741

Component repository created: https://github.com/googleapis/google-cloud-php-edgenetwork


RBAC role names

This doc seems to be referring to "Network Admin and Database Admin" as names of RBAC roles.
To the best of my knowledge, these are not valid role names. Should they perhaps be referring to the following roles?
Network Contributor
https://docs.microsoft.com/en-us/azure/role-based-access-control/built-in-roles?toc=/azure/virtual-network/toc.json#network-contributor
SQL DB Contributor
https://docs.microsoft.com/en-us/azure/role-based-access-control/built-in-roles?toc=/azure/virtual-network/toc.json#sql-db-contributor
I noticed "SQL Security Manager" also explicitly has the permission required to create service endpoints, so that might be a good one to mention as well.
Microsoft.Network/virtualNetworks/subnets/joinViaServiceEndpoint/action
I would submit an edit myself but I'm not sure what the current content is referring to, and any changes here should be properly validated with PG before specific roles are mentioned.

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

ID: 2eb5097c-fc6e-f0a5-07a6-a84e05aa0bb6
Version Independent ID: 1004e6b8-344f-6fdc-391a-35c540f4bf9d
Content: VNet endpoints and rules for databases in Azure SQL Database - Azure SQL Database
Content Source: articles/azure-sql/database/vnet-service-endpoint-rule-overview.md
Service: sql-database
Sub-service: security
GitHub Login: @rohitnayakmsft
Microsoft Alias: rohitna

@Thank you for the detailed feedback. We are actively investigating and will get back to you soon.

@dcbrown16 This is being assigned to the content author to evaluate and update as appropriate.

Ok thanks

Sounds good @VanMSFT  could you please update the doc with the above changes when you get a chance?

#assign:VanMSFT

Need clarification. Following-up with Rohit.

@dcbrown16 - Hey Dillon, I'm adding the associated roles that we reference as Network Admin and Database Admin in the doc. Thanks for the feedback on this! We really appreciate it.


Homepage Redesign

Fixes #1070

Hey @endigo9740, awesome redesign!
Just did a quick scan and found some minor typos:
Provides an extensive

quickly to create

thorough, accessibility, and implement

@bennymi Thanks Benny, I've requested a few other folks review this as well. I'll aim to implement all changes tomorrow.

Typos and missing words:

The living would provide supplies, such food and water
Shoud be:
The living would provide supplies, such as food and water

This showcases Skeleton's Card, Typography, Chips, and a Divider elements.
Should be:
This showcases Skeleton's Card, Typography, Chips, and Divider elements.

approach to CSS styling. Allowing you to create or extend your own personal design system.
Should probably be:
approach to CSS styling, allowing you to create or extend your own personal design system.

Copy-Feedback:

Uses Typescript by default and provides direct IntelliSense feedback directly within your editor as you build your app.
Can probably use one less direct(ly)?


Adopt vscode.diff and vscode.open for tree commands

Start with the GitFileChangeNode
Part of #2333

@RMacfarlane I just picked a place to start. I figured it would be easier if we just to one or two uses of vscode.diff or vscode.open at a time.

I'm a bit confused about these changes overall, previously the commands for the tree items eventually called vscode.diff or vscode.open, does using them directly as the commands change the UI? Or is the goal to simplify the open commands?

Yes, using vscode.diff and vscode.open directly now let the tree handle the commands in the same way they are handled from th explorer and SCM view. For example alt+click will open to the side. Full context is in this issue, with the decided on solution starting aproximately with this comment: https://github.com/microsoft/vscode/issues/85636#issuecomment-724844387

Even after approval, I won't merge this change until insiders is updated, since it requires changes in the core to add the TreeItem's command to the properties resolved by resolveTreeItem.


Added enable-logs feature

Added a lib named android_logger and added a feature for enabling logging on Android

Thanks for the PR.


Updated S3 location in script-evaluation-test.yml

Okay, we need a comment here. Ideally one that explains the intended structure of the bucket so we don't get this wrong in the future. @zliu41 could you draft something?

I'll add a comment. This is temporary; once the other job is restored we should change it back. The correct S3 path should always be s3://plutus/mainnet-script-dump/.

On a related note, there's a problem with script dump job: if we upgrade the cardano-ledger version and the new version has an incompatible LedgerState, we'll need to re-create the dump from scratch since the job won't be able to deserialize the checkpoint file (which is basically LedgerState). That is why I moved the old event files - I was going to re-create them.
We have a ticket for this: PLT-497. I heard there's a ledger state transition function that converts ledger state from on era to the next. We should talk to Jared about how we can use it to solve the problem.


WIP: Array malloc/free

Here's some old patches I had laying around to avoid using the array versions of new/delete instead of malloc/free that I wrote while trying to solve some CRT issues that I was able to work around in another way in the end.
But I recently realized that these save us 32 bytes, so perhaps it's worth it?
TBH, I suspect that a lot of these allocations can also be turned into constant allocations of reasonable bounds. For instance, I doubt we'll really need to be able to support more than maybe 8 threads for the renderer... probably even less. Such a limit could even be turned into a build-time constant if needed, if we'd want even more flexibility.
Anyway, I thought I'd post them, perhaps someone else has some thoughts here?
Note: these aren't properly tested yet, so please don't merge yet.

32 bytes isn't very much, so it doesn't seem like a great idea to me on the surface admittedly.
However, one thing that this may help with is removing dependencies. I recall having to shim these at one point, and I remember that coming up again when we added support for more recent VS versions. I don't remember if that was an issue and how we got around that if so?

32 bytes isn't very much, so it doesn't seem like a great idea to me on the surface admittedly.
However, one thing that this may help with is removing dependencies. I recall having to shim these at one point, and I remember that coming up again when we added support for more recent VS versions. I don't remember if that was an issue and how we got around that if so?

Yeah, I'm far from convinced about this PR as well.
The work that sparked this was to get things building on newer VS versions, but we've crossed that bridge already. So these patches aren't directly useful anymore as far as I'm aware.

I think I'll just close this now. Things are fine as-is, I think.


allow the Ended action Event to fire as previously it was calling dec…

…line if triggered whilst in a call
I've removed the Timeout event from this as it doesn't seem to be relevant.
Right now, this fixes it for me, with my use case, and decline and end call now work, but I think more work is going to be needed to handle situations where users are accepting and ending calls from the same event.  (e.g., call waiting)

this is for issue #88


JS highlight issues after #50293

Issue Type: Bug

open https://github.com/mishoo/UglifyJS2/blob/9e19e63551907be19c8c6b5b75217d7650504436/lib/compress.js
notice that syntax highlighting starts to go haywire around line 4673

VS Code version: Code 1.24.0 (6a6e02cef0f2122ee1469765b704faf5d0e0d859, 2018-06-06T17:35:40.560Z)
OS version: Windows_NT x64 10.0.14393
Extensions: none

Duplicate of https://github.com/Microsoft/TypeScript-TmLanguage/issues/625

@mjbvz I don't see any change on the release branch for this yet. I assume there will eventually be one?

I’ll pull it in If we have a recovery build with other fixes. Not important enough to force a recovery build on its own imo

@mjbvz There are a few fixes queued already. You can bring this one in.

Pulled in with b7557aa2056a1e46ab460a678e0882ad7a834544


Implement bug/feature bug_long_head_change

Bug: If the snake gets too long, its length inverts and its head is now at the end of its body (instead of the tail being at the end).
Feature: This feature of moving "head-last" allows the rest of the snake's body to overflow through the canvas, thus it easier to handle and it the player will have an easy time staying alive.
Todo:
after the snake getting long

switch the head and tail images
change collision detection so the user only loses when the snakes entire body is outside not just the "front"

also added tail #65


Ctrl+C on cargo run

When I Ctrl+C on cargo run and my Rust program is set up to intercept the signal, the Cargo process stops and returns control to the terminal, even though my Rust program may still be running. I've observed this on latest nightly.
I think it would be best for cargo run to also intercept Ctrl+C when running a program, and ignore it – instead, it should just check regularly to see if the program it's running has exited. This would certainly be a more reliable approach.

I believe this was fixed by https://github.com/rust-lang/cargo/pull/2818, so I'm curious, which version of Cargo were you running?

The very latest.
rustc 1.14.0-nightly (cae6ab1c4 2016-11-05)

On 11 Nov 2016, at 03:23, Alex Crichton<EMAIL_ADDRESS>wrote:
I believe this was fixed by #2818 https://github.com/rust-lang/cargo/pull/2818, so I'm curious, which version of Cargo were you running?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/rust-lang/cargo/issues/3281#issuecomment-259871963, or mute the thread https://github.com/notifications/unsubscribe-auth/AAEF3EWjgpHz-BaBDVCaZwGV79v_2eDdks5q89-ygaJpZM4KvUIT.

Oh are you using rustup? If so then that's the same wrapper problem so this'd actually be a rustup bug at that point.

Ah, fair point. Sorry about that. Do you know if it's already been reported for Rustup?
Sent from my iPhone

On 11 Nov 2016, at 03:32, Alex Crichton<EMAIL_ADDRESS>wrote:
Oh are you using rustup? If so then that's the same wrapper problem so this'd actually be a rustup bug at that point.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

AFAIK no

This still happens on Windows.
Steps to reproduce

Create the following program:// [dependencies]
// ctrlc = "*"
fn main() {
    ctrlc::set_handler(move || {
        println!("Ctrl-C received, but I'm not exiting.");
    });
    loop {}
}

cargo run (where cargo is rustup)
Press Ctrl-C.

Expected result
The program should print Ctrl-C received, but I'm not exiting. and continue running.
Actual result
The program terminates.
Version info
> rustup --version
rustup 1.18.3 (435397f48 2019-05-22)

I tried it without (C:\Users\username\.rustup\toolchains\nightly-x86_64-pc-windows-msvc\bin\cargo.exe run). It works as expected.
Cargo itself used to have this problem before (https://github.com/rust-lang/cargo/issues/4575), but it was solved with this workaround: https://github.com/rust-lang/cargo/pull/6004.


Use mainClass instead of removed main

Fixes #46

Fixed with https://github.com/reyerizo/jcstress-gradle-plugin/pull/48


do_heartbeat (--no-heartbeat) in testing version 3.3.0

Hello, quick question.
Can we hope for the "do_heartbeat" function (--no-heartbeat argument) to be implemented into the "testing" version (currently 3.3.0) of PyInstaLive?
Is there a reason why it didn't get implemented yet (as it is in the master version)?
Thank you in advance.

Because I intend to largely restructure and rewrite the code and it'll be added back when that's done. But I currently don't have any interest in working on this project because I don't personally have a need for it anymore.
The test version has working new authentication that does not rely on an unmaintained API module which was really all I planned on changing but I ended up having to rewrite a bunch of functions anyway to make it work again.
Maybe i'll work on it again when I have a bunch of extra time on my hands but for all intents and purposes do consider this project inactive.

@dvingerh Interesting. PyInstaLive is so, so useful for all those who're monitoring the Instagram livestreams 24/24. The only tool doing the job properly.
Hope you'll find a bit of time for fixing/developping and we'll have the opportunity in the future to see a new pyinstalive_win.exe in the releases but anyway a big thanks to you💯👍

i think its already implemented now, but im not sure

why did you close this?

See https://github.com/dvingerh/PyInstaLive/issues/134#issuecomment-1152977303


Black window on Ubuntu 22.04.1 LTS

Binary package or full compiled application shows briefly the SHADERed logo and then it remains a black window at startup.
No controls or widgets shown.

@papyDoctor
Same problem here.
But you can use SHADERed catharsis mode, works fine:
https://github.com/animaone/shadered-catharsis-mode
flatpak also runs, but for some reason the flatpak version is not saving system variables, so I had to build it in docker.

variables are still not being saved. I don't know why...

My error. :P It is because if declared variables are not used they are automatically pruned. In options there is a way of disabling this, or you just declare them as uniform in the code.
So there is nothing wrong with the flatpak version, you can use it, or just use the docker version.

Same problem here. The current version of SHADERed has a hard dependency on libGLEW.so.2.1 and Ubuntu 22.04 comes with 2.2. I downloaded 2.1 to the SHADERed folder and used that to run the program.
It would be nice if, if possible with libGLEW versioning, it just used any version 2.x.


Fixes nuget-config-file description of globalPackagesFolder  behaviour

Fix for issue https://github.com/NuGet/docs.microsoft.com-nuget/issues/214

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/Schema/nuget-config-file.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Thanks.


Use StartOperation on PageViewTelemetry

I would love to be able to start an operation on a  PageViewTelemetry but right now that is  not possible. I am trying to instrument a desktop application using AI but while I can get the requests and dependency working I cannot track a PageView using StartOperation and have the requests and dependencies as a suboperation of PageView.
For example, given the code below, any subsequent calls are childs of the request, but how can I track a PageViewTelemetry that has a request as a child?

            using (_telemetryClient.StartOperation<RequestTelemetry>(nameof(SayHello)))
            {
                try
                {
                    var greeting = $"Hello {to}";
                    _telemetryClient.TrackTrace($"Sending {greeting}");

                    response = new SomeService(_telemetryClient).Send(greeting);
                }
                catch (Exception exception)
                {
                    _telemetryClient.TrackException(exception);
                }

                return response;
            }

I cannot do using (_telemetryClient.StartOperation<PageViewTelemetry>(nameof(Form1)))
When I go to the AI resource in the Azure Portal I see this:
PageView
Request
=    Dependency
==       Exception
so it is not clear that the request is caused by the pageview. I would like to get this:
PageView
=  Request
====    Dependency
========     Exception

@lmolkova , what would be a right way to track operation that is started manually with PageView telemetry in .Net Code?

Today I tried to manually get this working using OperationId and ParentId but I can't get it right. Is it even supported without using StartOperation or is PageView just not designed for this scenario?

This issue is a little bit old, but we also have need of this behavior. The SDK currently assumes that correlation with PageViewTelemetry will happen naturally because the JavaScript SDK will associate its DependencyTelemetry with the root page view and, as the headers arrive to the server when receiving a request, will be passed into the RequestTelemetry. We actually use the PageViewTelemetry class from the SDK directly in our application and are unable (out of box) to associate dependencies with these.
It looks like it wouldn't be particularly complicated, though it may prefer a different naming convention (since PageViewTelemetry is not considered an OperationTelemetry). All the properties being store in context here in TelemetryClientExtensions already exist in PageViewTelemetry.
As a workaround we may wind up creating something like
class PageViewDecoratorOperationTelemetry : OperationTelemetry
to decorate our PageViewTelemetry and pass all the implementation on to the PageViewTelemetry just so we can correlate, but ideally it would be really nice to not need to work around this detail just to take advantage of the contextual correlation feature.

Actually another note about the incompleteness of PageViewTelemetry in the .NET SDK is that it doesn't automatically generate IDs like other telemetry does; OperationContext has an internal GenerateId method that would need to be run in the PageViewTelemetry constructor; since the context operation will likely always be null, it might be more helpful for the equivalent of StartOperation to do something like the following:
                var operationContext = new OperationContextForCallContext
                {
                    ParentOperationId = pageViewTelemetry.Id,
                    RootOperationId = pageViewTelemetry.Id,
                    RootOperationName = pageViewTelemetry.Name,
                };
                CallContextHelpers.SaveOperationContext(operationContext);

For reference, this is not a newly discussed issue and works correctly as long as your PageViewTelemetry is from the JavaScript SDK. I mentioned this previously and we're now upgrading our components for .NET Core support and are still having the same issue. In #47 I asked if it would be possible to make sure page views were included which I think might have been glossed over, although I do think I better understand why PageViewTelemetry is not an OperationTelemetry, since "Operation" seems to imply "nested operation" (as per this comment by @SergeyKanzhelev).
In a followup issue opened there (#361), approximately 2 years later @jpiyali replied stating

This is actually available already today. For each new page in browser
which further supports my view that PageViewTelemetry, in the .NET SDK, is simply not complete because it isn't commonly used.

I may be willing to submit a Pull Request as I've taken a pretty solid look around the code at this point, but I'd like confirmation that my thoughts so far are accurate. It looks like there's a release coming and I'd really much rather have this functionality in the framework than have to maintain additional workaround code. Since PageViewTelemetry doesn't intend to be nested inside of other operations, rather than StartOperation (which makes the implementation needs much more simple).
I'm proposing a new extension method in OperationTelemetryExtensions that would be consumed like the following:
using (client.CorrelateOperations(pageViewTelemetry))
{
    // some operation that later tracks DependencyTelemetry; e.g:
    return new SomeService(_telemetryClient).Send(greeting);
}

and the implementation, unless someone can identify some flaw in my thinking, would be as simple as:
public static IOperationHolder<OperationTelemetry> CorrelateOperations(this TelemetryClient telemetryClient, PageViewTelemetry pageViewTelemetry)
{
	if (telemetryClient == null)
	{
		throw new ArgumentNullException(nameof(telemetryClient));
	}

	if (pageViewTelemetry== null)
	{
		throw new ArgumentNullException(nameof(pageViewTelemetry));
	}
	var operationContext = new OperationContextForCallContext
	{
		ParentOperationId = pageViewTelemetry.Id,
		RootOperationId = pageViewTelemetry.Id,
		RootOperationName = pageViewTelemetry.Name,
	};
	CallContextHelpers.SaveOperationContext(operationContext);

    return new PageViewOperationHolder();
}

The one smell I don't much like a bout this is that PageViewOperationHolder would implement IOperationHolder<OperationTelemetry> but wouldn't have an actual instance to return from the T Telemetry property, nor would there be any notion of a ParentContext (since PageViews are apparently always the root) when it went to call CallContextHelpers.RestoreOperationContext (as per this line) it would end up passing null. It's technically accurate, in a way, but I wonder if there's much code depending on IOperationHolder that would suffer for these details (it doesn't seem like there should be, since it's really primarily there for the Dispose method). Alternatively, we could just return an IDisposable that did exactly these same steps.

CC @TimothyMothra since it seems you're the main person maintaining the BASE SDK and @lmolkova since you have made changes related to the W3C IDs in recent history. This issue hasn't had much comment history besides myself in recent history, but it's somewhat important to us and I was hoping someone with current responsibility in this area could help me get the ball rolling?

Hi @TheXenocide, yeah our team has been stretched thin recently.
@Dmitry-Matveev, @lmolkova is there a historical reason that PageViewTelemetry doesn't implement OperationalTelemetry?
https://github.com/microsoft/ApplicationInsights-dotnet/blob/5e1092514e8e84677baa706421b6685f1c4c0d6a/BASE/src/Microsoft.ApplicationInsights/DataContracts/PageViewTelemetry.cs#L21
https://github.com/microsoft/ApplicationInsights-dotnet/blob/5e1092514e8e84677baa706421b6685f1c4c0d6a/BASE/src/Microsoft.ApplicationInsights/DataContracts/RequestTelemetry.cs#L23

Just a note that above I said to add an extension method to OperationTelemetryExtensions but I'm in the code and it becomes apparent I meant TelemetryClientExtensions

@TimothyMothra ,@TheXenocide , @Expecho , speaking of historical reason, I think I remembered the origins of AI stopping to consider PageView as operation. Several years ago, monitoring charters for services and desktop/mobile were split between Application Insights and App Center. AI stopped supporting WPF apps where page view was the top-most operation, so our modifications for operation API at any given point after that were not thinking of PageView as an operation that would happen within AI Services SDK.
In the services world, JavaScript SDK defines the client piece and therefore starts the operation that services SDK later picks up from the headers and continues.
If we are still in that mode of Services v. Client (we, mostly, are), adding desktop-related concept into AI Services SDK won't be aligned. However, I do not see how it would hurt either and we should be open to a PR like that, unless others have some concerns. I do not think this will get onto the actual team backlog due to being off-charter, though.

Yeah, that's the impression I had, though it makes the PageViewTelemetry class pretty much pointless to have. There's nothing about it that works on its own for any purpose beyond registering a Name. The OperationTelemetry have mechanisms for automatically calculating duration and correlating and such. I almost wonder why the class exists at all, since it provides some false sense of implementation where there is none, but I don't mean to be overly critical and that's all beside the point.
I've started to hack away at the code trying to get something working, but it looks like the Activity side of OperationTelemetry is getting in my way now. I'm also feeling like the logic is a bit misplaced, making it a little trickier to understand. I've done a fair bit with disposable contexts myself (i.e. CallContext/AsyncLocal) and, perhaps this is colored with bias from those experiences, but it seems to me that a lot of the logic in the TelemetryClientExtensions Start/Stop methods could be better organized into the OperationHolder type as Constructor/Dispose logic. As of right now it feels like there's a lot of mingling between the extension methods and the holder where each is effectively dependent on the implementation details of the other.
My original approach to having a separate path for PageViewTelemetry (i.e. CorrelateOperationTelemetry) feels like it will require more work rather than less and it seems there would be a lot of duplication of logic. Would you have any problem with me making PageViewTelemetry inherit from OperationTelemetry? I think this would make optimal use of the existing code and PageViewTelemetry already implements all the interfaces/members required of OperationTelemetry.
I think the one oddity to consider in this scenario is that, unless I go out of my way to prevent it (which I can do, if you prefer), using StartOperation to create a PageViewTelemetry while inside an existing request/dependency context would cause pageViewTelemetry.Operation.Context to reflect that it was nested inside some other operation, which is atypical. I still have to solve the problem of creating a context for correlating operations with a page view after the original operation has been recorded (since page views can have related dependencies after the initial operation has completed and been tracked), but I think that's much easier to solve than the Activity scenario.

I also have some questions about things that came up along the way:
I'm wondering if you have any guidance for what seems like a bug I ran into with the PublicApiAnalyzers? Even after adding the new public/internal virtual methods to the PublicAPI.Unshipped.txt I still get the same analysis errors, though I can no longer apply the code fix to them. In order to build and run unit tests I've had to disable the package temporarily, though it feels like everything looks right.
I'm also a little confused about the purpose of the ActivityExtensions.TryRun method since it seems all it does is try to load the System.Diagnostics.DiagnosticSource assembly, but since the calling method directly utilizes the Activity class, I'm pretty confident simply using the TelemetryClientExtensions class at all (or any class that has any internal usage of the Activity class) should force the assembly to be resolved and would otherwise fail to execute at all. Is it possible there was more reflection around Activity in the past? It certainly doesn't seem like an optional/loose reference at all, at this point.

So as I look at it, apparently only the lambda expressions use Activity which I think is how TryRun can work the way it does. Feel free to disregard that part.

It's a deprecated project, but we haven't found a better alternative yet.
For compile issues it's usually an access modifier is missing, or maybe a statement didn't get copied to another text file (There's a text file each framework.)
The compile-time errors are often not helpful. Feel free to disable that package if it unblocks you and one of us will help clean it up in the PR.

@SergeyKanzhelev , what were our plans on PageViewTelemetry type and would they contradict or support @TheXenocide 's suggestion?

Would you have any problem with me making PageViewTelemetry inherit from OperationTelemetry? I think this would make optimal use of the existing code and PageViewTelemetry already implements all the interfaces/members required of OperationTelemetry.
I think the one oddity to consider in this scenario is that, unless I go out of my way to prevent it (which I can do, if you prefer), using StartOperation to create a PageViewTelemetry while inside an existing request/dependency context would cause pageViewTelemetry.Operation.Context to reflect that it was nested inside some other operation, which is atypical. I still have to solve the problem of creating a context for correlating operations with a page view after the original operation has been recorded (since page views can have related dependencies after the initial operation has completed and been tracked), but I think that's much easier to solve than the Activity scenario.

Any news? Maybe we can continue creating a (Draft) PR to not loose the momentum gained.

I had started one, but my approach was too simple and the path I go down next is rather dependent on these correlation questions regarding whether it's okay for a PageView to be nested or not.
Another related subject came up around here as we've just had a conversation with @danroth27 regarding Blazor + Application Insights and I've been digging in to see how best we would take advantage of our existing Application Insights investments alongside our Blazor efforts. I found dotnet/aspnetcore#5461 from the Blazor backlog which I'm linking here because I think a solution to the .NET SDK portion of PageViewTelemetry may also be helpful for a pure C# solution for Blazor as well, though perhaps not the only solution (there are some significant differences between Blazor server-side and Web Assembly implementations in regards to telemetry). I'll walk through some of the stories I've considered though, for reference:
In either scenario JavaScript-interop with external libraries could result in Ajax requests, so there is some merit to the existing JavaScript SDK, but this is not the primary usage pattern expected of Blazor applications in general (and often those requests would be to 3rd party service providers which could frequently be configured to either not record telemetry or filter correlation headers).
Blazor Web Assembly currently uses the browser's HTTP stack via JavaScript interop, which may make the JavaScript SDK a viable implementation with a little extra support for its routing system akin to those used with Angular and other JavaScript SPA frameworks. Assuming WASM forever needs to use importObject to access the browser's fetch/XHR stack, this may be a durable enough solution.
Blazor server-side performs all C# operations in an ASP.NET Core 3.1 applications, including any Web API calls performed via HttpClient. The DOM is synchronized and any JS interop is performed across a SignalR-based WebSocket mechanism referred to as a Circuit. All navigation events are available (and primarily managed) server side and any call from C# to JavaScript is a message from the server to the client. In this scenario the ongoing context of a Page View is known to the server more than to the client; navigation events integrating with Application Insights will first be understood on the C# side and any JS-interop to track a page view will be sent from the server to the client and then from the client to AI (which makes the page view come from the actual browser's IP address, but otherwise requires an additional hop to record telemetry). Additionally, whatever HTTP or SQL dependencies are performed by the server should be correlated with the current page view; no HTTP headers will be received after the first request because all subsequent operations will be conducted through the Circuit, so existing middleware to correlate ASP.NET Core Requests with Dependencies will not work out-of-box and all dependencies (as-is) would be root telemetry operations. As per microsoft/ApplicationInsights-node.js#154 it doesn't seem like every event that crosses the Circuit would want to be identified as a request, but ideally any dependency performed on the backend would be associated with the page view. To this end I believe a successful Blazor server-side story requires a C# mechanism to correlate downstream operation telemetry with a page view that isn't received via an HTTP header.

Also, @novotnyllc (or @onovotny - not sure if one is preferable over the other for issue discussion) - it seems like it might be worth looping you into this discussion? I've recently stumbled on your AppInsights.WindowsDesktop library and I'm wondering if you have any experiences with correlating your PageViewTelemetry with outgoing DependencyTelemetry? The previous comment is about our newer investments with Blazor but our existing code-base is aligned around a WPF application. We find being able to associate  Page View->Service Dependency->Service Request->SQL Dependency to be an important aspect of the value we seek from Application Insights. I'm not sure if your library either supports that or if you have much experience with end-to-end telemetry correlation, but I at least thought you might have helpful insight or find interest in this discussion?

I know us Desktop Application people are not the most common use case, but here's a little bit about our environment, where it came from and where we're trying to go with Application Insights. Hopefully it helps put the wider picture into perspective.
In the past we had to implement our own correlation and tracking in all ways since the non-ASP.NET libraries didn't perform any form of automatic dependency tracking (neither HTTP nor SQL) or correlation. Our services were implemented with WCF so we already had to pass correlation across the wire with custom service behaviors. We took advantage of our existing ambient correlation structures (previously based on CallContext and now based on AsyncLocal<T>) in conjunction with some PostSharp aspects we developed to reconstitute context for the current tab ("page view") within event handlers, allowing all downstream WCF calls to correlate their dependency (and pass headers over the wire for the server-side request and subsequent SQL dependencies). All of that code is rather outdated and was implemented just before the correlation story was really worked out. It's all using Guids for IDs instead of W3C IDs with spans and such. We also had to write an aspect to handle our SQL Dependencies since that automatic tracking also didn't exist outside of the ASP.NET world (I think it was the web module that set up all the HttpContext-based correlation and dependency tracking for HttpClient and SQLCommand).
Now that .NET Standard has improved library portability and the official libraries have mechanisms for ambient correlation and automatic tracking of dependencies built in for non-ASP.NET environments (like the WorkerService package) we've been trying to update to the latest and cut all of our outdated code out. We're super close to being able to remove almost everything redundant (we'll keep some nice convenience features, the WPF aspect, some telemetry initializers that incorporate our other diagnostic work, etc.) but page view correlation is an important piece of our story, and Blazor is our next wave client which it feels like will also need attention in this area.

Today PageView is mostly used by JS SDK and semantically it indicates the fact of a page visit. We've been even discussed to remove the duration from it and today it is in properties in JS definition https://github.com/microsoft/ApplicationInsights-JS/blob/c9f43a610ae8a0f2fbd976ef6107c58abf0e3ede/shared/AppInsightsCommon/src/Interfaces/IPageViewTelemetry.ts#L38-L44
PageView also doesn't have a response code of any sort - it doesn't indicate aborted or pages failed to load.
That's said it's not how UI visualizes PageView now. And I understand where the desire of having StartOperation<PageView> desire is coming from.
So we are in the situation when we want to make PageView about visits and promote PageViewPerformance or analogous to be about latency, duration, correlation and results. But with no specific plans.
Would the solution of having an external API work for you? You can create a new class inherited from OperationTelemetry that will serialize itself in a form of a PageView. Should be easy to implement and it will work the way you would expect.

Thanks for the details.
We discussed the possibility of this workaround, but we'd rather avoid the technical debt of maintaining it. We previously maintained our own correlation layer because we adopted before the SDK implementation was complete, which has caused a pretty major overhaul when updating to the latest packages migrating to the latest frameworks. We'd rather avoid doing things like that going forward if we can. Just to entertain a though, though, if we took the approach of making our own telemetry type and wanted to correlate operations with it after the initial operation has completed (which is standard behavior in page views with the JS SDK today) would it seem functional/acceptable to have another custom telemetry type inherit from OperationTelemetry just to restablish the same context id/parent id? Maybe we could use a filter to prevent tracking this operation while still leveraging the existing logic for ambient correlation?
Given that PageView doesn't aim to be what it was before, it seems like it's nothing more than an event. Is there a reason to keep it separate from event besides backwards compatibility? It could just be an event sub-type, for that matter.
Would there be any consideration for PageViewPerformance to be implemented with correlation in the .NET SDK? I'm not getting the impression that PageViewPerformance will be OperationTelemetry either, but if it intends to support duration, status, and correlation perhaps it should be?
Do you have any guidance on what we should do if we were to try to create a pull request that would satisfy our need to have the JS SDK capabilities available in the .NET SDK? Is a pull request for such a thing likely to be accepted?
Also, I've poked around in SignalR and Blazor a bit trying to ponder what might be the right place to integrate with Blazor, both from a Blazor WebAssembly perspective as well as a Blazor server-side perspective (which is a bigger priority for us at the moment). It seems like the WebAssembly version may be able to use the JavaScript SDK well enough, but since server-side performs all navigations and service calls through a SignalR-based WebSocket, it's difficult to see how one might cause all Dependencies to be associated with a common operation id. Do you have any advice on what the best approach might be to establish ambient correlation there? Ideally we would start new operations with new page views by tying in to NavigationManager events (which provides a pretty convenient place to establish that), but even apart from getting page view support out of it, we would hope that all dependencies for that particular Circuit (essentially a living tab) would be correlated with whatever its most recent page view was and it's just not clear how to establish this kind of ambient context. Perhaps we could treat PageViews as Requests instead, though their duration would essentially be "until the user navigates again or disconnects" and even then I think it may still be tricky to keep the ExecutionContext of the Activity or AsyncLocal properly scoped (though that seems more of a concern with finding the right place in Blazor to integrate with).
I know that's a ton of questions in different directions. I appreciate any insight. Thanks for your time.

I haven't tried correlating the requests from a PageView, but I agree that it would be useful to have. I also thought about Blazor and that it would probably be useful to have AppInsights working for those apps as well.
For the Desktop scenario, I'm open to any PR's to implement things before they're available in any base SDK.

@ramthi what's your position on the future of PageView and PageViewPerformance?
I'd say neither will become an operation. Directionally we will move to OpenTelemetry data model where distributed tracing will be represented by Spans and PageView will be moving to the "usage" scenarios.
As of data model for messaging patterns - we are thinking of a few possibilities:

use sessionId field
use OpenTelemetry Events pattern to represent messages
use OpenTelemetry Links pattern to represent non-parent relationships

Depending on specific technology and scenario one of the approaches must work. But again, this is la bit longer term thing.
As of now - I'd advice to implement StartOperation<PageView> outside of this SDK. It is as much supportability burden  on this SDK as on any external library. If not more.

Would the solution of having an external API work for you? You can create a new class inherited from OperationTelemetry that will serialize itself in a form of a PageView. Should be easy to implement and it will work the way you would expect.

I've tried something but it shows up as a custom event with an event name "ConvertedTelemetry". How can I make sure it is seen as a PageView?

@SergeyKanzhelev could you elaborate regarding

Would the solution of having an external API work for you? You can create a new class inherited from OperationTelemetry that will serialize itself in a form of a PageView. Should be easy to implement and it will work the way you would expect.

I've tried something (created a custom telemetry type which inherits from OperationTelemetry   but it shows up as a custom event with an event name "ConvertedTelemetry". How can I make sure it is seen as a PageView?

Maybe @cijothomas can help. I'm not part of this team any longer and the exact extensibility points that are available in SDK to implement this are fading away from my memory.


Flurry crash received in my crashlytics  java.lang.NullPointerException uriString android.net.Uri$StringUri.

I am using the below flurry versions in my app. I have received the below crash.
 implementation 'com.flurry.android:analytics:12.13.0'
 implementation 'com.flurry.android:ads:12.13.0'

FlurryBrowserActivity$1 line 45
com.flurry.android.FlurryBrowserActivity$1.a

Fatal Exception: java.lang.NullPointerException
uriString
android.net.Uri$StringUri.<init> (Uri.java:490)

android.net.Uri.parse (Uri.java:452)
com.flurry.android.FlurryBrowserActivity$1.a (SourceFile:45)
com.flurry.sdk.ads.dj.a (SourceFile:244)

com.flurry.android.FlurryCustomTabsServiceConnection.onCustomTabsServiceConnected (SourceFile:28)

@jesphinpt Flurry Ads is no longer supported and you need to migrate over to Yahoo Mobile SDK for your ad needs. If your use case is analytics only, you can remove the ads dependency.
You can find Yahoo Mobile SDK at https://sdk.yahooinc.com/yahoo-ads/integration.html.


As a user, I want to schedule my cards so that the cards I am weaker at will appear more often in gameplay

I intend to use the Supermemo algorithm to implement this feature: https://www.supermemo.com/english/ol/sm2.htm
We can use four levels instead of five(since we have four buttons) and we can label  each card with an E-Factor. On a daily basis, we can then reorder or hide cards as needed


ECR ReplicationConfiguration not working as expected

What happened?
I am using provider-aws-ecr and having issues with ReplicationConfiguration where when I add repositoryFilter array, I get the following error

CannotObserveExternalResource  38s (x16 over 12m)  managed/ecr.aws.upbound.io/v1beta1, kind=replicationconfiguration  cannot run refresh: refresh failed: Insufficient destination blocks: A │
│ t least 1 "destination" blocks are required.

I am following the ReplicationConfiguration Schema and I should be able to have rule with destination and reposityFilter. However whenever I add respositoryFilter I get the error above.
apiVersion: ecr.aws.upbound.io/v1beta1
kind: ReplicationConfiguration
metadata:
  name: peo-server-repo
spec:
  forProvider:
    region: us-west-2
    replicationConfiguration:
      - rule:
          - destination:
              - region: us-west-2
                registryId: <some-string>
          - repositoryFilter:
              - filter: peo-server-repo-prod
                filterType: PREFIX_MATCH
  providerConfigRef:
    name: aws-provider

How can we reproduce it?
The below does create an ecr repository, but fails to to create ReplicationConfiguration
---
apiVersion: pkg.crossplane.io/v1
kind: Provider
metadata:
  name: provider-aws-ecr
spec:
  package: xpkg.upbound.io/upbound/provider-aws-ecr:v0.40.0
  controllerConfigRef:
    name: aws-config
---
apiVersion: ecr.aws.upbound.io/v1beta1
kind: Repository
metadata:
  name: peo-server-repo
spec:
  forProvider:
    imageScanningConfiguration:
      - scanOnPush: false
    imageTagMutability: IMMUTABLE
    region: us-west-2
    tags:
      team: peo
      env: dev
  providerConfigRef:
    name: aws-provider
---
apiVersion: ecr.aws.upbound.io/v1beta1
kind: ReplicationConfiguration
metadata:
  name: peo-server-repo
spec:
  forProvider:
    region: us-west-2
    replicationConfiguration:
      - rule:
          - destination:
              - region: us-west-2
                registryId: <some-string>
          - repositoryFilter:
              - filter: peo-server-repo-prod
                filterType: PREFIX_MATCH
  providerConfigRef:
    name: aws-provider
---

Note if I remove the repositoryFilter configuration as seen below the ReplicationConfiguration is created and there are no issues.
---
apiVersion: ecr.aws.upbound.io/v1beta1
kind: ReplicationConfiguration
metadata:
  name: peo-server-repo
spec:
  forProvider:
    region: us-west-2
    replicationConfiguration:
      - rule:
          - destination:
              - region: us-west-2
                registryId: <some-string>
          #- repositoryFilter:
          #    - filter: peo-server-repo-prod
          #     filterType: PREFIX_MATCH
  providerConfigRef:
    name: aws-provider
---

What environment did it happen in?
Crossplane version: 1.13

Cloud provider or hardware configuration: EKS m5.large
Kubernetes version: 1.27
Kubernetes distribution: EKS
OS: bottlerocket
Kernel: latest bottlerocket os
-->

You are using the official Upbound provider but this is old provider-aws that is maintained by the community. You want to file a bug at https://github.com/upbound/provider-aws.


[AIRFLOW-1869] Write more error messages into gcs and file logs

Keep more exception messages when writing task logs from gcs.
Also, clean up the file task messages a bit.
Dear Airflow maintainers,
Please accept this PR. I understand that it will not be reviewed until I have checked off all the steps below!
JIRA

[x] My PR addresses the following Airflow JIRA issues and references them in the PR title. For example, "[AIRFLOW-XXX] My Airflow PR"

https://issues.apache.org/jira/browse/AIRFLOW-1869

Description

[x] Here are some details about my PR, including screenshots of any UI changes:
Keep exception strings in the gcs logs for tasks

Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason: This is primarily changing logging information and adds no new code paths.  Current testing should be sufficient

Commits

[x] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Codecov Report

Merging #2826 into master will increase coverage by 0.03%.
The diff coverage is 20%.

@@            Coverage Diff             @@
##           master    #2826      +/-   ##
==========================================
+ Coverage   73.85%   73.88%   +0.03%
==========================================
  Files         160      160
  Lines       12114    12109       -5
==========================================
  Hits         8947     8947
+ Misses       3167     3162       -5

Impacted Files
Coverage Δ

airflow/utils/log/gcs_task_handler.py
0% <0%> (ø)
:arrow_up:

airflow/utils/log/file_task_handler.py
87.05% <66.66%> (-0.9%)
:arrow_down:

airflow/executors/dask_executor.py
79.06% <0%> (-2.33%)
:arrow_down:

Continue to review full report at Codecov.

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

LGTM merged!


tests: net: increase stack size for code coverage

This is a partial fix for #17323
Fixes issues with tests/net/lib/dns_addremove
Signed-off-by: Cami Carballo<EMAIL_ADDRESS>

@camicarballo, Please rebase on the latest master.

dns_addremove tests seem to be failing.


Add note about 1809 and point to workaround

Added note that this does not work on machines running 1809 and there are no plans to fix it. As per
https://github.com/MicrosoftDocs/windows-uwp/issues/1389#issuecomment-457346505

A new version of the SDK that addresses the issue was released last month. There is no need for this change now.
Se updated note at https://stackoverflow.com/questions/53378611/storeservicesfeedbacklauncher-is-not-working/54299697#54299697


Validate classifiers

I'd much rather just have some docs on how to validate with classifiers. I'd prefer not to have a mandatory dependency, and don't want to vendor and have to keep updating, and if the build backend, etc. want to do this, they can decide how they want to include classifiers.

Classifier validation
You can easily validate the classifiers using the PyPI package trove_classifiers, which is the canonical source of trove classifier info.
import trove_classifiers

invalid_classifiers = set(metadata.classifiers) - trove_classifiers.classifiers

# Also the deprecated dict if you want it
dep_names = set(metadata.classifiers) & set(trove_classifiers.deprecated_classifiers)
deprecated_classifiers = {k: trove_classifiers.deprecated_classifiers[k] for k in dep_names)


Fix URL for SendGrid Helper.

Previous link was broken.

Never mind. Closing as duplicate of #301


sharding-jdbc支持多个数据库同时配置读写分离么

Question
For English only, other languages will not accept.
Before asking a question, make sure you have:

Googled your question.
Searched open and closed GitHub issues.
Read documentation: Sharding-Sphere Doc.

Please pay attention on issues you submitted, because we maybe need more details.
If no response more than 7 days and we cannot reproduce it on current information, we will close it.

For English only, other languages will not accept.


Search results page UI design

overall page layout
how to list items
we will use #180 for listing dandisets
search filters
faceted search and display [to be done in a later pass for implementation]

Fixed


adding support for CBDDLP file format

Needed to do this for an older Elegoo Mars printer which would not upgrade to support the CTB file format
Still hopeful that this model can upgrade to handle CTB - already contacted Elegoo Support
Meanwhile, these code changes will successfully run prints for both CTB and CBDDLP formats
Decided to add a new file format definition file, even though it is a copy of CTB - in the future, if other machines out there only supporting CBDDLP show up and require specific offset tweaks, at least the file format is separate from CTB
Unrelated, detected issues operating print sequences on my Mars machine:

Both Print and Print > Pause > Resume sequences work as intended
The Stop sequence doesn't operate as intended on this specific Mars model:

The touch screen resets to the home screen
The fan, LEDs and exposure screen all remain active

To perform a graceful print stop, the following sequence works: Print > Pause > (wait for Pause to finish) > Stop

Z is raised to stop position, LEDs, fan and exposure screen turn off

dealing with gpg signing.....................


Bulkrax improvements

Create admin sets (as necessary) and associate works with admin sets (based upon admin set title)
Transform DOIs into acceptable format in csv preparation script

I'm going to go ahead and merge this and test it out.


 It truely is awful. Just straight up awful.

lol, what else would we use? We can't go back to the old C codegen.

What's awful about it? And what could be better?

I'm just being salty about LLVM breaking compilation of bindings with every small update..

LLVM caved under the might of hell-fire and thus does not need to be killed. Closing issue.

lol


[Backend] Route for adding broadcasts

Add a route for adding broadcasts. Authenticated API
Model for broadcasts need to be added.
Estimated Fields:

Title
Content (long html formatted text)
Link
Timestamps
isExpired
imageURL
category

API Testing is required.
For help on API Testing refer #81

Hi Aman Jha here, I want to work on this issue as part of gssoc'21. But I am not getting what you mean by broadcast. Please clarify and assign it to me so, I can start solving this issue. Thanks

We publish broadcasts(posts) related to several important open source events, internships, etc. We required a db for it so that we can control it via apis
Head over to https://hitk-tech-community.netlify.app/broadcasts to view our current broadcasts(static right now)
Let me know if you have any queries

@amanjha21 Can you please share an update?

@jackfrost13  I am currently working on this issue. I slowed down due to my exams but they are now over I will soon make a PR If everything goes fine else I will DM you.

can you provide a timeframe? Also I believe you must be aware of your exams prior taking this up?
Please don't take up issues when you do not plan to complete it timely. It deprives others from taking the issue and blocks it

Sorry, I will keep in mind from next time.
I will resolve this issue latest by the 22nd.


Operator instance status and version status  are  shown UNKNOWN even after the successfull deployment.

My Operator instance and version status is shown unknown even after the successful deployment. Please let me know how to update the status and version filed of instance.
Thanks in Advance.

Hi @AnilKumarBandi27,
Shows that all worked fine. Note that the info is describing that has no status to show in the Kind database. So, if in this scenario a status should appear then it is in the implementation of the is operator project and not in the SDK tool. In this way, if your questions are related to the expected behaviours of this project please try to contact its maintainers.
However, if you are looking for help to understand better how OLM works and what would be the good practices to work with it then, could you please raise your questions in this repo?
I am closing this one since shows have no relation with the SDK tool project.